diff options
author | Ursula Braun <ursula.braun@de.ibm.com> | 2013-12-16 08:44:51 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-12-17 22:17:13 (GMT) |
commit | d239ae339e0740d2405f4d8a4d482dbe7b001c69 (patch) | |
tree | d1e394c219b36ddc9e4cd504d522fbdf8fef6bd7 | |
parent | d207cf4c1ad27c6f8881857ca301f8489826116d (diff) | |
download | linux-d239ae339e0740d2405f4d8a4d482dbe7b001c69.tar.xz |
netiucv: improve state checking in conn_action_txdone
state checking in conn_action_txdone() is inconsistent.
This patch makes it consistent and issues a trace message
if an unexpected state is detected for the netiucv device.
Signed-off-by: Ursula Braun <ursula.braun@de.ibm.com>
Signed-off-by: Frank Blaschka <frank.blaschka@de.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/s390/net/netiucv.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/s390/net/netiucv.c b/drivers/s390/net/netiucv.c index 9b333fc..ce16d1b 100644 --- a/drivers/s390/net/netiucv.c +++ b/drivers/s390/net/netiucv.c @@ -739,8 +739,12 @@ static void conn_action_txdone(fsm_instance *fi, int event, void *arg) IUCV_DBF_TEXT(trace, 4, __func__); - if (conn && conn->netdev) - privptr = netdev_priv(conn->netdev); + if (!conn || !conn->netdev) { + IUCV_DBF_TEXT(data, 2, + "Send confirmation for unlinked connection\n"); + return; + } + privptr = netdev_priv(conn->netdev); conn->prof.tx_pending--; if (single_flag) { if ((skb = skb_dequeue(&conn->commit_queue))) { |