binderfs: remove redundant assignment to pointer ctx
authorColin Ian King <colin.king@canonical.com>
Thu, 2 Apr 2020 10:50:00 +0000 (11:50 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Apr 2020 14:48:11 +0000 (16:48 +0200)
The pointer ctx is being initialized with a value that is never read
and it is being updated later with a new value. The initialization
is redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Christian Brauner <christian.brauner@ubuntu.com>
Link: https://lore.kernel.org/r/20200402105000.506296-1-colin.king@canonical.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/android/binderfs.c

index 78528e1b7ec14543b89f2241536c95f495e8a8af..7cf566aafe1f4ddb2e56b0533a8733bdcf0a0123 100644 (file)
@@ -747,7 +747,7 @@ static const struct fs_context_operations binderfs_fs_context_ops = {
 
 static int binderfs_init_fs_context(struct fs_context *fc)
 {
-       struct binderfs_mount_opts *ctx = fc->fs_private;
+       struct binderfs_mount_opts *ctx;
 
        ctx = kzalloc(sizeof(struct binderfs_mount_opts), GFP_KERNEL);
        if (!ctx)