#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 |
| Cc: |
Description
QGIS bug http://trac.osgeo.org/qgis/ticket/2303
It is just stupid bug in GRASS driver, G_free_key_value() called on NULL, patch attached.
May I submit to svn?
Attachments (1)
Change History (6)
by , 13 years ago
| Attachment: | grass57dataset.cpp.patch-noproj added |
|---|
comment:1 by , 13 years ago
| Component: | default → GDAL_Raster |
|---|---|
| Keywords: | grass added |
comment:2 by , 13 years ago
Radim,
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 , 13 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
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 , 13 years ago
Radim,
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 , 13 years ago
The issue has been fixed in GRASS (https://trac.osgeo.org/grass/ticket/866)
Radim,
Please go ahead, and it may be appropriate to backport to 1.6.