Re: [PATCH v2] Bluetooth: Fix memory leak at end of hci requests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Fred,

> In hci_req_sync_complete the event skb is referenced in hdev->req_skb.
> It is used (via hci_req_run_skb) from either __hci_cmd_sync_ev which will
> pass the skb to the caller, or __hci_req_sync which leaks.
> 
> unreferenced object 0xffff880005339a00 (size 256):
>  comm "kworker/u3:1", pid 1011, jiffies 4294671976 (age 107.389s)
>  backtrace:
>    [<ffffffff818d89d9>] kmemleak_alloc+0x49/0xa0
>    [<ffffffff8116bba8>] kmem_cache_alloc+0x128/0x180
>    [<ffffffff8167c1df>] skb_clone+0x4f/0xa0
>    [<ffffffff817aa351>] hci_event_packet+0xc1/0x3290
>    [<ffffffff8179a57b>] hci_rx_work+0x18b/0x360
>    [<ffffffff810692ea>] process_one_work+0x14a/0x440
>    [<ffffffff81069623>] worker_thread+0x43/0x4d0
>    [<ffffffff8106ead4>] kthread+0xc4/0xe0
>    [<ffffffff818dd38f>] ret_from_fork+0x1f/0x40
>    [<ffffffffffffffff>] 0xffffffffffffffff
> 
> Signed-off-by: Frédéric Dalleau <frederic.dalleau@xxxxxxxxxxxxxxx>
> ---
> net/bluetooth/hci_request.c | 2 ++
> 1 file changed, 2 insertions(+)

patch has been applied to bluetooth-stable tree.

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux