diff options
author | David S. Miller <davem@davemloft.net> | 2011-05-19 18:58:39 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-05-19 18:58:39 -0400 |
commit | 8ce6eb1a4a4bc46820f23e927377d386d3ec5404 (patch) | |
tree | 03365bd164a28786402539083147041a2864525f /net/irda | |
parent | ad6376434c945495ea8c4082ad8a392cc58c3406 (diff) |
irda: Fix error propagation in ircomm_lmp_connect_response()
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>
Diffstat (limited to 'net/irda')
-rw-r--r-- | net/irda/ircomm/ircomm_lmp.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/net/irda/ircomm/ircomm_lmp.c b/net/irda/ircomm/ircomm_lmp.c index 08fb54dc8c41..3b8095c771d4 100644 --- a/net/irda/ircomm/ircomm_lmp.c +++ b/net/irda/ircomm/ircomm_lmp.c | |||
@@ -75,7 +75,6 @@ static int ircomm_lmp_connect_response(struct ircomm_cb *self, | |||
75 | struct sk_buff *userdata) | 75 | struct sk_buff *userdata) |
76 | { | 76 | { |
77 | struct sk_buff *tx_skb; | 77 | struct sk_buff *tx_skb; |
78 | int ret; | ||
79 | 78 | ||
80 | IRDA_DEBUG(0, "%s()\n", __func__ ); | 79 | IRDA_DEBUG(0, "%s()\n", __func__ ); |
81 | 80 | ||
@@ -100,9 +99,7 @@ static int ircomm_lmp_connect_response(struct ircomm_cb *self, | |||
100 | tx_skb = userdata; | 99 | tx_skb = userdata; |
101 | } | 100 | } |
102 | 101 | ||
103 | ret = irlmp_connect_response(self->lsap, tx_skb); | 102 | return irlmp_connect_response(self->lsap, tx_skb); |
104 | |||
105 | return 0; | ||
106 | } | 103 | } |
107 | 104 | ||
108 | static int ircomm_lmp_disconnect_request(struct ircomm_cb *self, | 105 | static int ircomm_lmp_disconnect_request(struct ircomm_cb *self, |