irda: Fix error propagation in ircomm_lmp_connect_response()
authorDavid S. Miller <davem@davemloft.net>
Thu, 19 May 2011 22:58:39 +0000 (18:58 -0400)
committerDavid S. Miller <davem@davemloft.net>
Thu, 19 May 2011 22:58:39 +0000 (18:58 -0400)
The variable 'ret' is set but unused, and this pointed out that
errors from irlmp_connect_response() are not propagated to the
caller.

Note that this is currently academic since irlmp_connect_response()
always returns 0. :-)

Signed-off-by: David S. Miller <davem@davemloft.net>
net/irda/ircomm/ircomm_lmp.c

index 08fb54dc8c412cbcb04721ca7df9f034ff06dcbf..3b8095c771d44371cfdb97b5607fb28817791fb8 100644 (file)
@@ -75,7 +75,6 @@ static int ircomm_lmp_connect_response(struct ircomm_cb *self,
                                       struct sk_buff *userdata)
 {
        struct sk_buff *tx_skb;
-       int ret;
 
        IRDA_DEBUG(0, "%s()\n", __func__ );
 
@@ -100,9 +99,7 @@ static int ircomm_lmp_connect_response(struct ircomm_cb *self,
                tx_skb = userdata;
        }
 
-       ret = irlmp_connect_response(self->lsap, tx_skb);
-
-       return 0;
+       return irlmp_connect_response(self->lsap, tx_skb);
 }
 
 static int ircomm_lmp_disconnect_request(struct ircomm_cb *self,