davidbyttow / govips / 22742309762

37%

push

github

Fix memory leaks in vipsGetPoint and vipsImageGetAsString (#510)

* Fix memory leak in vipsGetPoint

The deferred g_free captured a nil pointer (evaluated at defer time,
before the C call populated it), so the C-allocated array was never
freed. Additionally, the returned Go slice was backed by C memory that
could be freed or reused.

Fix: copy C data into a Go-owned slice, then free the C allocation
immediately.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix memory leak in vipsImageGetAsString

The deferred freeCString captured a nil pointer (evaluated at defer
time, before the C call populated it), so the g_free-allocated string
from vips_image_get_as_string was never freed.

Fix: wrap in a closure so the variable is captured by reference and
evaluated when the deferred function actually executes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

5 of 5 new or added lines in 1 file covered. (100.0%)

2626 of 7149 relevant lines covered (36.73%)

0.39 hits per line