Commit b77b36cb authored by Colin Ian King's avatar Colin Ian King Committed by Martin K. Petersen

scsi: pmcraid: remove redundant check to see if request_size is less than zero

The 2nd check to see if request_size is less than zero is redundant
because the first check takes error exit path on this condition. So,
since it is redundant, remove it.

Detected by CoverityScan, CID#146149 ("Logically Dead Code")
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Reviewed-by: default avatarTyrel Datwyler <tyreld@linux.vnet.ibm.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 019c0d66
...@@ -3770,9 +3770,6 @@ static long pmcraid_ioctl_passthrough( ...@@ -3770,9 +3770,6 @@ static long pmcraid_ioctl_passthrough(
pmcraid_err("couldn't build passthrough ioadls\n"); pmcraid_err("couldn't build passthrough ioadls\n");
goto out_free_cmd; goto out_free_cmd;
} }
} else if (request_size < 0) {
rc = -EINVAL;
goto out_free_cmd;
} }
/* If data is being written into the device, copy the data from user /* If data is being written into the device, copy the data from user
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment