diff options
author | Octavian Purdila <opurdila@ixiacom.com> | 2009-12-26 06:51:08 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-12-26 23:47:23 -0500 |
commit | 3100aa9d74db9c6d8d9a3b6421721fc1aef4728f (patch) | |
tree | e0a6cea30070c594f9addf79d762cbe521b49c29 | |
parent | 8beb9ab6c2df203e8d68cb1f48cf42604a6bed86 (diff) |
llc: fix SAP reference counting w.r.t. socket handling
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>
-rw-r--r-- | net/llc/af_llc.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/net/llc/af_llc.c b/net/llc/af_llc.c index f49f3dd6fbd3..e35d907fba2c 100644 --- a/net/llc/af_llc.c +++ b/net/llc/af_llc.c | |||
@@ -197,10 +197,8 @@ static int llc_ui_release(struct socket *sock) | |||
197 | llc->laddr.lsap, llc->daddr.lsap); | 197 | llc->laddr.lsap, llc->daddr.lsap); |
198 | if (!llc_send_disc(sk)) | 198 | if (!llc_send_disc(sk)) |
199 | llc_ui_wait_for_disc(sk, sk->sk_rcvtimeo); | 199 | llc_ui_wait_for_disc(sk, sk->sk_rcvtimeo); |
200 | if (!sock_flag(sk, SOCK_ZAPPED)) { | 200 | if (!sock_flag(sk, SOCK_ZAPPED)) |
201 | llc_sap_put(llc->sap); | ||
202 | llc_sap_remove_socket(llc->sap, sk); | 201 | llc_sap_remove_socket(llc->sap, sk); |
203 | } | ||
204 | release_sock(sk); | 202 | release_sock(sk); |
205 | if (llc->dev) | 203 | if (llc->dev) |
206 | dev_put(llc->dev); | 204 | dev_put(llc->dev); |
@@ -352,7 +350,6 @@ static int llc_ui_bind(struct socket *sock, struct sockaddr *uaddr, int addrlen) | |||
352 | rc = -EBUSY; /* some other network layer is using the sap */ | 350 | rc = -EBUSY; /* some other network layer is using the sap */ |
353 | if (!sap) | 351 | if (!sap) |
354 | goto out; | 352 | goto out; |
355 | llc_sap_hold(sap); | ||
356 | } else { | 353 | } else { |
357 | struct llc_addr laddr, daddr; | 354 | struct llc_addr laddr, daddr; |
358 | struct sock *ask; | 355 | struct sock *ask; |