File 0262-esp-ensure-cmdfifo-is-not-empty-and.patch of Package qemu.20749

Overview Repositories Revisions Requests Users Attributes Meta

File 0262-esp-ensure-cmdfifo-is-not-empty-and.patch of Package qemu.20749

From: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Date: Wed, 7 Apr 2021 20:57:55 +0100
Subject: esp: ensure cmdfifo is not empty and current_dev is non-NULL
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Git-commit: 99545751734035b76bd372c4e7215bb337428d89
References: bsc#1180433, CVE-2020-35504
            bsc#1180434, CVE-2020-35505
            bsc#1180435, CVE-2020-35506

When about to execute a SCSI command, ensure that cmdfifo is not empty and
current_dev is non-NULL. This can happen if the guest tries to execute a TI
(Transfer Information) command without issuing one of the select commands
first.

Buglink: https://bugs.launchpad.net/qemu/+bug/1910723
Buglink: https://bugs.launchpad.net/qemu/+bug/1909247
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Tested-by: Alexander Bulekov <alxndr@bu.edu>
Message-Id: <20210407195801.685-7-mark.cave-ayland@ilande.co.uk>
Signed-off-by: Jose R Ziviani <jose.ziviani@suse.com>
---
 hw/scsi/esp.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c
index f50c3f6e0a5692a534a391bbde41..b10707087af26cb9b6f3fa5e5891 100644
--- a/hw/scsi/esp.c
+++ b/hw/scsi/esp.c
@@ -136,6 +136,11 @@ static void do_busid_cmd(ESPState *s, uint8_t *buf, uint8_t busid)
 
     trace_esp_do_busid_cmd(busid);
     lun = busid & 7;
+
+    if (!s->current_dev) {
+        return;
+    }
+
     current_lun = scsi_device_find(&s->bus, 0, s->current_dev->id, lun);
     s->current_req = scsi_req_new(current_lun, 0, lun, buf, s);
     datalen = scsi_req_enqueue(s->current_req);

Places

File 0262-esp-ensure-cmdfifo-is-not-empty-and.patch of Package qemu.20749

Places