diff options
author | Bradley Grove <bgrove@attotech.com> | 2013-08-29 21:55:43 +0200 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2013-09-06 20:13:53 +0200 |
commit | 8d3ac484089aee9b1bd700af59017d685ae4803b (patch) | |
tree | 7b424917e049fd356440c298859a332f73636309 /drivers/scsi/esas2r/esas2r_ioctl.c | |
parent | [SCSI] esas2r: smatch - Fix overrun due to sprintf appending NULL (diff) | |
download | linux-8d3ac484089aee9b1bd700af59017d685ae4803b.tar.xz linux-8d3ac484089aee9b1bd700af59017d685ae4803b.zip |
[SCSI] esas2r: smatch - Remove test for impossible condition (uint8 > 255)
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Bradley Grove <bgrove@attotech.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi/esas2r/esas2r_ioctl.c')
-rw-r--r-- | drivers/scsi/esas2r/esas2r_ioctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/esas2r/esas2r_ioctl.c b/drivers/scsi/esas2r/esas2r_ioctl.c index f3d0cb885972..e5b09027e066 100644 --- a/drivers/scsi/esas2r/esas2r_ioctl.c +++ b/drivers/scsi/esas2r/esas2r_ioctl.c @@ -415,7 +415,7 @@ static int csmi_ioctl_callback(struct esas2r_adapter *a, lun = tm->lun; } - if (path > 0 || tid > ESAS2R_MAX_ID) { + if (path > 0) { rq->func_rsp.ioctl_rsp.csmi.csmi_status = cpu_to_le32( CSMI_STS_INV_PARAM); return false; |