]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
rds: send: Fix dead code in rds_sendmsg
authorGustavo A. R. Silva <gustavo@embeddedor.com>
Wed, 25 Jul 2018 15:22:27 +0000 (10:22 -0500)
committerDavid S. Miller <davem@davemloft.net>
Thu, 26 Jul 2018 05:37:31 +0000 (22:37 -0700)
Currently, code at label *out* is unreachable. Fix this by updating
variable *ret* with -EINVAL, so the jump to *out* can be properly
executed instead of directly returning from function.

Addresses-Coverity-ID: 1472059 ("Structurally dead code")
Fixes: 1e2b44e78eea ("rds: Enable RDS IPv6 support")
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Acked-by: Sowmini Varadhan <sowmini.varadhan@oracle.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/rds/send.c

index 9604e1faa5648f65ec14ed3753a0be32aa3adcf9..18e2b4d3931fdcac06c99ed07d6fa76ff36d85c7 100644 (file)
@@ -1126,7 +1126,7 @@ int rds_sendmsg(struct socket *sock, struct msghdr *msg, size_t payload_len)
                                if (addr4 == htonl(INADDR_ANY) ||
                                    addr4 == htonl(INADDR_BROADCAST) ||
                                    IN_MULTICAST(ntohl(addr4))) {
-                                       return -EINVAL;
+                                       ret = -EINVAL;
                                        goto out;
                                }
                        }