[sr-dev] git:master:6711306d: rtpengine: Fix coredump on kamailio shutdown

Stefan Mititelu stefan.mititelu at 1and1.ro
Wed Jan 27 09:42:51 CET 2016


Module: kamailio
Branch: master
Commit: 6711306d48be16fcf1dff860f72d0db14cac02fc
URL: https://github.com/kamailio/kamailio/commit/6711306d48be16fcf1dff860f72d0db14cac02fc

Author: Stefan Mititelu <stefan.mititelu at 1and1.ro>
Committer: Stefan Mititelu <stefan.mititelu at 1and1.ro>
Date: 2016-01-27T10:41:06+02:00

rtpengine: Fix coredump on kamailio shutdown

Fix release_lock leading to coredump, when kamailio shutdown.

---

Modified: modules/rtpengine/rtpengine.c

---

Diff:  https://github.com/kamailio/kamailio/commit/6711306d48be16fcf1dff860f72d0db14cac02fc.diff
Patch: https://github.com/kamailio/kamailio/commit/6711306d48be16fcf1dff860f72d0db14cac02fc.patch

---

diff --git a/modules/rtpengine/rtpengine.c b/modules/rtpengine/rtpengine.c
index 1782258..30db81a 100644
--- a/modules/rtpengine/rtpengine.c
+++ b/modules/rtpengine/rtpengine.c
@@ -1951,16 +1951,16 @@ static void mod_destroy(void)
 
 	lock_get(rtpp_set_list->rset_head_lock);
 	for(crt_list = rtpp_set_list->rset_first; crt_list != NULL; ){
+		last_list = crt_list;
 
 		if (!crt_list->rset_lock) {
-			last_list = crt_list;
 			crt_list = last_list->rset_next;
 			shm_free(last_list);
 			last_list = NULL;
 			continue;
 		}
 
-		lock_get(crt_list->rset_lock);
+		lock_get(last_list->rset_lock);
 		for(crt_rtpp = crt_list->rn_first; crt_rtpp != NULL;  ){
 
 			if(crt_rtpp->rn_url.s)
@@ -1970,9 +1970,8 @@ static void mod_destroy(void)
 			crt_rtpp = last_rtpp->rn_next;
 			shm_free(last_rtpp);
 		}
-		last_list = crt_list;
 		crt_list = last_list->rset_next;
-		lock_release(crt_list->rset_lock);
+		lock_release(last_list->rset_lock);
 
 		lock_destroy(last_list->rset_lock);
 		lock_dealloc((void*)last_list->rset_lock);




More information about the sr-dev mailing list