Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#2721 closed defect (fixed)

Use GRASS_GISBASE for GDAL GRASS driver

Reported by: Even Rouault Owned by: Even Rouault
Priority: normal Milestone: 1.7.0
Component: default Version: unspecified
Severity: normal Keywords: grass GRASS_GISBASE
Cc: warmerdam, kyngchaos

Description

This ticket is to track implementation of William's suggestion :

The configured GRASS prefix is already compiled into the OGR GRASS  
driver as GRASS_GISBASE, why not do the same for the GDAL GRASS  
driver?  (I thought it was, but GRASS_GISBASE only turns up in  
ogrgrassdatasource.cpp).

Change History (2)

comment:1 by Even Rouault, 15 years ago

Resolution: fixed
Status: newclosed

Use GRASS_GISBASE for GDAL GRASS driver instead of hard-coded path in trunk (r15901)

Minor fix in trunk (r15902) for OGR GRASS driver to prevent memory leak and potential buffer overflow

comment:2 by kyngchaos, 15 years ago

Tested patch on 1.6, autotest grass_2 now succeeds.

thanks

Note: See TracTickets for help on using tickets.