davidbyttow / govips

37%

master: 39%

LAST BUILD ON BRANCH fix/memory-leaks

branch: fix/memory-leaks

CHANGE BRANCH

x

Reset

• fix/memory-leaks
• 0.4.4
• c_format
• david/fix-tests
• david/minor-cleanups
• david/revert-obsoletion
• davidbyttow/fix-gnu-warnings
• dependabot/go_modules/golang.org/x/image-0.10.0
• dependabot/go_modules/golang.org/x/image-0.18.0
• dependabot/go_modules/golang.org/x/image-0.5.0
• dependabot/go_modules/golang.org/x/net-0.17.0
• dependabot/go_modules/golang.org/x/net-0.23.0
• dependabot/go_modules/golang.org/x/net-0.7.0
• dependabot/npm_and_yarn/examples/tiff/minimist-1.2.6
• dependabot/npm_and_yarn/examples/tiff/semver-7.5.3
• dependabot/npm_and_yarn/examples/tiff/semver-7.5.4
• dependabot/npm_and_yarn/examples/tiff/sharp-0.30.5
• dependabot/npm_and_yarn/examples/tiff/sharp-0.32.6
• dependabot/npm_and_yarn/examples/tiff/simple-get-4.0.1
• devops-update
• devops-update2
• export_params
• export_tiff
• fix-#144
• fix/247-test-reorg
• fix/275-cmyk-icc-profile
• fix/287-grey-gradient
• fix/306-transparent-canvas
• fix/350-animated-resize-rounding
• fix/359-gif-quality-docs
• fix/381-loop-metadata
• fix/457-jxl-detection
• fix/463-tiff-tile-zero
• fix/489-480-image-interop
• fix/c-code-bugs
• fix/ci-go-version
• fix/keepalive-cgo
• fix/minor-cleanup
• fix/quick-wins
• fix/readme-deprecated-api
• fix/remove-incorrect-defers
• fix/replace-deprecated-getpageheight
• fix/startup-error-handling
• fix/test-panics-to-fatal
• fix_21
• fix_embed_alpha
• gobject_save
• goldentests
• images
• import_params
• importparams
• jake/webp-shrink-on-load
• jammy-ref
• logging
• mac_version
• master
• merge-files
• newgoldens
• reduce-handwritten-c-bridge-code
• refresh
• refs/tags/2.10.0
• refs/tags/v2.1.0
• refs/tags/v2.10.0
• refs/tags/v2.11.0
• refs/tags/v2.12.0
• refs/tags/v2.13.0
• refs/tags/v2.14.0
• refs/tags/v2.15.0
• refs/tags/v2.17.0
• refs/tags/v2.2.0
• refs/tags/v2.3.0
• refs/tags/v2.3.1
• refs/tags/v2.4.0
• refs/tags/v2.5.0
• refs/tags/v2.6.0
• refs/tags/v2.7.0
• refs/tags/v2.8.0
• refs/tags/v2.8.1
• refs/tags/v2.9.0
• revert-merge
• test/tier1-coverage
• toni-test
• toni-testing
• tonimelisma-doubleunref
• tonimelisma-patch-1
• update
• v2.0.0
• v2.0.1
• v2.0.2
• vipsgen
• worktree-fix-avis-detection

Build # 22742083420

Build Type

Pull #510

github

Committed by davidbyttow

Commit Message

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>

Pull Request Pull Request #510: Fix memory leaks in vipsGetPoint and vipsImageGetAsString

Coverage Stats

2626 of 7149 relevant lines covered (36.73%)

0.39 hits per line