[asterisk-commits] mmichelson: trunk r143609 - /trunk/channels/chan_agent.c

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Fri Sep 19 10:43:28 2008
New Revision: 143609

URL: http://svn.digium.com/view/asterisk?view=rev&rev=143609
Log:
We should only unsubscribe to the device state event
subscription if we have previously subscribed. Otherwise
a segfault will occur.

(closes issue #13476)
Reported by: jonnt
Patches:
      13476.patch uploaded by putnopvut (license 60)
Tested by: jonnt


Modified:
    trunk/channels/chan_agent.c

Modified: trunk/channels/chan_agent.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_agent.c?view=diff&rev=143609&r1=143608&r2=143609
==============================================================================
--- trunk/channels/chan_agent.c (original)
+++ trunk/channels/chan_agent.c Fri Sep 19 10:43:28 2008
@@ -2535,7 +2535,9 @@
 	/* First, take us out of the channel loop */
 	ast_channel_unregister(&agent_tech);
 	/* Delete devicestate subscription */
-	agent_devicestate_sub = ast_event_unsubscribe(agent_devicestate_sub);
+	if (agent_devicestate_sub) {
+		agent_devicestate_sub = ast_event_unsubscribe(agent_devicestate_sub);
+	}
 	/* Unregister dialplan functions */
 	ast_custom_function_unregister(&agent_function);	
 	/* Unregister CLI commands */