On 04/10/2013 11:24 AM, michaelc@xxxxxxxxxxx wrote: > From: Masatake YAMATO <yamato@xxxxxxxxxx> > > 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 <yamato@xxxxxxxxxx> > Reviewed-by: Mike Christie <michaelc@xxxxxxxxxxx> > Signed-off-by: Mike Christie <michaelc@xxxxxxxxxxx> Sorry about the incorrect signed of chain James. When I sent/made the patches I added both a reviewed-by and signed-off to this one by accident. Masatake made the patch. I am just resending to the list. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html