Commit 65bd6422 authored by Daniel Verkamp's avatar Daniel Verkamp
Browse files

iscsi: remove pointless assert(task != NULL) 



We already dereference task in the line above, and all callers ensure
that task is not NULL already.

Change-Id: Iff23503959116083375082b1ab022a156438cc8b
Signed-off-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-on: https://review.gerrithub.io/367109


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent e09174b1

lib/iscsi/conn.c

+0 −1
Original line number Diff line number Diff line
@@ -915,7 +915,6 @@ spdk_iscsi_task_cpl(struct spdk_scsi_task *scsi_task) 
	struct spdk_iscsi_task *task = spdk_iscsi_task_from_scsi_task(scsi_task);

	struct spdk_iscsi_conn *conn = task->conn;



	assert(task != NULL);

	spdk_trace_record(TRACE_ISCSI_TASK_DONE, conn->id, 0, (uintptr_t)task, 0);

	conn->last_activity_tsc = spdk_get_ticks();