diff options
| author |   Yang Li <yang.lee@linux.alibaba.com> | 2021-04-30 17:21:23 +0800 |
|---|---|---|
| committer |   Johannes Berg <johannes.berg@intel.com> | 2021-06-23 10:57:55 +0200 |
| commit | 5eae2705004895a9aa917f5df6c1a2da8eeb4fd5 (patch) | |
| tree | eda2a3a5c97f3293e5167155e52c7ab6ace6ef5e | |
| parent | net: wireless: wext_compat.c: Remove redundant assignment to ps (diff) | |
| download | linux-dev-5eae2705004895a9aa917f5df6c1a2da8eeb4fd5.tar.xz linux-dev-5eae2705004895a9aa917f5df6c1a2da8eeb4fd5.zip | |
mac80211: Remove redundant assignment to ret
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>
| -rw-r--r-- | net/mac80211/debugfs_netdev.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/mac80211/debugfs_netdev.c b/net/mac80211/debugfs_netdev.c index 0ad3860852ff..f7aac8955681 100644 --- a/net/mac80211/debugfs_netdev.c +++ b/net/mac80211/debugfs_netdev.c @@ -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(); |