Skip to content

Fix Valgrind leaks in examples; use __jule_Str for conversions#2

Open
pgmtx wants to merge 1 commit into
koibtw:mainfrom
pgmtx:update-curlwrapper
Open

Fix Valgrind leaks in examples; use __jule_Str for conversions#2
pgmtx wants to merge 1 commit into
koibtw:mainfrom
pgmtx:update-curlwrapper

Conversation

@pgmtx
Copy link
Copy Markdown
Contributor

@pgmtx pgmtx commented May 18, 2026

I found memory leaks when running the examples under Valgrind: some error/early-return paths missed curl_slist_free_all and curl_easy_cleanup calls. This code fixes those leaks and removes the small strToString helper in favor of __jule_Str casts.

Notes:

  • Behavior unchanged except proper cleanup.
  • Small downside: more .get() calls in places.
  • Tested with Valgrind; leak traces I saw are gone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant