mac80211: Remove redundant assignment to ret
authorYang Li <yang.lee@linux.alibaba.com>
Fri, 30 Apr 2021 09:21:23 +0000 (17:21 +0800)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 23 Jun 2021 08:57:55 +0000 (10:57 +0200)
Variable 'ret' is set to -ENODEV but this value is never read as it
is overwritten with a new value later on, hence it is a redundant
assignment and can be removed.

Clean up the following clang-analyzer warning:

net/mac80211/debugfs_netdev.c:60:2: warning: Value stored to 'ret' is
never read [clang-analyzer-deadcode.DeadStores]

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
Link: https://lore.kernel.org/r/1619774483-116805-1-git-send-email-yang.lee@linux.alibaba.com
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/debugfs_netdev.c

index 0ad3860..f7aac89 100644 (file)
@@ -57,7 +57,6 @@ static ssize_t ieee80211_if_write(
                return -EFAULT;
        buf[count] = '\0';
 
-       ret = -ENODEV;
        rtnl_lock();
        ret = (*write)(sdata, buf, count);
        rtnl_unlock();