Revision
16871
Author
anthm
Date
2010-03-02 17:35:26 -0600 (Tue, 02 Mar 2010)

Log Message

re-arrange location of events

Modified Paths

Diff

Modified: freeswitch/trunk/src/switch_core_media_bug.c (16870 => 16871)


--- freeswitch/trunk/src/switch_core_media_bug.c	2010-03-02 21:14:46 UTC (rev 16870)
+++ freeswitch/trunk/src/switch_core_media_bug.c	2010-03-02 23:35:26 UTC (rev 16871)
@@ -37,6 +37,8 @@
 
 static void switch_core_media_bug_destroy(switch_media_bug_t *bug)
 {
+	switch_event_t *event = NULL;
+
 	if (bug->raw_read_buffer) {
 		switch_buffer_destroy(&bug->raw_read_buffer);
 	}
@@ -44,6 +46,13 @@
 	if (bug->raw_write_buffer) {
 		switch_buffer_destroy(&bug->raw_write_buffer);
 	}
+
+	if (switch_event_create(&event, SWITCH_EVENT_MEDIA_BUG_STOP) == SWITCH_STATUS_SUCCESS) {
+		switch_event_add_header(event, SWITCH_STACK_BOTTOM, "Media-Bug-Function", "%s", bug->function);
+		switch_event_add_header(event, SWITCH_STACK_BOTTOM, "Media-Bug-Target", "%s", bug->target);
+		if (bug->session) switch_channel_event_set_data(bug->session->channel, event);
+		switch_event_fire(&event);
+	}
 }
 
 SWITCH_DECLARE(void) switch_core_media_bug_pause(switch_core_session_t *session)
@@ -357,13 +366,12 @@
 	*new_bug = bug;
 
 	if (switch_event_create(&event, SWITCH_EVENT_MEDIA_BUG_START) == SWITCH_STATUS_SUCCESS) {
-		switch_channel_event_set_data(session->channel, event);
 		switch_event_add_header(event, SWITCH_STACK_BOTTOM, "Media-Bug-Function", "%s", bug->function);
 		switch_event_add_header(event, SWITCH_STACK_BOTTOM, "Media-Bug-Target", "%s", bug->target);
+		switch_channel_event_set_data(session->channel, event);
 		switch_event_fire(&event);
 	}
 
-
 	return SWITCH_STATUS_SUCCESS;
 }
 
@@ -461,19 +469,7 @@
 		}
 		switch_thread_rwlock_unlock(session->bug_rwlock);
 		if (bp) {
-			switch_event_t *event = NULL;
-
-			if (switch_event_create(&event, SWITCH_EVENT_MEDIA_BUG_STOP) == SWITCH_STATUS_SUCCESS) {
-				switch_channel_event_set_data(session->channel, event);
-				switch_event_add_header(event, SWITCH_STACK_BOTTOM, "Media-Bug-Function", "%s", bp->function);
-				switch_event_add_header(event, SWITCH_STACK_BOTTOM, "Media-Bug-Target", "%s", bp->target);
-			}
-
-			if ((status = switch_core_media_bug_close(&bp)) == SWITCH_STATUS_SUCCESS) {
-				switch_event_fire(&event);
-			} else {
-				switch_event_destroy(&event);
-			}
+			status = switch_core_media_bug_close(&bp);
 		}
 	}