Whoever called iucv_sk(sk)->sk_txnotify() must already know that they're
dealing with an af_iucv socket.

Signed-off-by: Julian Wiedmann <j...@linux.ibm.com>
---
 net/iucv/af_iucv.c | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)

diff --git a/net/iucv/af_iucv.c b/net/iucv/af_iucv.c
index 427a1abce0a8..8683b6939f45 100644
--- a/net/iucv/af_iucv.c
+++ b/net/iucv/af_iucv.c
@@ -2134,23 +2134,14 @@ static int afiucv_hs_rcv(struct sk_buff *skb, struct 
net_device *dev,
 static void afiucv_hs_callback_txnotify(struct sk_buff *skb,
                                        enum iucv_tx_notify n)
 {
-       struct sock *isk = skb->sk;
-       struct sock *sk = NULL;
-       struct iucv_sock *iucv = NULL;
+       struct iucv_sock *iucv = iucv_sk(skb->sk);
+       struct sock *sk = skb->sk;
        struct sk_buff_head *list;
        struct sk_buff *list_skb;
        struct sk_buff *nskb;
        unsigned long flags;
 
-       read_lock_irqsave(&iucv_sk_list.lock, flags);
-       sk_for_each(sk, &iucv_sk_list.head)
-               if (sk == isk) {
-                       iucv = iucv_sk(sk);
-                       break;
-               }
-       read_unlock_irqrestore(&iucv_sk_list.lock, flags);
-
-       if (!iucv || sock_flag(sk, SOCK_ZAPPED))
+       if (sock_flag(sk, SOCK_ZAPPED))
                return;
 
        list = &iucv->send_skb_q;
-- 
2.17.1

Reply via email to