diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-11-27 17:27:01 +0300 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-11-27 22:49:21 -0800 |
| commit | 6b64e1fe2ba80ac3ce12a21f1200bac594fa1a1c (patch) | |
| tree | 700d6a4076387f7e2059d70ea005f1a4209d34c7 /drivers | |
| parent | 3c989d7603872bf878840f7ce3ea49b73bea4c6c (diff) | |
target: update error handling for sbc_setup_write_same()
We recently changed this to return positive subsystem error codes so the
error handling needs to be updated.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/target/target_core_sbc.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/target/target_core_sbc.c b/drivers/target/target_core_sbc.c index 39ced25b70c..94fd004ea3f 100644 --- a/drivers/target/target_core_sbc.c +++ b/drivers/target/target_core_sbc.c @@ -437,7 +437,7 @@ sbc_parse_cdb(struct se_cmd *cmd, struct sbc_ops *ops) cmd->t_task_lba = get_unaligned_be64(&cdb[12]); ret = sbc_setup_write_same(cmd, &cdb[10], ops); - if (ret < 0) + if (ret) return ret; break; default: @@ -510,7 +510,7 @@ sbc_parse_cdb(struct se_cmd *cmd, struct sbc_ops *ops) cmd->t_task_lba = get_unaligned_be64(&cdb[2]); ret = sbc_setup_write_same(cmd, &cdb[1], ops); - if (ret < 0) + if (ret) return ret; break; case WRITE_SAME: @@ -528,7 +528,7 @@ sbc_parse_cdb(struct se_cmd *cmd, struct sbc_ops *ops) * of byte 1 bit 3 UNMAP instead of original reserved field */ ret = sbc_setup_write_same(cmd, &cdb[1], ops); - if (ret < 0) + if (ret) return ret; break; case VERIFY: |