[raster] memory leaks

[yngwar]

Every time when "rt_raster_deserialize(...)" is used in some functions (for example in "RASTER_out") - on exit you destroy memory allocated structures by call "rt_raster_destroy(...)". This function calls "rtdealloc(raster->bands)" to free array of pointers to "rt_band_t". Then it calls "rtdealloc(raster)" to free "rt_raster_t". But you never free the memory allocated by structure "rt_band_t" for each band in band pointer array "raster->bands". Why? I think it couses the memory leaks.

[Bborie Park]

PostgreSQL memory contexts automatically clean up those "dangling" memory allocations when the context itself is freed (I noticed you mentioned RASTER_out). For code running outside of PostgreSQL (such as raster2pgsql and the CUnit unit tests), we explicitly free those memory allocations.

All raster code gets repeatedly tested with valgrind to find these memory issues and I can't find any at the present time.

[robe]

I'm going to close this out - sounds like you aren't seeing anything wrong.