diff options
| author | David S. Miller <davem@davemloft.net> | 2011-05-20 00:58:39 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-05-20 00:58:39 +0200 |
| commit | 8ce6eb1a4a4bc46820f23e927377d386d3ec5404 (patch) | |
| tree | 03365bd164a28786402539083147041a2864525f /net | |
| parent | irda: Kill set but unused variable 'bytes' in irlan_check_command_param() (diff) | |
| download | linux-8ce6eb1a4a4bc46820f23e927377d386d3ec5404.tar.xz linux-8ce6eb1a4a4bc46820f23e927377d386d3ec5404.zip | |
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')
| -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, 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, |