<div dir="ltr">Reverted,<div>Thanks.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 9, 2016 at 2:58 PM, Gioh Kim <span dir="ltr"><<a href="mailto:gi-oh.kim@profitbricks.com" target="_blank">gi-oh.kim@profitbricks.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
I have found "free(pp->getuid)" is added into orphan_path().<br>
<br>
I think it'd better to revert the patch: <a href="http://git.opensvc.com/gitweb.cgi?p=multipath-tools/.git;a=commit;h=44b753d34ff82ecc229407fe8247b11f15f435ce" rel="noreferrer" target="_blank">http://git.opensvc.com/gitweb.<wbr>cgi?p=multipath-tools/.git;a=c<wbr>ommit;h=44b753d34ff82ecc229407<wbr>fe8247b11f15f435ce</a><br>
<br>
<br>
The latest code does not allocation memory for pp->getuid, but just share the global value.<br>
<br>
Therefore pp->getuid should not be freed.<br>
<br>
I proposed it but I am wrong.<br>
<br>
I'm very sorry to confuse you.<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
<br>
-- <br>
Best regards,<br>
Gioh Kim<br>
TEL: 0176 2697 8962<br>
<br>
--<br>
dm-devel mailing list<br>
<a href="mailto:dm-devel@redhat.com" target="_blank">dm-devel@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/dm-devel" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/dm-devel</a><br>
</font></span></blockquote></div><br></div>