Public bug reported:

qemu/hw/block/onenand.c:520] -> [qemu/hw/block/onenand.c:521]: (warning)
Opposite inner 'if' condition leads to a dead code block.

Source code is

        for (b = 0; b < s->blocks; b ++) {
            if (b >= s->blocks) {
                s->status |= ONEN_ERR_CMD;
                break;
            }

I am not sure if the if condition can be merely deleted, or something more
complex needs to be implemented.

Recent qemu trunk.

** Affects: qemu
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1715007

Title:
  hw/block/onenand.c:520: dead code ?

Status in QEMU:
  New

Bug description:
  qemu/hw/block/onenand.c:520] -> [qemu/hw/block/onenand.c:521]:
  (warning) Opposite inner 'if' condition leads to a dead code block.

  Source code is

          for (b = 0; b < s->blocks; b ++) {
              if (b >= s->blocks) {
                  s->status |= ONEN_ERR_CMD;
                  break;
              }

  I am not sure if the if condition can be merely deleted, or something more
  complex needs to be implemented.

  Recent qemu trunk.

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1715007/+subscriptions

Reply via email to