Opened 9 years ago

Closed 7 years ago

Last modified 7 years ago

#5661 closed defect (fixed)

failure reading grib files with png compression

Reported by: robertwagner Owned by: warmerdam
Priority: normal Milestone: 2.2.0
Component: default Version: 1.11.0
Severity: normal Keywords:
Cc: Kyle Shannon


‘Data Representation Template 5.41 (Grid Point Data - PNG Compression)’

1) The Makefile for the grib format data (gdal-<version>/frmts/grib/GNUmakefile) DOES NOT check for

nor include the proper PNG settings and does not build the png wrapper code in the g2clib module (gdal-<version>/frmts/grib/degrib18/g2clib-1.0.4/{ dec_png.c, enc_png.c, pngpack.c, pngunpack.c } ).

2) Starting with RHEL 6.x attempted compiles of dec_png.c & enc_png.c fail with the error “undefined reference ‘voidp’”

Note: The dec_png.c and enc_png.c is not the latest version of the 3rd party library in gdal-1.11.0. The latest version has the voidp issue fixed. If you would like the patch form gdal-1.11.0 I can provide it.

Attachments (1)

patch.txt (2.1 KB ) - added by robertwagner 9 years ago.
Patch file

Download all attachments as: .zip

Change History (7)

comment:1 by Kyle Shannon, 9 years ago

Cc: Kyle Shannon added

Can you provide the patch please?

by robertwagner, 9 years ago

Attachment: patch.txt added

Patch file

in reply to:  1 comment:2 by robertwagner, 9 years ago

Replying to kyle:

Can you provide the patch please?

I just added it to the ticket.

comment:3 by Jukka Rahkonen, 9 years ago

Great opportunity for someone to review the patch which is not big at all, accept or improve it, apply and make GDAL a bit better. Plus close the ticket and make the list of open issues shorter.

comment:4 by Even Rouault, 9 years ago

kyle, do you still want to handle this ? Windows makefile would need an update too. A test file in the testsuite would be a plus too.

comment:5 by Even Rouault, 7 years ago

Resolution: fixed
Status: newclosed

In 35760:

GRIB: PNG decoder. Fix compiler warnings, undefined behaviour, and optimizations. Mixes of patch by zflamig + fixes #5661 by robertwagner + securing dec_png() by myself)

comment:6 by Even Rouault, 7 years ago

Milestone: 2.2.0

Related commits: r35758 + r35759

Note: See TracTickets for help on using tickets.