Repository : ssh://git@open-mesh.org/alfred On branch : master commit 274133de799a15f9a44f5a919cad8038c57c95aa Author: Antonio Quartulli <antonio(a)open-mesh.com> Date: Tue Feb 19 15:36:46 2013 +0100 alfred: in recv_alfred_packet() add break at the end of the case block In recv_alfred_packet() a "break" statement is missing at the end of a case block and the function is returning -1 instead of 0 even if the packet type was known. Add a break to let the function behave correctly. However this bug is harmless at the moment because the return value of the function is not checked anywhere. Signed-off-by: Antonio Quartulli <antonio(a)open-mesh.com> Signed-off-by: Simon Wunderlich <siwu(a)hrz.tu-chemnitz.de> 274133de799a15f9a44f5a919cad8038c57c95aa recv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/recv.c b/recv.c index 0170695..ffabba5 100644 --- a/recv.c +++ b/recv.c @@ -416,6 +416,7 @@ int recv_alfred_packet(struct globals *globals) case ALFRED_STATUS_TXEND: process_alfred_status_txend(globals, &source.sin6_addr, (struct alfred_status_v0 *)packet); + break; default: /* unknown packet type */ return -1;