From: Colin Ian King <colin.king@xxxxxxxxxxxxx> Variable ret is being assigned with a value that is never read as there is return statement immediately afterwards. The assignment is redundant and hence can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> --- drivers/scsi/snic/snic_disc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/snic/snic_disc.c b/drivers/scsi/snic/snic_disc.c index e9ccfb97773f..d89c75991323 100644 --- a/drivers/scsi/snic/snic_disc.c +++ b/drivers/scsi/snic/snic_disc.c @@ -261,8 +261,6 @@ snic_tgt_create(struct snic *snic, struct snic_tgt_id *tgtid) tgt = kzalloc(sizeof(*tgt), GFP_KERNEL); if (!tgt) { SNIC_HOST_ERR(snic->shost, "Failure to allocate snic_tgt.\n"); - ret = -ENOMEM; - return tgt; } -- 2.20.1