On Tue, Feb 19, 2013 at 03:36:46PM +0100, Antonio Quartulli wrote:

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: Antonio Quartulli antonio@open-mesh.com

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;
  

Applied (git rev 274133d)

Thanks, Simon