[Qemu-devel] Re: [PATCH STABLE 2/2] net: Fix -net socket parameter checks

[Glauber Costa]

On Thu, May 21, 2009 at 08:23:22AM +0200, Jan Kiszka wrote:
> Glauber Costa wrote:
> > My commit ea053add700d8abe203cd79a9ffb082aee4eabc0 broke -net socket by
> > overwriting an intermediate buffer in the added check_param. Fix this
> > by switching check_param to automatic buffer allocation and release, ie.
> > callers no longer have to worry about providing a scratch buffer.
> 
> That commit was buggy, too, and is awaiting to be fixed in unstable by
> http://permalink.gmane.org/gmane.comp.emulators.qemu/43041.
> 
> But more importantly, it makes no sense in stable because its "heart",
> param checking for -net is only in unstable (and that's perfectly fine).
My interest was in the previous one, since it fixes a bug reported by a Fedora
user. I picked this one, only because it fixes the fix ;-)
But I can happily leave it out, if you think it is better.