`j2s` `array` and `object` generated code leak memory. #17

New issue

Closed

opened 2023-11-09 12:44:27 +00:00 by jordan · 0 comments

jordan commented

2023-11-09 12:44:27 +00:00

Owner

This line of j2s.c generates code that produces a memory leak. So does the analogous line for objects. It leaks a JsonValue in an effort to duplicate the data. This JsonValue should be stored in an intermediate variable and freed when it is no longer needed.

This might actually be trickier than it seems because just calling JsonValueFree() will also free the associated data, which we want to keep. We literally want to free just the JsonValue wrapper and keep the actual data inside of it.

[This line](https://git.telodendria.io/Telodendria/Cytoplasm/src/branch/master/tools/j2s.c#L663) of `j2s.c` generates code that produces a memory leak. So does the analogous line for objects. It leaks a `JsonValue` in an effort to duplicate the data. This `JsonValue` should be stored in an intermediate variable and freed when it is no longer needed. This might actually be trickier than it seems because just calling `JsonValueFree()` will also free the associated data, which we want to keep. We literally want to free *just* the `JsonValue` wrapper and keep the actual data inside of it.