diff options
| author |   Bradley Grove <bgrove@attotech.com> | 2013-08-29 15:55:43 -0400 |
|---|---|---|
| committer |   James Bottomley <JBottomley@Parallels.com> | 2013-09-06 11:13:53 -0700 |
| commit | 8d3ac484089aee9b1bd700af59017d685ae4803b (patch) | |
| tree | 7b424917e049fd356440c298859a332f73636309 /drivers/scsi | |
| parent | [SCSI] esas2r: smatch - Fix overrun due to sprintf appending NULL (diff) | |
| download | linux-dev-8d3ac484089aee9b1bd700af59017d685ae4803b.tar.xz linux-dev-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')
| -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; |