Re: [Qemu-devel] [PATCH] Permit zero-sized qemu_malloc() & friends

[Kevin Wolf]

Am 07.12.2009 18:09, schrieb malc:
> On Mon, 7 Dec 2009, Anthony Liguori wrote:
> 
>> malc wrote:
>>>> Does anyone object to this moving forward?
>>>>     
>>>
>>> Yeah, i object to the split production/development qemu_malloc[z].
>>>   
>>
>> It's clear to me that there are still improper callers of qemu_malloc() in 
>> the
>> tree.  How do you propose we address this for 0.12?
>>
>> Aborting in a production build is a rather hostile thing to do if it can be
>> avoided.
> 
> The only real issue encountered so far was eb0b64f7a, there are claims
> that "maybe there are more", 

702ef63f was the one that started this discussion. And Markus had list
of five other places that can very likely crash with the abort. Every
single crash is one crash too much in a production environment. Turning
the abort off is the only sane option there.

Having the abort in place in the development branch is a concession to
you. Crashes don't have as bad impact there, so we probably can afford
it. I'm fine with that as long as there is a plan to move forward to a
better API.

> well i can also claim that there are abusers
> of the interface that just weren't encoutered yet,

Right, but have you found a single one of them yet? If not, in terms of
committed patches the score is 4:0 for legitimate users broken by
abort() vs. abusers found by abort().

Kevin