[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 7/7] block/iscsi: fix segfault if writesame fails
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PULL 7/7] block/iscsi: fix segfault if writesame fails |
Date: |
Thu, 27 Feb 2014 17:55:26 +0100 |
From: Peter Lieven <address@hidden>
commit fa6252b0 introduced a segfault because it tries
to read iTask.task->sense after iTask.task has been
freed.
Signed-off-by: Peter Lieven <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
---
block/iscsi.c | 17 +++++++++--------
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/block/iscsi.c b/block/iscsi.c
index 4360cf1..0a15f53 100644
--- a/block/iscsi.c
+++ b/block/iscsi.c
@@ -836,6 +836,15 @@ retry:
qemu_coroutine_yield();
}
+ if (iTask.status == SCSI_STATUS_CHECK_CONDITION &&
+ iTask.task->sense.key == SCSI_SENSE_ILLEGAL_REQUEST &&
+ iTask.task->sense.ascq == SCSI_SENSE_ASCQ_INVALID_OPERATION_CODE) {
+ /* WRITE SAME is not supported by the target */
+ iscsilun->has_write_same = false;
+ scsi_free_scsi_task(iTask.task);
+ return -ENOTSUP;
+ }
+
if (iTask.task != NULL) {
scsi_free_scsi_task(iTask.task);
iTask.task = NULL;
@@ -847,14 +856,6 @@ retry:
}
if (iTask.status != SCSI_STATUS_GOOD) {
- if (iTask.status == SCSI_STATUS_CHECK_CONDITION &&
- iTask.task->sense.key == SCSI_SENSE_ILLEGAL_REQUEST &&
- iTask.task->sense.ascq == SCSI_SENSE_ASCQ_INVALID_OPERATION_CODE) {
- /* WRITE SAME is not supported by the target */
- iscsilun->has_write_same = false;
- return -ENOTSUP;
- }
-
return -EIO;
}
--
1.8.5.3
- [Qemu-devel] [PULL 0/7] SCSI patches for 2014-02-28, Paolo Bonzini, 2014/02/27
- [Qemu-devel] [PULL 1/7] scsi: Change scsi sense buf size to 252, Paolo Bonzini, 2014/02/27
- [Qemu-devel] [PULL 3/7] scsi-bus: Fix transfer length for VERIFY with BYTCHK=11b, Paolo Bonzini, 2014/02/27
- [Qemu-devel] [PULL 4/7] block/iscsi: fix deadlock on scsi check condition, Paolo Bonzini, 2014/02/27
- [Qemu-devel] [PULL 5/7] block/iscsi: query for supported VPD pages, Paolo Bonzini, 2014/02/27
- [Qemu-devel] [PULL 2/7] scsi: report thin provisioning errors with werror=report, Paolo Bonzini, 2014/02/27
- [Qemu-devel] [PULL 6/7] scsi-disk: Add support for port WWN and index descriptors in VPD page 83h, Paolo Bonzini, 2014/02/27
- [Qemu-devel] [PULL 7/7] block/iscsi: fix segfault if writesame fails,
Paolo Bonzini <=