llc: fix SAP reference counting w.r.t. socket handling
authorOctavian Purdila <opurdila@ixiacom.com>
Sat, 26 Dec 2009 11:51:08 +0000 (11:51 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 27 Dec 2009 04:47:23 +0000 (20:47 -0800)
The SAP ref counter gets decremented twice when deleting a socket,
although for all but the first socket of a SAP the SAP ref counter was
incremented only once.

Signed-off-by: Octavian Purdila <opurdila@ixiacom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/llc/af_llc.c

index f49f3dd6fbd3bb12ea8f4f226984733af0d7f699..e35d907fba2c20e5b931751584ecacd68f0c4c9e 100644 (file)
@@ -197,10 +197,8 @@ static int llc_ui_release(struct socket *sock)
                llc->laddr.lsap, llc->daddr.lsap);
        if (!llc_send_disc(sk))
                llc_ui_wait_for_disc(sk, sk->sk_rcvtimeo);
-       if (!sock_flag(sk, SOCK_ZAPPED)) {
-               llc_sap_put(llc->sap);
+       if (!sock_flag(sk, SOCK_ZAPPED))
                llc_sap_remove_socket(llc->sap, sk);
-       }
        release_sock(sk);
        if (llc->dev)
                dev_put(llc->dev);
@@ -352,7 +350,6 @@ static int llc_ui_bind(struct socket *sock, struct sockaddr *uaddr, int addrlen)
                rc = -EBUSY; /* some other network layer is using the sap */
                if (!sap)
                        goto out;
-               llc_sap_hold(sap);
        } else {
                struct llc_addr laddr, daddr;
                struct sock *ask;