j2s array and object generated code leak memory. #17

New Issue

jordan · 2023-11-09T07:44:27-05:00

jordan commented

2023-11-09 07:44:27 -05:00

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.

jordan added the

bug

label 2023-11-09 07:44:27 -05:00

jordan referenced this issue from Telodendria/Telodendria

2023-11-09 07:45:41 -05:00

Implement Registration Token Administrator API #43

jordan added a new dependency 2023-11-09 07:46:04 -05:00

Telodendria/Telodendria#43 - Implement Registration Token Administrator API

jordan referenced this issue from a commit

2023-11-10 09:28:56 -05:00

Remove TODO comment that isn't relevant.

jordan removed a dependency 2023-11-10 09:30:44 -05:00