Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#3323 closed defect (fixed)

GRASS driver crash for location without projection

Reported by: rblazek Owned by: warmerdam
Priority: normal Milestone:
Component: GDAL_Raster Version: unspecified
Severity: normal Keywords: grass


QGIS bug

It is just stupid bug in GRASS driver, G_free_key_value() called on NULL, patch attached.

May I submit to svn?

Attachments (1)

grass57dataset.cpp.patch-noproj (653 bytes ) - added by rblazek 14 years ago.

Download all attachments as: .zip

Change History (6)

by rblazek, 14 years ago

comment:1 by warmerdam, 14 years ago

Component: defaultGDAL_Raster
Keywords: grass added


Please go ahead, and it may be appropriate to backport to 1.6.

comment:2 by Mateusz Łoskot, 14 years ago


Wouldn't it be simpler to fix G_free_key_value function in GRASS to follow semantic of free() and do nothing for null pointer? The G_free in GRASS already does nothing for null pointer.

comment:3 by rblazek, 14 years ago

Resolution: fixed
Status: newclosed

It would be definitely good to fix also in GRASS lib but simpler no because you have to wait ages until it gets into distros.

Fixed in changeset:18537.

comment:4 by Mateusz Łoskot, 14 years ago


Understood, though I still suggest to apply the fix to GRASS as well. So, once new fixed version is released, messing 3rd parties code is no longer needed.

comment:5 by Mateusz Łoskot, 14 years ago

The issue has been fixed in GRASS (

Note: See TracTickets for help on using tickets.