Hi,

took a snapshot of yesterdays "master" after "rbd: Fix use after free in rbd_open()"... Works perfectly ;)

Thank you very much for the bleeding fast reaction...

Oliver.

On 04/25/2013 07:32 PM, Kevin Wolf wrote:
Am 25.04.2013 um 19:25 hat Stefan Priebe - Profihost AG geschrieben:
Ist this qemu 1.4.1? Did you received the patch?

Stefan
No, this is actually about git master. Not sure why this was directed to
qemu-stable.

Kevin

Am 25.04.2013 um 16:13 schrieb Oliver Francke <oliver.fran...@filoo.de>:

Cool,

thnx for the quick response... Not a C-coder any more, only wild-not-too-bad 
guessing ;)

Oliver.

On 04/25/2013 03:51 PM, Kevin Wolf wrote:
Am 25.04.2013 um 15:01 hat Oliver Francke geschrieben:
Hi,

just encountered a bug while playing with latest aio_flush option in
ceph + qemu handling.

I got a "invalid argument" even with "qemu-img info rbd:pool/image.rbd".

Perhaps qemu_rbd_open is called with wrong number of params
somewhere else after expanding params with "QDict *options" in
block/rbd.c line around line 444 with current master.
( talking of qemu-54f106d)

Just as a quick note ;)
Thanks for the report, Oliver. The problem is the qemu_opts_del() in
line 481, which is too early. This frees the filename already when it's
still needed for the parsing.

I'm copying qemu-devel and Josh, just so everyone is informed. I'll try
to send a patch myself in a few minutes.

Kevin

--

Oliver Francke

filoo GmbH
Moltkestraße 25a
33330 Gütersloh
HRB4355 AG Gütersloh

Geschäftsführer: S.Grewing | J.Rehpöhler | C.Kunz

Folgen Sie uns auf Twitter: http://twitter.com/filoogmbh




--

Oliver Francke

filoo GmbH
Moltkestraße 25a
33330 Gütersloh
HRB4355 AG Gütersloh

Geschäftsführer: S.Grewing | J.Rehpöhler | C.Kunz

Folgen Sie uns auf Twitter: http://twitter.com/filoogmbh


Reply via email to