binder: remove redundant assignment to pointer n
authorColin Ian King <colin.king@canonical.com>
Thu, 10 Sep 2020 15:12:21 +0000 (16:12 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 16 Sep 2020 15:30:37 +0000 (17:30 +0200)
The pointer n 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.

Acked-by: Todd Kjos <tkjos@google.com>
Acked-by: Christian Brauner <christian.brauner@ubuntu.com>
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20200910151221.751464-1-colin.king@canonical.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/android/binder_alloc.c

index 910c53b..2f846b7 100644 (file)
@@ -347,7 +347,7 @@ static void debug_low_async_space_locked(struct binder_alloc *alloc, int pid)
         * and at some point we'll catch them in the act. This is more efficient
         * than keeping a map per pid.
         */
-       struct rb_node *n = alloc->free_buffers.rb_node;
+       struct rb_node *n;
        struct binder_buffer *buffer;
        size_t total_alloc_size = 0;
        size_t num_buffers = 0;