[asterisk-commits] mmichelson: branch group/CCSS r236755 - /team/group/CCSS/channels/chan_sip.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Dec 28 16:48:33 CST 2009


Author: mmichelson
Date: Mon Dec 28 16:48:32 2009
New Revision: 236755

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=236755
Log:
Fix some refcount bugs.


Modified:
    team/group/CCSS/channels/chan_sip.c

Modified: team/group/CCSS/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/team/group/CCSS/channels/chan_sip.c?view=diff&rev=236755&r1=236754&r2=236755
==============================================================================
--- team/group/CCSS/channels/chan_sip.c (original)
+++ team/group/CCSS/channels/chan_sip.c Mon Dec 28 16:48:32 2009
@@ -3745,6 +3745,7 @@
 		/* We haven't yet allocated the suspension entry, so let's give it a shot */
 		if (!(monitor_instance->suspension_entry = create_epa_entry("call-completion", monitor_instance->notify_uri))) {
 			ast_log(LOG_WARNING, "Unable to allocate sip EPA entry for call-completion\n");
+			ao2_ref(monitor_instance, -1);
 			return -1;
 		}
 		publish_type = SIP_PUBLISH_INITIAL;
@@ -3755,10 +3756,7 @@
 	construct_pidf_body(CC_CLOSED, pidf_body, sizeof(pidf_body));
 	transmit_publish(monitor_instance->suspension_entry, publish_type, pidf_body);
 
-	/* Find the appropriate sip_monitor_instance based on the given core_id.
-	 * Allocate teh sip_monitor_instance publish_pvt
-	 * Send a PUBLISH to the notify_uri with PIDF state "closed"
-	 */
+	ao2_ref(monitor_instance, -1);
 	return 0;
 }
 
@@ -3784,6 +3782,7 @@
 	construct_pidf_body(CC_OPEN, pidf_body, sizeof(pidf_body));
 	transmit_publish(monitor_instance->suspension_entry, SIP_PUBLISH_MODIFY, pidf_body);
 
+	ao2_ref(monitor_instance, -1);
 	return 0;
 }
 




More information about the asterisk-commits mailing list