[PATCH 1/1] SCSI: bnx2i: remove redundant check

this check hba is dereferenced without validation. Signed-off-by: Maninder Singh maninder...@samsung.com --- drivers/scsi/bnx2i/bnx2i_iscsi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/bnx2i/bnx2i_iscsi.c b/drivers/scsi/bnx2i/bnx2i_iscsi.c index 7289437..cbec2ea

[PATCH 1/1] SCSI/bfa: Use port pointer after NULL check

Currently port pointer is derefrenced before NULL check. So NULL check is misleading, Thus it is better to use port pointer after NULL Check. Signed-off-by: Maninder Singh maninder...@samsung.com Reviewed-by: Akhilesh Kumar akhiles...@samsung.com --- drivers/scsi/bfa/bfa_fcs_lport.c |3

[PATCH 1/1] scsi/snic: fixing the error print

Currently SNIC_HOST_ERR is called in both cases whether snic_disc_start returns zero or non-zero. From the code it looks like it has to print only if snic_disc_start return non-zero value, So calling SNIC_HOST_ERR only for non-zero return from snic_disc_start. Signed-off-by: Maninder Singh

[PATCH 1/1] scsi: remove check around pci_dev_put

pci_dev_put checks for NULL Pointer itself. Signed-off-by: Maninder Singh maninder...@samsung.com --- drivers/scsi/eata_pio.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/scsi/eata_pio.c b/drivers/scsi/eata_pio.c index ca8003f..481ecac 100644 --- a/drivers

[PATCH 1/1] scsi: remove redundant NULL check before kfree

There is no need of NULL check before kfree, So removing the same. Signed-off-by: Maninder Singh maninder...@samsung.com Reviewed-by: Rohit Thapliyal r.thapli...@samsung.com --- drivers/scsi/aic94xx/aic94xx_init.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers

[PATCH 1/1] scsi: use kzalloc for allocating one thing

Use kzalloc rather than kcalloc(1,...) for allocating one thing Signed-off-by: Maninder Singh maninder...@samsung.com Reviewed-by: Vaneet Narang v.nar...@samsung.com --- drivers/scsi/mvsas/mv_init.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/mvsas

[PATCH 1/1] scsi: Initialize sdp after NULL check of cmnd

Currently cmnd pointer is already dereferenced before NULL check and thus getting below warning in static analysis: warn: variable dereferenced before check 'cmnd' So initialize struct scsi_device *sdp after NULL check of cmnd Signed-off-by: Maninder Singh maninder...@samsung.com Reviewed