From: Masatake YAMATO Date: Wed, 10 Apr 2013 16:24:38 +0000 (-0500) Subject: [SCSI] libiscsi: avoid unnecessary multiple NULL assignments X-Git-Tag: v3.10-rc1~150^2~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1fc2b00fba55f70ab32ecc493efbca9a1a76a4e3;p=profile%2Fivi%2Fkernel-x86-ivi.git [SCSI] libiscsi: avoid unnecessary multiple NULL assignments In iscsi_free_task, NULL is assigned to task->sc twice: before and after kfifo_in invocatoin. Allocating and freeing iscsi_task are guarded with session->lock, so multiple NULL assignments cause no trouble. But people reading the source code may be confused. The second NULL assignment comes from commit: 3e5c28ad0391389959ccae81c938c7533efb3490 It seems that the line after kfifo_in invocation was introduced accidentally. Signed-off-by: Masatake YAMATO Signed-off-by: Mike Christie Signed-off-by: James Bottomley --- diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c index d5e5c44..5de9469 100644 --- a/drivers/scsi/libiscsi.c +++ b/drivers/scsi/libiscsi.c @@ -507,7 +507,6 @@ static void iscsi_free_task(struct iscsi_task *task) kfifo_in(&session->cmdpool.queue, (void*)&task, sizeof(void*)); if (sc) { - task->sc = NULL; /* SCSI eh reuses commands to verify us */ sc->SCp.ptr = NULL; /*