Re: [PATCH] scsi: ufs: fix potential memory leak

2016-06-07 Thread Tiezhu Yang
At 2016-06-08 03:12:08, "James Bottomley"   wrote: >On Wed, 2016-06-08 at 02:00 +0800, Tiezhu Yang wrote: >> There exists potential memory leak in ufshcd_parse_clock_info(), >> this patch fixes it. > >What makes you think there's a leak here?  These are all 

Re: [PATCH] scsi: ufs: fix potential memory leak

2016-06-07 Thread James Bottomley
On Wed, 2016-06-08 at 02:00 +0800, Tiezhu Yang wrote: > There exists potential memory leak in ufshcd_parse_clock_info(), > this patch fixes it. What makes you think there's a leak here? These are all devm_ allocations, so they're all freed when the device is. If an error is returned, the device

[PATCH] scsi: ufs: fix potential memory leak

2016-06-07 Thread Tiezhu Yang
There exists potential memory leak in ufshcd_parse_clock_info(), this patch fixes it. Signed-off-by: Tiezhu Yang --- drivers/scsi/ufs/ufshcd-pltfrm.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/ufs/ufshcd-pltfrm.c