Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: fb4ODeR6K4HnUcc5en7EwlOV0CHQel+cdHnQ1wwgm2Ff12wQJFqvaz59NKeRHHlLl/y0rmNlGc
 RAqKM1MlnI9A==
IronPort-SDR: YCM9EXbK/jh/TXTYVnB9ql35sqCQ845r9LoyiK6mYKlUsLwY5MO8wcXm99P/dz9QDI27s6GwGf
 DqUIg5Y4IDJA==
Date:   Sat, 8 Aug 2020 11:35:45 +0800
From:   kernel test robot <lkp@intel.com>
To:     Christoph Hellwig <hch@lst.de>
Cc:     kbuild-all@lists.01.org, linux-kernel@vger.kernel.org
Subject: fs/init.c:72:8: warning: Variable 'error' is reassigned a value
 before the old one has been used.
Message-ID: <202008081143.D8Y8xO2X%lkp@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   30185b69a2d533c4ba6ca926b8390ce7de495e29
commit: 4b7ca5014cbef51cdb99fd644eae4f3773747a05 init: add an init_chroot helper
date:   8 days ago
compiler: hppa-linux-gcc (GCC) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>


cppcheck warnings: (new ones prefixed by >>)

>> fs/init.c:72:8: warning: Variable 'error' is reassigned a value before the old one has been used. [redundantAssignment]
    error = security_path_chroot(&path);
          ^
   fs/init.c:69:8: note: Variable 'error' is reassigned a value before the old one has been used.
    error = -EPERM;
          ^
   fs/init.c:72:8: note: Variable 'error' is reassigned a value before the old one has been used.
    error = security_path_chroot(&path);
          ^

vim +/error +72 fs/init.c

    57	
    58	int __init init_chroot(const char *filename)
    59	{
    60		struct path path;
    61		int error;
    62	
    63		error = kern_path(filename, LOOKUP_FOLLOW | LOOKUP_DIRECTORY, &path);
    64		if (error)
    65			return error;
    66		error = inode_permission(path.dentry->d_inode, MAY_EXEC | MAY_CHDIR);
    67		if (error)
    68			goto dput_and_out;
    69		error = -EPERM;
    70		if (!ns_capable(current_user_ns(), CAP_SYS_CHROOT))
    71			goto dput_and_out;
  > 72		error = security_path_chroot(&path);
    73		if (error)
    74			goto dput_and_out;
    75		set_fs_root(current->fs, &path);
    76	dput_and_out:
    77		path_put(&path);
    78		return error;
    79	}
    80	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org