[asterisk-commits] russell: trunk r65375 - /trunk/apps/app_queue.c

[asterisk-commits at lists.digium.com]

Author: russell
Date: Tue May 22 07:52:03 2007
New Revision: 65375

URL: http://svn.digium.com/view/asterisk?view=rev&rev=65375
Log:
Fix a couple of spots in the handling of device states that could lead to a
double free.  (issue #9772, reported by Mike Anikienko, fix by me)

Modified:
    trunk/apps/app_queue.c

Modified: trunk/apps/app_queue.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_queue.c?view=diff&rev=65375&r1=65374&r2=65375
==============================================================================
--- trunk/apps/app_queue.c (original)
+++ trunk/apps/app_queue.c Tue May 22 07:52:03 2007
@@ -551,7 +551,6 @@
 	if (loc) {
 		*loc++ = '\0';
 	} else {
-		free(sc);
 		return NULL;
 	}
 
@@ -572,7 +571,6 @@
 	if (!curint) {
 		if (option_debug > 2)
 			ast_log(LOG_DEBUG, "Device '%s/%s' changed to state '%d' (%s) but we don't care because they're not a member of any queue.\n", technology, loc, sc->state, devstate2str(sc->state));
-		free(sc);
 		return NULL;
 	}