memory issue for recursive function about yyjson HOT 7 CLOSED

@Edwardmark I don't have much experience with cpp, but you are passing the same doc pointer to the recursive function convertTreeNodeToJSON, is this a problem?

from yyjson.

@nixzhu Thanks for you reply. I want to convert a tree structure to json, so I have to use the same doc pointer. The problem is about the jsonNode or SubNode in the recursion function. I print the value, it shows when the bottom recursion end, the upper function call cannot get their value, print result is null.

from yyjson.

@Edwardmark I don't have much experience with cpp, but you are passing the same doc pointer to the recursive function convertTreeNodeToJSON, is this a problem?

It is not about cpp, I want to verify if the doc owns its subnode memory when calling the recursion function? And what should I do to fix the issue? Thanks .

from yyjson.

@Edwardmark The doc holds memory for all json values, so the recursive call seems good.

There are two string handling issues:

1. yyjson_mut_str() doesn't take ownership of the string. If the passed string is modified or freed during use, errors like \u0000 may occur. Consider using yyjson_mut_strcpy() instead.
2. The string returned by yyjson_mut_val_write() needs to be freed after use.

If you share the TreeNode source code, we can better understand and replicate the issue.

from yyjson.

@ibireme Thanks for your reply. It is the problem. I change yyjson_mut_str() to yyjson_mut_strcpy() and it worked.
And for problem2, how to free the string? Could you give me a example? Thanks.

from yyjson.

@ibireme Thanks for your reply. It is the problem. I change yyjson_mut_str() to yyjson_mut_strcpy() and it worked. And for problem2, how to free the string? Could you give me a example? Thanks.

Just call free(json_str).

from yyjson.

@ibireme Thanks.

from yyjson.