From 137b293963c2782e86154ad4c689a686ab0c13c3 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Fri, 21 Aug 2015 20:02:10 +0300 Subject: [PATCH] Remove unnecessary NULL check from LOG_LEVEL handler cmd cannot be NULL here, so there is no need to check it before calling os_strlen(). Signed-off-by: Jouni Malinen --- hostapd/ctrl_iface.c | 2 +- wpa_supplicant/ctrl_iface.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hostapd/ctrl_iface.c b/hostapd/ctrl_iface.c index 835feb052..306c53ca3 100644 --- a/hostapd/ctrl_iface.c +++ b/hostapd/ctrl_iface.c @@ -1992,7 +1992,7 @@ static int hostapd_ctrl_iface_log_level(struct hostapd_data *hapd, char *cmd, } } - if (cmd && os_strlen(cmd)) { + if (os_strlen(cmd)) { int level = str_to_debug_level(cmd); if (level < 0) return -1; diff --git a/wpa_supplicant/ctrl_iface.c b/wpa_supplicant/ctrl_iface.c index 4d4d51ad3..62375e13a 100644 --- a/wpa_supplicant/ctrl_iface.c +++ b/wpa_supplicant/ctrl_iface.c @@ -2168,7 +2168,7 @@ static int wpa_supplicant_ctrl_iface_log_level(struct wpa_supplicant *wpa_s, } } - if (cmd && os_strlen(cmd)) { + if (os_strlen(cmd)) { int level = str_to_debug_level(cmd); if (level < 0) return -1;