Error in printbuf.c

[aestrada]

We came across an error in printbuf.c sprintbuf method call, the free(t) should be CPLFree(t); since a CPL function was used to allocate T. Doing a straight free in my testing is causing a heap error.

[warmerdam]

Corrected in trunk (r22983).

How does it happen that in your environment free() and CPLFree() are doing something different? Are you building the geojson driver as a plugin with a different heap on windows?

My perception is that your problem will not come up commonly in which case there is little need to backport the fix to 1.8, but I'd like to know if it ought to be backported.

[aestrada]

This only shows up on Windows when building from dev studio. I have not seen the same thing when using makefiles on linux.

[Even Rouault]

r23032 /trunk/gdal/port/cpl_string.cpp: CPLVASPrintf() : allocate *buf with CPLStrdup() to make the VSI debugging allocation routines happy, due to change done in r22983 (#4210)

I also believe the r22983 change is not necessary in normal conditions and does not need backporting. I'd assume that aestrada has included the jsonc code in its own project, or build his elasticsearch driver as a plugin.