desktop-packages team mailing list archive

[Garima <garima.g@xxxxxxxxxxx>]

Public bug reported:

check on variable iRawlen is redundant at some places in
libmng_chunk_io.c

At #3850: Redundant comparison '(5 <= iRawlen)' is always 'true' because it is already checked at #3775.
At #6346: Redundant comparison '(0 != iRawlen)'  is always 'true' because it is already checked at #6323.
At #3407: Redundant comparison '(0 != iRawlen)'  is always 'true' because it is already checked at #6380.

For detail information please check the attached patch file.

** Affects: libmng (Ubuntu)
     Importance: Undecided
         Status: New

** Patch added: "redundant_comparison.patch"
   https://bugs.launchpad.net/bugs/1501261/+attachment/4479765/+files/redundant_comparison.patch

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to libmng in Ubuntu.
https://bugs.launchpad.net/bugs/1501261

Title:
  redundant comparison libmng_chunk_io.c

Status in libmng package in Ubuntu:
  New

Bug description:
  check on variable iRawlen is redundant at some places in
  libmng_chunk_io.c

  At #3850: Redundant comparison '(5 <= iRawlen)' is always 'true' because it is already checked at #3775.
  At #6346: Redundant comparison '(0 != iRawlen)'  is always 'true' because it is already checked at #6323.
  At #3407: Redundant comparison '(0 != iRawlen)'  is always 'true' because it is already checked at #6380.

  For detail information please check the attached patch file.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/libmng/+bug/1501261/+subscriptions