diff options
author | Yang Li <yang.lee@linux.alibaba.com> | 2021-04-28 12:57:32 +0300 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2021-04-28 23:58:58 +0300 |
commit | 808337bec7366f948663952d8e739eb6c235a90f (patch) | |
tree | 6694563dfd6774ce33bdd7cd4a18b77c2a2d53f4 | |
parent | 65ad85f63b15af6995473724ab8562772db22753 (diff) | |
download | linux-808337bec7366f948663952d8e739eb6c235a90f.tar.xz |
net: tun: Remove redundant assignment to ret
Variable 'ret' is set to zero 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.
Cleans up the following clang-analyzer warning:
drivers/net/tun.c:3008: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/1619603852-114996-1-git-send-email-yang.lee@linux.alibaba.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
-rw-r--r-- | drivers/net/tun.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/tun.c b/drivers/net/tun.c index 36443d506b67..84f832806313 100644 --- a/drivers/net/tun.c +++ b/drivers/net/tun.c @@ -3008,7 +3008,6 @@ static long __tun_chr_ioctl(struct file *file, unsigned int cmd, return open_related_ns(&net->ns, get_net_ns); } - ret = 0; rtnl_lock(); tun = tun_get(tfile); |