From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Date: Thu, 14 Oct 2010 18:23:30 +0000 Subject: [patch -next] scsi_transport_fc: remove unneeded check Message-Id: <20101014182330.GD6614@bicker> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "James E.J. Bottomley" Cc: James Smart , Christof Schmitt , Hannes Reinecke , Mike Christie , linux-scsi@vger.kernel.org, kernel-janitors@vger.kernel.org "val" can never be less than zero here. It's unsigned and simple_strtoul() always returns positive values. I also cleaned up the check. "*cp != '\0'" is more explicit than "*cp" and I removed some parenthesis that weren't needed. Signed-off-by: Dan Carpenter --- I sent this patch some months back but didn't recieve any feedback. diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/scsi_transport_fc.c index 0681378..b108a9b 100644 --- a/drivers/scsi/scsi_transport_fc.c +++ b/drivers/scsi/scsi_transport_fc.c @@ -844,7 +844,7 @@ store_fc_rport_dev_loss_tmo(struct device *dev, struct device_attribute *attr, (rport->port_state = FC_PORTSTATE_NOTPRESENT)) return -EBUSY; val = simple_strtoul(buf, &cp, 0); - if ((*cp && (*cp != '\n')) || (val < 0)) + if (*cp != '\0' && *cp != '\n') return -EINVAL; /*