Merge pull request #44 from LarryLaffer-dev/diameter_crash

LarryLaffer-dev · web-flow · commit 02e36f36bf58 · 2026-01-28T12:29:19.000+01:00
Fix memory leak by setting avps to NULL after deletion in dm_send_req…
diff --git a/modules/aaa_diameter/aaa_diameter.c b/modules/aaa_diameter/aaa_diameter.c
@@ -329,6 +329,7 @@ static int dm_send_request(struct sip_msg *msg, int *app_id, int *cmd_code,
 		goto error;
 	}
 	cJSON_Delete(avps);
+	avps = NULL;
 
 	if (_dm_send_message(NULL, dmsg, &rpl) != 0)
 		goto error;

Original file line number	Diff line number	Diff line change
`@@ -329,6 +329,7 @@ static int dm_send_request(struct sip_msg msg, int app_id, int *cmd_code,`
`329`	`329`	`goto error;`
`330`	`330`	`}`
`331`	`331`	`cJSON_Delete(avps);`
	`332`	`+ avps = NULL;`
`332`	`333`
`333`	`334`	`if (_dm_send_message(NULL, dmsg, &rpl) != 0)`
`334`	`335`	`goto error;`