Summary:	ASTERISK-21906: [patch] Fix memory leaks, invalid reads and more reported by valgrind
Reporter:	John Hardin (jhardin)	Labels:
Date Opened:	2013-06-13 14:23:16	Date Closed:	2013-06-24 20:14:36
Priority:	Major	Regression?
Status:	Closed/Complete	Components:	General
Versions:	12	Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) cel.patch ( 1) chan_motif.patch ( 2) config_options.patch ( 3) devicestate.patch ( 4) http.patch ( 5) manager.patch ( 6) named_acl.patch ( 7) presencestate.patch ( 8) res_calendar.patch ( 9) retriever-channel-snapshot.patch (10) sorcery.patch
Description:	Attached are several patches to fix memory leaks and invalid reads reported by valgrind.
Comments:	By: John Hardin (jhardin) 2013-06-13 14:28:03.556-0500 The changes to aco_option_register_custom() fix an invalid read past the end of allocated memory due to the missing default value. By: John Hardin (jhardin) 2013-06-13 14:31:08.698-0500 The while-loop was exited before types[0]->internal->opts was processed. ao2_unref() was moved out of the loop to avoid multiple derefs. By: Matt Jordan (mjordan) 2013-06-24 18:28:58.363-0500 * cel.c - fixed in trunk * chan_motif.c - fixed in 11+ * config_options.c - fixed in 11+ * devicestate.c - fixed in trunk * http.c - fixed in 11+ * manager.c - fixed in trunk * named_acl.c - fixed in 11+ * presencestate.c - fixed in trunk * res_calendar.c - fixed in 11+ * parking - fixed in trunk * sorcery - fixed in trunk By: Matt Jordan (mjordan) 2013-06-24 20:14:36.236-0500 Fixed in both 11 and trunk.