[#GEOT-1925] GeoTiffReader causes NullException when numOverviews == 1 - jira.codehaus.org

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 2.4.4
Fix Version/s: 2.4.5
Component/s: geotiff
Labels:
None
Environment:
RHEL 3, Java 1.5.0_12, JBoss 4.0.3SP1, GeoServer 1.6.4

Testcase included:
yes

Description

GeoTools does not support GeoTiffs that have only one overview. Details below:

On line 308 of GeoTiffReader.java from 2.4.4 tag the overViewResolutions array only gets initialized if numOverviews > 1.
Later on, on line 275 of AbstractGridCoverage2DReader.java (which is base class of GeoTiffReader), overViewResolutions gets dereferenced if numOverviews > 0. In the case where the GeoTiff has exactly one overview, a null exception is thrown.

The fix is to change GeoTiffReader.java:308 to check if numOverviews >= 1.

More Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

$i18n.getText("admin.common.words.hide")
box123_IR-addo.tiff.zip

16/Jul/08 7:11 PM

111 kB

Ryan Hofschneider
box123_IR-addo.tiff 1.50 MB

Download Zip
$i18n.getText("admin.common.words.show")

box123_IR-addo.tiff.zip

16/Jul/08 7:11 PM

111 kB

Ryan Hofschneider

Issue Links

This issue is depended upon by:
GEOS-2044	Can't add GeoTIFF coverage if it has exactly one overview

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Ryan Hofschneider added a comment - 16/Jul/08 7:11 PM

A GeoTiff with one overview that results in NullPointerException.

Show

Ryan Hofschneider added a comment - 16/Jul/08 7:11 PM A GeoTiff with one overview that results in NullPointerException.

Hide

Permalink

Daniele Romagnoli added a comment - 17/Jul/08 2:42 AM

The bug has been fixed on trunk 2 months ago (revision 30432).
I will backport the fix to 2.4.x.

Show

Daniele Romagnoli added a comment - 17/Jul/08 2:42 AM The bug has been fixed on trunk 2 months ago (revision 30432). I will backport the fix to 2.4.x.

Hide

Permalink

Daniele Romagnoli added a comment - 17/Jul/08 4:25 AM

I have fixed the bug and I have tested the plugin with your data sample.
Now it works fine

Show

Daniele Romagnoli added a comment - 17/Jul/08 4:25 AM I have fixed the bug and I have tested the plugin with your data sample. Now it works fine

People

Assignee:

Daniele Romagnoli

Reporter:

Ryan Hofschneider

Vote (0)

Watch (0)

Dates

Created:

16/Jul/08 7:11 PM

Updated:

22/Oct/08 3:42 AM

Resolved:

17/Jul/08 4:26 AM