On 10/30/2015 4:15 AM, David Miller wrote:
From: Santosh Shilimkar <santosh.shilim...@oracle.com>
Date: Wed, 28 Oct 2015 15:51:12 -0700

+                                           &rs->rs_bound_node, ht_parms)) {
                        *port = rs->rs_bound_port;
                        ret = 0;
                        rdsdebug("rs %p binding to %pI4:%d\n",
                          rs, &addr, (int)ntohs(*port));
                        break;
                } else {
-                       rds_sock_put(rrs);
+                       rds_sock_put(rs);
+                       ret = -ENOMEM;
+                       break;
                }
+
        } while (rover++ != last);

        return ret;

This final insert of an empty line is both inappropriate and even
if it was it would not be related to this change.

Right. I just posted v2, dropping the empty line. Thanks !!

Regards,
Santosh
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to