diff options
author | Jouni Malinen <j@w1.fi> | 2018-12-08 14:48:33 (GMT) |
---|---|---|
committer | Jouni Malinen <j@w1.fi> | 2018-12-08 14:48:33 (GMT) |
commit | 22d8bb04d94c2311defc2d3faa0ea9495e2d484f (patch) | |
tree | b17af61bbfb5080fd17b48040060174d4ea79003 /wpa_supplicant/wmm_ac.c | |
parent | 30810f5d9790f6e1a21d37438502caa9006e1d02 (diff) | |
download | hostap-22d8bb04d94c2311defc2d3faa0ea9495e2d484f.zip hostap-22d8bb04d94c2311defc2d3faa0ea9495e2d484f.tar.gz hostap-22d8bb04d94c2311defc2d3faa0ea9495e2d484f.tar.bz2 |
WMM AC: Do not write ERROR level log entries when WMM AC is not in use
These two wpa_printf() calls with MSG_ERROR level could be reached when
connecting without (Re)Association Response frame elements being
available. That would be the case for wired connections and IBSS. Those
cases are not supposed to use WMM AC in the first place, so do not
confuse logs with ERROR messages in them for normal conditions.
Signed-off-by: Jouni Malinen <j@w1.fi>
Diffstat (limited to 'wpa_supplicant/wmm_ac.c')
-rw-r--r-- | wpa_supplicant/wmm_ac.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/wpa_supplicant/wmm_ac.c b/wpa_supplicant/wmm_ac.c index a88cc46..1b78c5f 100644 --- a/wpa_supplicant/wmm_ac.c +++ b/wpa_supplicant/wmm_ac.c @@ -471,13 +471,8 @@ static int wmm_ac_init(struct wpa_supplicant *wpa_s, const u8 *ies, return -1; } - if (!ies) { - wpa_printf(MSG_ERROR, "WMM AC: Missing IEs"); - return -1; - } - - if (!(wmm_params->info_bitmap & WMM_PARAMS_UAPSD_QUEUES_INFO)) { - wpa_printf(MSG_DEBUG, "WMM AC: Missing U-APSD configuration"); + if (!ies || !(wmm_params->info_bitmap & WMM_PARAMS_UAPSD_QUEUES_INFO)) { + /* WMM AC not in use for this connection */ return -1; } |