Commit 7c3a23b8 authored Dec 09, 2016 by Colin Ian King Committed by Christoph Hellwig Dec 21, 2016

nvmet/fcloop: remove some logically dead code performing redundant ret checks



The check to see if ret is non-zero and return this rather than count
is redundant in two occassions.  It is redundant because prior to this
check, the return code ret is already checked for a non-zero error
return value and we return from the function at that point.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-off-by: James Smart <james.smart@broadcom.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>

parent 6c73f949

drivers/nvme/target/fcloop.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -845,7 +845,7 @@ fcloop_create_remote_port(struct device dev, struct device_attribute attr,
		rport->lport = nport->lport;
		nport->rport = rport;

		return ret ? ret : count;
		return count;
		}


		@@ -952,7 +952,7 @@ fcloop_create_target_port(struct device dev, struct device_attribute attr,
		tport->lport = nport->lport;
		nport->tport = tport;

		return ret ? ret : count;
		return count;
		}