Il 09/05/2012 14:22, Kevin Wolf ha scritto:
> Am 08.05.2012 16:51, schrieb Paolo Bonzini:
>> We are reusing bs->file across close/open, which may not cause any
>> known bugs but is a recipe for trouble.  Prefer bdrv_delete, and
>> enjoy the new invariant in the implementation of bdrv_delete.
>>
>> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
> 
> I think bs->file is changed during bdrv_open() for all block drivers
> that make use of it. We do however leak the old bs->file then.

Right, bdrv_open_common has bs->file = NULL.  See also patch 9 in the
series...

Paolo


Reply via email to