Re: [PATCH] pm80xx : Fix missing NULL pointer checks and memory leaks

2014-05-12 Thread Jack Wang
On 05/09/2014 08:01 AM, Suresh Thiagarajan wrote: > Checking return value for the memory allocattion and freeing it > while exiting the function > > Signed-off-by: Viswas G > Signed-off-by: Suresh Thiagarajan > --- > drivers/scsi/pm8001/pm8001_ctl.c |3 +++ > 1 files changed, 3 insertions(+

Re: [PATCH] pm80xx : Fix missing NULL pointer checks and memory leaks

2014-05-10 Thread Tomas Henzl
On 05/09/2014 08:01 AM, Suresh Thiagarajan wrote: > Checking return value for the memory allocattion and freeing it > while exiting the function The fix seems fine to me. Reviewed-by: Tomas Henzl Tomas > > Signed-off-by: Viswas G > Signed-off-by: Suresh Thiagarajan > --- > drivers/scsi/pm800

[PATCH] pm80xx : Fix missing NULL pointer checks and memory leaks

2014-05-08 Thread Suresh Thiagarajan
Checking return value for the memory allocattion and freeing it while exiting the function Signed-off-by: Viswas G Signed-off-by: Suresh Thiagarajan --- drivers/scsi/pm8001/pm8001_ctl.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/pm8001/pm8001_ctl.c b