From: luo penghao <luo.penghao@xxxxxxxxxx> Subject: sysctl: remove redundant ret assignment Subsequent if judgments will assign new values to ret, so the statement here should be deleted The clang_analyzer complains as follows: fs/proc/proc_sysctl.c: Value stored to 'ret' is never read Link: https://lkml.kernel.org/r/20211230063622.586360-1-luo.penghao@xxxxxxxxxx Signed-off-by: luo penghao <luo.penghao@xxxxxxxxxx> Reported-by: Zeal Robot <zealci@xxxxxxxxxx> Acked-by: Luis Chamberlain <mcgrof@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/proc/proc_sysctl.c | 1 - 1 file changed, 1 deletion(-) --- a/fs/proc/proc_sysctl.c~sysctl-remove-redundant-ret-assignment +++ a/fs/proc/proc_sysctl.c @@ -1053,7 +1053,6 @@ static int sysctl_follow_link(struct ctl struct ctl_dir *dir; int ret; - ret = 0; spin_lock(&sysctl_lock); root = (*pentry)->data; set = lookup_header_set(root); _