diff options
| author | Gustavo F. Padovan | 2011-04-25 20:10:41 +0200 |
|---|---|---|
| committer | Gustavo F. Padovan | 2011-04-28 06:10:01 +0200 |
| commit | 6ff5abbf4e4aa88feb9c2367d4fbd9ea081bf98c (patch) | |
| tree | 05848e91d3ac481432b3c0718dbb9fd91a5067c1 /net/bluetooth/l2cap_core.c | |
| parent | Bluetooth: Move conn to struct l2cap_chan (diff) | |
| download | kernel-qcow2-linux-6ff5abbf4e4aa88feb9c2367d4fbd9ea081bf98c.tar.gz kernel-qcow2-linux-6ff5abbf4e4aa88feb9c2367d4fbd9ea081bf98c.tar.xz kernel-qcow2-linux-6ff5abbf4e4aa88feb9c2367d4fbd9ea081bf98c.zip | |
Bluetooth: Fix memory leak with L2CAP channels
A new l2cap_chan_free() is added to free the channels.
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
Diffstat (limited to 'net/bluetooth/l2cap_core.c')
| -rw-r--r-- | net/bluetooth/l2cap_core.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c index 8562ac1ba947..338d8c3eedab 100644 --- a/net/bluetooth/l2cap_core.c +++ b/net/bluetooth/l2cap_core.c @@ -160,6 +160,11 @@ struct l2cap_chan *l2cap_chan_alloc(struct sock *sk) return chan; } +void l2cap_chan_free(struct l2cap_chan *chan) +{ + kfree(chan); +} + static void __l2cap_chan_add(struct l2cap_conn *conn, struct l2cap_chan *chan) { struct sock *sk = chan->sk; @@ -236,7 +241,7 @@ void l2cap_chan_del(struct l2cap_chan *chan, int err) if (!(chan->conf_state & L2CAP_CONF_OUTPUT_DONE && chan->conf_state & L2CAP_CONF_INPUT_DONE)) - goto free; + return; skb_queue_purge(&chan->tx_q); @@ -255,9 +260,6 @@ void l2cap_chan_del(struct l2cap_chan *chan, int err) kfree(l); } } - -free: - kfree(chan); } static inline u8 l2cap_get_auth_type(struct l2cap_chan *chan) |