[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH] hw/bt/sdp: Fix resource leak det
Re: [Qemu-trivial] [Qemu-devel] [PATCH] hw/bt/sdp: Fix resource leak detect by coverity
Mon, 16 Mar 2015 08:29:52 +0100
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)
Paolo Bonzini <address@hidden> writes:
> On 15/03/2015 11:23, Michael Tokarev wrote:
>> Or, alternatively, to keep this `data' pointer in sdp to use it in
>>> > In any case, it seems simpler to just leave this code aside.
>> How many times this code is called?
>> We have many many places in qemu where resources are allocated once
>> at startup and never freed just because there's no need to.
> Well, in this case the bug in bt_l2cap_sdp_close_ch is much worse than a
> resource leak. But bluetooth is not the utmost priority in QEMU
To put it more bluntly: it's rotting in peace.
Occasional drive-by fixes won't stop the rot, a dedicated maintener