From 896607d36b43da6d900dd29180d7cff0ea0ab97c Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Thu, 7 Jan 2016 16:47:17 +0200 Subject: [PATCH] Remove a pointer check that can never be true chan is set to the result of pointer arithmetic (pointer to an entry in an array) that can never be NULL. As such, there is no need to check for it to be non-NULL before deference. Remove this check to avoid complaints from static analyzers. Signed-off-by: Jouni Malinen --- src/ap/drv_callbacks.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/ap/drv_callbacks.c b/src/ap/drv_callbacks.c index aad0d81af..9f53660a0 100644 --- a/src/ap/drv_callbacks.c +++ b/src/ap/drv_callbacks.c @@ -956,8 +956,6 @@ static struct hostapd_channel_data * hostapd_get_mode_channel( for (i = 0; i < iface->current_mode->num_channels; i++) { chan = &iface->current_mode->channels[i]; - if (!chan) - return NULL; if ((unsigned int) chan->freq == freq) return chan; }