diff options
| author | Sorin Dumitru <sdumitru@ixiacom.com> | 2012-08-05 22:35:58 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-08-06 16:30:01 -0400 |
| commit | 91d27a8650d5359a7a320daeb35b88cdea15e3a8 (patch) | |
| tree | 5449b227bd8607d9faec4ae70dba20bc063560bc | |
| parent | 8b82f7c3c90fa67fd6d5be1deb3cec0e17cc32d9 (diff) | |
llc: free the right skb
We are freeing skb instead of nskb, resulting in a double
free on skb and a leak from nskb.
Signed-off-by: Sorin Dumitru <sdumitru@ixiacom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/llc/llc_station.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/llc/llc_station.c b/net/llc/llc_station.c index 39a8d8924b9c..6828e39ec2ec 100644 --- a/net/llc/llc_station.c +++ b/net/llc/llc_station.c | |||
| @@ -268,7 +268,7 @@ static int llc_station_ac_send_null_dsap_xid_c(struct sk_buff *skb) | |||
| 268 | out: | 268 | out: |
| 269 | return rc; | 269 | return rc; |
| 270 | free: | 270 | free: |
| 271 | kfree_skb(skb); | 271 | kfree_skb(nskb); |
| 272 | goto out; | 272 | goto out; |
| 273 | } | 273 | } |
| 274 | 274 | ||
| @@ -293,7 +293,7 @@ static int llc_station_ac_send_xid_r(struct sk_buff *skb) | |||
| 293 | out: | 293 | out: |
| 294 | return rc; | 294 | return rc; |
| 295 | free: | 295 | free: |
| 296 | kfree_skb(skb); | 296 | kfree_skb(nskb); |
| 297 | goto out; | 297 | goto out; |
| 298 | } | 298 | } |
| 299 | 299 | ||
| @@ -322,7 +322,7 @@ static int llc_station_ac_send_test_r(struct sk_buff *skb) | |||
| 322 | out: | 322 | out: |
| 323 | return rc; | 323 | return rc; |
| 324 | free: | 324 | free: |
| 325 | kfree_skb(skb); | 325 | kfree_skb(nskb); |
| 326 | goto out; | 326 | goto out; |
| 327 | } | 327 | } |
| 328 | 328 | ||
