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

Gioh Kim gi-oh.kim at profitbricks.com
Tue Aug 9 12:58:17 UTC 2016


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=44b753d34ff82ecc229407fe8247b11f15f435ce


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




More information about the dm-devel mailing list