search

lloyd / yajl

A fast streaming JSON parsing library in C.
http://lloyd.github.com/yajl
ISC License
2.15k stars 435 forks source link

Memory leak in yajl_tree_parse #168

Open hstern opened 9 years ago

hstern commented 9 years ago

The yajl_tree_parse function does not deallocate ctx.stack when yajl_complete_parse returns an error condition, resulting in a memory leak. The solution is to pop all elements off the stack and yajl_tree_free their values.

hstern commented 8 years ago

Rebased against master

ludocode commented 8 years ago

I can confirm that this memory leak still exists in YAJL, and that this fixes it. It would be nice to see this merged.

hstern commented 8 years ago

This has been open for 3 months now and it's an easy fix. What's the hold up?

hstern commented 8 years ago

Checking in. This has been open for 9 months now.

Jesin commented 7 years ago

1 year and 4 months now. Does @lloyd still exist?

cloderic commented 7 years ago

There's been lots of forks of this lib and still no @lloyd maybe it's time to find a maintainer and create an "official" fork.

Jesin commented 7 years ago

Hey @lloyd, you there?

timgabets commented 6 years ago

Hi, Lloyd, is there any chances to have this PR merged?