[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hw/bt/sdp: Fix resource leak detect by coverity
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] hw/bt/sdp: Fix resource leak detect by coverity |
Date: |
Mon, 16 Mar 2015 08:29:52 +0100 |
User-agent: |
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
>> bt_l2cap_sdp_close_ch().
>
> Yes.
>
>>> > 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
> development...
To put it more bluntly: it's rotting in peace.
Occasional drive-by fixes won't stop the rot, a dedicated maintener
could.