From: Jouni Malinen <jouni@qca.qualcomm.com>
Date: Fri, 28 Nov 2014 17:43:58 +0000 (+0200)
Subject: proxyarp: Fix x_snoop multicast-to-unicast debug print
X-Git-Tag: hostap_2_4~998
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=ef3ea80c689fd2f842e0c974cdc7cc36e8ce4b59;p=thirdparty%2Fhostap.git

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 <jouni@qca.qualcomm.com>
---

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));