nl80211: Fix conditional checks of nlmsg attributes for NAN publish

Some of the cleanup changes had lost the "goto fail" and broken the
logic. Restore correct behavior.

Fixes: 58f04221fdef ("nl80211: NAN USD commands for offloading")
Signed-off-by: Shivani Baranwal <quic_shivbara@quicinc.com>

diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c
index c8ca9f25100a11e31d57a8cfb983f332ed317383..5c2a721d11bdea26049b28183120974a756e4456 100644 (file)
--- a/src/drivers/driver_nl80211.c
+++ b/src/drivers/driver_nl80211.c
@@ -13682,6 +13682,7 @@ static int nl80211_nan_publish(void *priv, const u8 *src, int publish_id,
                        params->freq) ||
            add_freq_list(msg, QCA_WLAN_VENDOR_ATTR_USD_CHAN_CONFIG_FREQ_LIST,
                          params->freq_list))
+               goto fail;
        nla_nest_end(msg, attr);
 
        nla_nest_end(msg, container);