[svn-commits] beagles: branch 11 r374019 - in /branches/11: main/message.c res/res_xmpp.c

[SVN commits to the Digium repositories]

Author: beagles
Date: Fri Sep 28 08:02:17 2012
New Revision: 374019

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=374019
Log:
Reset hangup flags on channels created through messages and cleanup globals
in res_xmpp on unload.

This patch fixes an issue where hangup flags were not being reset on a
channel, affecting subsequent use of that channel. The patch also adds some
additional cleanup to res_xmpp to fix an issue with reloading the module.

(closes ASTERISK-20360)
Reported by: Noah Engelberth 
Tested by: beagles
Review: https://reviewboard.asterisk.org/r/2134/

Modified:
    branches/11/main/message.c
    branches/11/res/res_xmpp.c

Modified: branches/11/main/message.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/main/message.c?view=diff&rev=374019&r1=374018&r2=374019
==============================================================================
--- branches/11/main/message.c (original)
+++ branches/11/main/message.c Fri Sep 28 08:02:17 2012
@@ -752,6 +752,10 @@
 	if (msg_ds) {
 		ast_channel_datastore_add(chan, msg_ds);
 	}
+	/*
+	 * Clear softhangup flags.
+	 */
+	ast_channel_clear_softhangup(chan, AST_SOFTHANGUP_ALL);
 
 	ast_channel_unlock(chan);
 }

Modified: branches/11/res/res_xmpp.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/res/res_xmpp.c?view=diff&rev=374019&r1=374018&r2=374019
==============================================================================
--- branches/11/res/res_xmpp.c (original)
+++ branches/11/res/res_xmpp.c Fri Sep 28 08:02:17 2012
@@ -4199,6 +4199,8 @@
 	ast_manager_unregister("JabberSend");
 	ast_custom_function_unregister(&jabberstatus_function);
 	ast_custom_function_unregister(&jabberreceive_function);
+	aco_info_destroy(&cfg_info);
+	ao2_global_obj_release(globals);
 
 	ast_cond_destroy(&message_received_condition);
 	ast_mutex_destroy(&messagelock);