net/rose: remove redundant assignment to variable failed
authorColin Ian King <colin.king@canonical.com>
Tue, 7 Jan 2020 15:24:15 +0000 (15:24 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 7 Jan 2020 21:52:28 +0000 (13:52 -0800)
The variable failed is being assigned a value that is never read, the
following goto statement jumps to the end of the function and variable
failed is not referenced at all.  Remove the redundant assignment.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/rose/rose_route.c

index c533076..5277631 100644 (file)
@@ -696,7 +696,6 @@ struct rose_neigh *rose_get_neigh(rose_address *addr, unsigned char *cause,
                                for (i = 0; i < node->count; i++) {
                                        if (!rose_ftimer_running(node->neighbour[i])) {
                                                res = node->neighbour[i];
-                                               failed = 0;
                                                goto out;
                                        }
                                        failed = 1;