[Qemu-devel] [Bug 1753437] Re: qemu/pc-bios/s390-ccw/libc.c:82: bad test ?

[Thomas Huth]

Looks like the mailing list <-> launchpad bridge again ignored mails to the 
corresponding mailing list thread. It's not a real bug, see here for details:
https://lists.gnu.org/archive/html/qemu-devel/2018-03/msg01142.html
I'll try to remember to clean this up the next time we update the s390-ccw bios.

** Changed in: qemu
     Assignee: (unassigned) => Thomas Huth (th-huth)

** Changed in: qemu
   Importance: Undecided => Low

** Changed in: qemu
       Status: New => Confirmed

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

Title:
  qemu/pc-bios/s390-ccw/libc.c:82: bad test ?

Status in QEMU:
  Confirmed

Bug description:
  qemu/pc-bios/s390-ccw/libc.c:82]: (style) Unsigned variable 'num_idx'
  can't be negative so it is unnecessary to test it.

  Source code is

  
      while (num_idx >= 0) {

  but

      size_t num_idx = 1; /* account for NUL */

  So there is no escape from the while loop.

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