From ef3ea80c689fd2f842e0c974cdc7cc36e8ce4b59 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Fri, 28 Nov 2014 19:43:58 +0200 Subject: [PATCH] proxyarp: Fix x_snoop multicast-to-unicast debug print l2_packet_send() returns >= 0 on success, i.e., non-zero value does not mean failure. Fix this debug print to show up only on negative return values. Signed-off-by: Jouni Malinen --- src/ap/x_snoop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ap/x_snoop.c b/src/ap/x_snoop.c index 09c621cdd..8f77015ef 100644 --- a/src/ap/x_snoop.c +++ b/src/ap/x_snoop.c @@ -104,7 +104,7 @@ void x_snoop_mcast_to_ucast_convert_send(struct hostapd_data *hapd, os_memcpy(buf, sta->addr, ETH_ALEN); res = l2_packet_send(hapd->sock_dhcp, NULL, 0, buf, len); - if (res) { + if (res < 0) { wpa_printf(MSG_DEBUG, "x_snoop: Failed to send mcast to ucast converted packet to " MACSTR, MAC2STR(sta->addr));