search

iron261 / openjpeg

Automatically exported from code.google.com/p/openjpeg
Other
0 stars 0 forks source link

Bugs in openjpeg-1.5-r2833-1 #325

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
1. configure.ac has wrong VERSION

2. pngtoimage() in convert.c has missing code
   in case the PNG image is interlaced.

./image_to_j2k -i interlaced-1471-2105-7-160-1.PNG -o interlaced.jp2

 libpng warning:
  Interlace handling should be turned on when using png_read_image

3. j2k_to_image.c in convert lacks infile test.

4. cio_skip() in libopenjpeg has two different assert test.

./mj2_to_frames /home/IMG/MJ2/Speedway.mj2 speed.yuv

mj2_to_frames: ../../../applications/mj2/../../libopenjpeg/cio.c:191: cio_skip: 
Assertion `(cio->bp + n) >= cio->bp' failed.
Aborted

   The first test is wrong:

void cio_skip(opj_cio_t *cio, int n) {
  assert((cio->bp + n) >= cio->bp); <=================================
  if (((cio->bp + n) < cio->start) || ((cio->bp + n) > cio->end)) {<==
    assert(0);
  }
    cio->bp += n;
}

winfried

Original issue reported on code.google.com by szukw...@arcor.de on 3 Apr 2014 at 3:42

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by mathieu.malaterre on 28 Apr 2014 at 7:52