summaryrefslogtreecommitdiffstats
path: root/mailinglist/output_launchpad/1882065
diff options
context:
space:
mode:
authorChristian Krinitsin <mail@krinitsin.com>2025-05-30 14:51:13 +0000
committerChristian Krinitsin <mail@krinitsin.com>2025-05-30 14:51:13 +0000
commit225caa38269323af1bfc2daadff5ec8bd930747f (patch)
treee0a5fefde9ee100ba6f32fb36de6707490e4164e /mailinglist/output_launchpad/1882065
parent904141bfb8d5385b75eb3b7afec1dcda89af65a7 (diff)
downloademulator-bug-study-225caa38269323af1bfc2daadff5ec8bd930747f.tar.gz
emulator-bug-study-225caa38269323af1bfc2daadff5ec8bd930747f.zip
add mailinglist scraper results
Diffstat (limited to 'mailinglist/output_launchpad/1882065')
-rw-r--r--mailinglist/output_launchpad/188206537
1 files changed, 37 insertions, 0 deletions
diff --git a/mailinglist/output_launchpad/1882065 b/mailinglist/output_launchpad/1882065
new file mode 100644
index 00000000..1c219b65
--- /dev/null
+++ b/mailinglist/output_launchpad/1882065
@@ -0,0 +1,37 @@
+Could this cause OOB bug ?
+
+In function megasas_handle_scsi(hw/scsi/megasas.c):
+
+```c
+static int megasas_handle_scsi(MegasasState *s, MegasasCmd *cmd,
+ int frame_cmd)
+{
+ ............................................................................
+ cdb = cmd->frame->pass.cdb;
+ target_id = cmd->frame->header.target_id;
+ lun_id = cmd->frame->header.lun_id;
+ cdb_len = cmd->frame->header.cdb_len;
+ ............................................................................
+ if (cdb_len > 16) {
+ trace_megasas_scsi_invalid_cdb_len(
+ mfi_frame_desc[frame_cmd], is_logical,
+ target_id, lun_id, cdb_len);
+ megasas_write_sense(cmd, SENSE_CODE(INVALID_OPCODE));
+ cmd->frame->header.scsi_status = CHECK_CONDITION;
+ s->event_count++;
+ return MFI_STAT_SCSI_DONE_WITH_ERROR;
+ }
+}
+```
+
+Two variables, frame_cmd and cdb_len, can be controlled by guest os. So can mfi_frame_desc[frame_cmd] cause OOB bug ?
+
+QEMU emulator version 5.0.50 (v5.0.0-533-gdebe78ce14-dirty)
+
+You must start the trace function of QEMU to trigger this BUG!
+
+I think we should fix this anyway, even if it can only be triggered when trace functions are enabled
+
+Fix has been included:
+https://git.qemu.org/?p=qemu.git;a=commitdiff;h=ee760ac80ac1f1
+