sysmocom
/
asterisk
11
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

func_json: Fix memory leak. 

A memory leak was present in func_json due to
using ast_json_free, which just calls ast_free,
as opposed to recursively freeing the JSON
object as needed. This is now fixed to use the
right free functions.

ASTERISK-30293 #close

Change-Id: I982324dde841dc9147c8d8ad35c8719daf418b49
This commit is contained in:
Naveen Albert 2022-11-10 12:11:53 +00:00 committed by Friendly Automation
parent 1e77b8c473
commit 002afc3f2a
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
funcs/func_json.c
View File

@ -115,7 +115,7 @@ static int json_decode_read(struct ast_channel *chan, const char *cmd, char *dat
jsonval = ast_json_object_get(json, args.key);
if (!jsonval) { /* no error or warning should be thrown */
ast_debug(1, "Could not find key '%s' in parsed JSON\n", args.key);
ast_json_free(json);
ast_json_unref(json);
return -1;
}
switch(ast_json_typeof(jsonval)) {
@ -134,7 +134,7 @@ static int json_decode_read(struct ast_channel *chan, const char *cmd, char *dat
ast_json_free(result2);
break;
}
ast_json_free(json);
ast_json_unref(json);
return 0;
}