]> git.proxmox.com Git - mirror_ubuntu-hirsute-kernel.git/commitdiff
staging: exfat: remove return and error return via a goto
authorColin Ian King <colin.king@canonical.com>
Mon, 2 Sep 2019 09:40:52 +0000 (10:40 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 3 Sep 2019 07:36:07 +0000 (09:36 +0200)
The return statement is incorrect, the error exit should be by
assigning ret with the error code and exiting via label out.
Thanks to Valdis KlÄ“tnieks for correcting my original fix.

Addresses-Coverity: ("Structurally dead code")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20190902094052.28029-1-colin.king@canonical.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/exfat/exfat_super.c

index aaebd40ee6f37483fcaf01308eab69a78ad56319..881cd85cf677abb8fc4cd717eb1b0e63acfe5a23 100644 (file)
@@ -670,7 +670,7 @@ static int ffsLookupFile(struct inode *inode, char *path, struct file_id_t *fid)
        dentry = p_fs->fs_func->find_dir_entry(sb, &dir, &uni_name, num_entries,
                                               &dos_name, TYPE_ALL);
        if (dentry < -1) {
-               return FFS_NOTFOUND;
+               ret = FFS_NOTFOUND;
                goto out;
        }