From: Tamas Cseke <tamas.cseke@virtual-call-center.eu>
Date: Thu, 24 May 2012 14:19:01 +0000 (+0200)
Subject: doh FS-3423
X-Git-Tag: v1.2.0~290^2~17
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9dc9ee974c3de9246daff6551c46a14ffeefa61c;p=thirdparty%2Ffreeswitch.git

doh FS-3423
---

diff --git a/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.c b/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.c
index dd84d8de11..fc38f9e2b2 100644
--- a/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.c
+++ b/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.c
@@ -320,7 +320,6 @@ static void add_session_elem_to_listener(listener_t *listener, session_elem_t *s
 }
 
 
-/* TODO lock */
 static void remove_session_elem_from_listener(listener_t *listener, session_elem_t *session_element)
 {
 	switch_core_hash_delete(listener->sessions, session_element->uuid_str);
@@ -789,9 +788,8 @@ static void handle_exit(listener_t *listener, erlang_pid * pid)
 	/* TODO - eliminate session destroy races and we shouldn't lock the session hash */
 	switch_thread_rwlock_wrlock(listener->session_rwlock);
 	if ((s = find_session_elem_by_pid(listener, pid))) {
-		if (remove_session_elem_from_listener(listener, s) == SWITCH_STATUS_SUCCESS) {
-			destroy_session_elem(s);
-		}
+		remove_session_elem_from_listener(listener, s);
+		destroy_session_elem(s);
 	}
 	switch_thread_rwlock_wrlock(listener->session_rwlock);