Eric Blake <ebl...@redhat.com> writes:

> On 05/13/2014 10:02 AM, Markus Armbruster wrote:
>> Continues the conversion of the open method to Error started in commit
>> 015a103.
>> 
>> Signed-off-by: Markus Armbruster <arm...@redhat.com>
>> ---
>>  block/vvfat.c | 16 ++++++----------
>>  1 file changed, 6 insertions(+), 10 deletions(-)
>> 
>
>>      s->qcow = NULL;
>>      ret = bdrv_open(&s->qcow, s->qcow_filename, NULL, NULL,
>>              BDRV_O_RDWR | BDRV_O_CACHE_WB | BDRV_O_NO_FLUSH, bdrv_qcow,
>> -            &local_err);
>> +            errp);
>
> Pre-existing; indentation looks odd here if you want to touch it up at
> the same time.  But that's not a show-stopper.

Can take care of it when I respin to correct the leak in 04/18.

> Reviewed-by: Eric Blake <ebl...@redhat.com>

Thanks!

Reply via email to