[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] This patch adds a new block driver : iSCSI
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH] This patch adds a new block driver : iSCSI |
Date: |
Tue, 25 Oct 2011 10:46:41 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:7.0) Gecko/20110927 Thunderbird/7.0 |
Am 25.10.2011 10:23, schrieb ronnie sahlberg:
> Hi,
>
>
> On Tue, Oct 25, 2011 at 7:17 PM, Kevin Wolf <address@hidden> wrote:
>> Am 25.10.2011 10:04, schrieb ronnie sahlberg:
>>>> iscsi_destroy_url() is only ever called here. Do we leak it in the normal
>>>> path?
>>>
>>> It didnt leak but the the label was confusing. I have changed it from
>>> "failed" to "finished" since this part
>>> is executed for both normal and failures.
>>
>> But there's a return 0; immediately before the label?
>
> Right. Its been a long day :-)
>
> Please find attached a fixed version.
Thanks, applied to the block branch.
Kevin
- Re: [Qemu-devel] [PATCH] This patch adds a new block driver : iSCSI, (continued)
Re: [Qemu-devel] [PATCH] This patch adds a new block driver : iSCSI, Zhi Yong Wu, 2011/10/28