From fb81c0a3d11a90c6c42bdf29a87b1b1a52535130 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Thu, 9 Feb 2017 13:24:26 +0200 Subject: [PATCH] RRM: Merge similar error returns to a single one There is no need to maintain different return paths for STA being completely not present and not authorized, so merge these into a single case. Signed-off-by: Jouni Malinen --- src/ap/rrm.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/ap/rrm.c b/src/ap/rrm.c index 5318b2076..a49fd24d8 100644 --- a/src/ap/rrm.c +++ b/src/ap/rrm.c @@ -370,13 +370,7 @@ int hostapd_send_lci_req(struct hostapd_data *hapd, const u8 *addr) struct sta_info *sta = ap_get_sta(hapd, addr); int ret; - if (!sta) { - wpa_printf(MSG_INFO, - "Request LCI: Destination address is not in station list"); - return -1; - } - - if (!(sta->flags & WLAN_STA_AUTHORIZED)) { + if (!sta || !(sta->flags & WLAN_STA_AUTHORIZED)) { wpa_printf(MSG_INFO, "Request LCI: Destination address is not connected"); return -1;