Re: [PATCH] nvme: don't free uuid pointer before printing it

2018-01-25 Thread Christoph Hellwig
On Thu, Jan 25, 2018 at 09:09:25AM +0100, Johannes Thumshirn wrote: > Commit df351ef73789 ("nvme-fabrics: fix memory leak when parsing host ID > option") fixed the leak of 'p' but in case uuid_parse() fails the memory > is freed before the error print that is using it. > > Free it after printing e

Re: [PATCH] nvme: don't free uuid pointer before printing it

2018-01-25 Thread Max Gurtovoy
On 1/25/2018 10:09 AM, Johannes Thumshirn wrote: Commit df351ef73789 ("nvme-fabrics: fix memory leak when parsing host ID option") fixed the leak of 'p' but in case uuid_parse() fails the memory is freed before the error print that is using it. Free it after printing eventual errors. Signed-o

[PATCH] nvme: don't free uuid pointer before printing it

2018-01-25 Thread Johannes Thumshirn
Commit df351ef73789 ("nvme-fabrics: fix memory leak when parsing host ID option") fixed the leak of 'p' but in case uuid_parse() fails the memory is freed before the error print that is using it. Free it after printing eventual errors. Signed-off-by: Johannes Thumshirn Fixes: df351ef73789 ("nvme