Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Displaying raster in a Location with undefined projection causes QGIS to crash #12363

Closed
qgib opened this issue Dec 19, 2009 · 5 comments
Closed
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! GRASS
Milestone

Comments

@qgib
Copy link
Contributor

qgib commented Dec 19, 2009

Author Name: Micha Silver (@micha-silver)
Original Redmine Issue: 2303

Redmine category:grass
Assignee: Lorenzo Masini


When working in a Location with no projection information (an "X-Y" Location), you can display vector maps, but trying to display a raster causes QGIS to crash. Same results in both Linux and windows.

Recreate the issue as follows:
Using the "Create new Mapset" wizard create a new location with undefined CRS. Open a GRASS teminal and run r.mapcalc "test=1" (or equivalent in the GRASS Toolbox).
Trying to add the test raster causes the crash. Vectors display properly.

The same Location, mapset, and raster will display properly in GRASS itself - outside of QGIS.

@qgib
Copy link
Contributor Author

qgib commented Jan 11, 2010

Author Name: Redmine Admin (Redmine Admin)


It is bug in GDAL GRASS driver, I have reported the bug in GDAL trac http://trac.osgeo.org/gdal/ticket/3323 with patch attached.

Unfortunately there is no workaround in QGIS, I'll keep it open at least until the patch is applied in GDAL.

@qgib
Copy link
Contributor Author

qgib commented Jan 11, 2010

Author Name: Mateusz Loskot - (Mateusz Loskot -)


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, as standard free() promises.

@qgib
Copy link
Contributor Author

qgib commented Jan 12, 2010

Author Name: Redmine Admin (Redmine Admin)


Yes, better to fix also in GRASS, but it would take too long to wait for it in distributions.

I close it, the fix is applied in GDAL trunk.


  • resolution was changed from to fixed
  • status_id was changed from Open to Closed

@qgib
Copy link
Contributor Author

qgib commented Jan 12, 2010

Author Name: Mateusz Loskot - (Mateusz Loskot -)


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.

@qgib
Copy link
Contributor Author

qgib commented Jan 12, 2010

Author Name: Mateusz Loskot - (Mateusz Loskot -)


The issue has been fixed in GRASS (https://trac.osgeo.org/grass/ticket/866)

@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! GRASS labels May 24, 2019
@qgib qgib added this to the Version 1.5.0 milestone May 24, 2019
@qgib qgib closed this as completed May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! GRASS
Projects
None yet
Development

No branches or pull requests

1 participant