Paolo Bonzini <pbonz...@redhat.com> 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.
