[dm-devel] multipathd: cancel free(pp->getuid) in orphan_path()

Christophe Varoqui christophe.varoqui at opensvc.com
Tue Aug 9 15:15:50 UTC 2016


Reverted,
Thanks.

On Tue, Aug 9, 2016 at 2:58 PM, Gioh Kim <gi-oh.kim at profitbricks.com> wrote:

> Hello,
>
> I have found "free(pp->getuid)" is added into orphan_path().
>
> I think it'd better to revert the patch: http://git.opensvc.com/gitweb.
> cgi?p=multipath-tools/.git;a=commit;h=44b753d34ff82ecc229407
> fe8247b11f15f435ce
>
>
> The latest code does not allocation memory for pp->getuid, but just share
> the global value.
>
> Therefore pp->getuid should not be freed.
>
> I proposed it but I am wrong.
>
> I'm very sorry to confuse you.
>
>
>
> --
> Best regards,
> Gioh Kim
> TEL: 0176 2697 8962
>
> --
> dm-devel mailing list
> dm-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/dm-devel/attachments/20160809/79e42686/attachment.htm>


More information about the dm-devel mailing list