#866: Semantic of free() and G_free() for G_free_key_value()
-------------------------+--------------------------------------------------
Reporter: mloskot | Owner: grass-dev@lists.osgeo.org
Type: defect | Status: new
Priority: normal | Milestone:
Component: libgis | Version: svn-trunk
Keywords: gdal | Platform: All
Cpu: Unspecified |
-------------------------+--------------------------------------------------
Recently, a bug has been submitted [http://trac.osgeo.org/qgis/ticket/2303
to QGIS] and forwarded [http://trac.osgeo.org/gdal/ticket/3323 to GDAL].
IMHO, the problem is in GRASS.
The function is [source:grass/trunk/lib/gis/key_value1.c?@35855#L149
G_free_key_value] not proof against nullptr and, given purpose of this
function, if pointer to key is null, it should do nothing. This is a
standard semantic of free() function in C library. Moreover, G_free
function already follows semantic of free(), so G_free_key_value() should
do it as well.
#866: Semantic of free() and G_free() for G_free_key_value()
----------------------+-----------------------------------------------------
Reporter: mloskot | Owner: grass-dev@lists.osgeo.org
Type: defect | Status: new
Priority: normal | Milestone:
Component: libgis | Version: svn-trunk
Resolution: | Keywords: gdal
Platform: All | Cpu: Unspecified
----------------------+-----------------------------------------------------
Comment (by glynn):
Replying to [ticket:866 mloskot]:
> The function is [source:grass/trunk/lib/gis/key_value1.c?@35855#L149
G_free_key_value] not proof against nullptr and, given purpose of this
function, if pointer to key is null, it should do nothing. This is a
standard semantic of free() function in C library. Moreover, G_free
function already follows semantic of free(), so G_free_key_value() should
do it as well.