From: Jouni Malinen <j@w1.fi>
Date: Sun, 17 Apr 2016 15:43:30 +0000 (+0300)
Subject: nl80211: Get rid of unused assignment warning
X-Git-Tag: hostap_2_6~575
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=bd86ea0808f1b1cc573ae81277612577c4651046;p=thirdparty%2Fhostap.git

nl80211: Get rid of unused assignment warning

The os_snprintf() call here cannot really fail in practice, but since
its result was stored into the local variable and not checked, static
analyzers could warn about the unused assignment. Clean this up by
checking the return value.

Signed-off-by: Jouni Malinen <j@w1.fi>
---

diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c
index 0dab4bdb8..7e8d57967 100644
--- a/src/drivers/driver_nl80211.c
+++ b/src/drivers/driver_nl80211.c
@@ -9101,6 +9101,8 @@ static int nl80211_configure_data_frame_filters(void *priv, u32 filter_flags)
 	ret = os_snprintf(path, sizeof(path),
 			  "/proc/sys/net/ipv4/conf/%s/drop_unicast_in_l2_multicast",
 			  bss->ifname);
+	if (os_snprintf_error(sizeof(path), ret))
+		return -1;
 
 	ret = nl80211_write_to_file(path,
 				    !!(filter_flags &