Am 25.10.2011 10:23, schrieb ronnie sahlberg: > Hi, > > > On Tue, Oct 25, 2011 at 7:17 PM, Kevin Wolf <kw...@redhat.com> 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