Repository : ssh://git@open-mesh.org/alfred
On branch : master
commit 274133de799a15f9a44f5a919cad8038c57c95aa Author: Antonio Quartulli antonio@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@open-mesh.com Signed-off-by: Simon Wunderlich siwu@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;