diff mbox series

[PULL,27/31] scsi-disk: add 'fall through' comment to switch VERIFY cases

Message ID a8802cd1d30fd153eac8f3a3ff9fdabf1cb069a5.1485250702.git.mjt@msgid.tls.msk.ru
State Accepted
Commit 4f04560ba93f179151d90b0e6c6c53235fe1a279
Headers show
Series None | expand

Commit Message

Michael Tokarev Jan. 24, 2017, 9:39 a.m. UTC 
From: Peter Maydell <peter.maydell@linaro.org>


Commit 166dbda7e131 added some extra cases to a switch() such
that the existing code is intended to fall through the new
case statements. It's clear from the commit that this is
intentional, but less clear to subsequent readers of the
code, and not clear at all to static analysis tools like
Coverity. Add a /* fall through */ comment to indicate the
intent. (Fixes CID 1368287.)

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>

---
 hw/scsi/scsi-disk.c | 1 +
 1 file changed, 1 insertion(+)

-- 
2.1.4
diff mbox series

Patch

diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
index c080888..cc06fe5 100644
--- a/hw/scsi/scsi-disk.c
+++ b/hw/scsi/scsi-disk.c
@@ -2157,6 +2157,7 @@  static int32_t scsi_disk_dma_command(SCSIRequest *req, uint8_t *buf)
         DPRINTF("Write %s(sector %" PRId64 ", count %u)\n",
                 (command & 0xe) == 0xe ? "And Verify " : "",
                 r->req.cmd.lba, len);
+        /* fall through */
     case VERIFY_10:
     case VERIFY_12:
     case VERIFY_16: