sta_send_addba: Return 1 (success) in case of no nl80211 support
Previously implementation to return a failure for no nl80211 support or
no vendor command support in the host driver would break the legacy
solution. Hence return 1 for such no support cases to match the earlier
behavior.
This is also aligned with the fallback behavior before the nl80211
support was introduced for this API. The comment in cmd_sta_send_addba()
seems to be clear that the explicit sending of ADDBA is not really
needed.
Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
diff --git a/sta.c b/sta.c
index e1d5b43..2888652 100644
--- a/sta.c
+++ b/sta.c
@@ -6930,18 +6930,18 @@
sigma_dut_print(dut, DUT_MSG_ERROR,
"%s: err in send_and_recv_msgs, ret=%d",
__func__, ret);
+ if (ret == -EOPNOTSUPP)
+ return 1;
send_resp(dut, conn, SIGMA_ERROR,
"ErrorCode,sta_send_addba err in send_and_recv_msgs");
return 0;
}
- return 1;
#else /* NL80211_SUPPORT */
sigma_dut_print(dut, DUT_MSG_ERROR,
"sta_send_addba not supported without NL80211_SUPPORT defined");
- send_resp(dut, conn, SIGMA_ERROR,
- "ErrorCode,sta_send_addba not supported, NL80211_SUPPORT not enabled");
- return 0;
#endif /* NL80211_SUPPORT */
+
+ return 1;
}