From 0f0ff11c6a9a3aca2a1edad6b0105a5757d62d16 Mon Sep 17 00:00:00 2001 From: Sebastian Kemper Date: Fri, 6 Aug 2021 20:41:19 +0200 Subject: [PATCH] sofia-sip: bump to 1.13.4 Fixes some issues, for instance a segfault during shutdown when sofia-sip wasn't fully loaded, see [1]. The segfault was reproducible by simply calling "freeswitch -help". The added patch is to address a library versioning regression, patch is from upstream. [1] https://github.com/freeswitch/sofia-sip/issues/58 Signed-off-by: Sebastian Kemper --- libs/sofia-sip/Makefile | 8 +-- .../patches/02-NUA_handles-mem-leak-fix.patch | 67 ------------------- 2 files changed, 4 insertions(+), 71 deletions(-) delete mode 100644 libs/sofia-sip/patches/02-NUA_handles-mem-leak-fix.patch diff --git a/libs/sofia-sip/Makefile b/libs/sofia-sip/Makefile index 708e88e..d7eac4a 100644 --- a/libs/sofia-sip/Makefile +++ b/libs/sofia-sip/Makefile @@ -11,10 +11,10 @@ PKG_NAME:=sofia-sip PKG_SOURCE_PROTO:=git PKG_SOURCE_URL:=https://github.com/freeswitch/sofia-sip.git -PKG_SOURCE_DATE=2021-02-23 -PKG_SOURCE_VERSION:=d2f3f91bc752f8bf5be74e5baf508100e52f503d -PKG_RELEASE:=2 -PKG_MIRROR_HASH:=9d0ad19c64c20aa7bc139d1e3eee75857d020dcbbd8607df720300c986f7b4a3 +PKG_SOURCE_DATE=2021-06-24 +PKG_SOURCE_VERSION:=f121ce44052831e3a0978fd4ddc306db73763c5d +PKG_RELEASE:=1 +PKG_MIRROR_HASH:=8ffa1c186e8b3e6fdcbdc991e4d63a22c6a0d2571d2ed0f48a2a9385fcbe02c9 # sofia-sip adds a version to include path # need to update this when the version changes diff --git a/libs/sofia-sip/patches/02-NUA_handles-mem-leak-fix.patch b/libs/sofia-sip/patches/02-NUA_handles-mem-leak-fix.patch deleted file mode 100644 index 54cc5b3..0000000 --- a/libs/sofia-sip/patches/02-NUA_handles-mem-leak-fix.patch +++ /dev/null @@ -1,67 +0,0 @@ -commit 8071e8accd250bd1d7758dd16166382b48a6e6d4 -Author: Andrey Volk -Date: Wed Mar 10 18:37:18 2021 +0300 - - NUA handles must be able to unref via stack (nua_handle_unref_user() API) when they are already destroyed or nua is in shutdown - ---- a/libsofia-sip-ua/nua/nua.c -+++ b/libsofia-sip-ua/nua/nua.c -@@ -1165,11 +1165,7 @@ void nua_unref_user(nua_t *nua) - - void nua_handle_unref_user(nua_handle_t *nh) - { -- enter; -- if (NH_IS_VALID(nh)) { -- nua_signal(nh->nh_nua, nh, NULL, nua_r_handle_unref, 0, NULL, TAG_NULL()); -- } -- else { -- SU_DEBUG_1(("nua: nua_r_handle_unref with invalid handle %p\n", (void *)nh)); -- } -+ assert(nh); -+ nh_enter; -+ nua_signal(nh->nh_nua, nh, NULL, nua_r_handle_unref, 0, NULL, TAG_NULL()); - } ---- a/libsofia-sip-ua/nua/nua_common.c -+++ b/libsofia-sip-ua/nua/nua_common.c -@@ -376,6 +376,8 @@ char const *nua_event_name(nua_event_t e - case nua_r_nit_respond: return "nua_r_nit_respond"; - case nua_r_set_params: return "nua_r_set_params"; - case nua_r_ack: return "nua_r_ack"; -+ case nua_r_handle_unref: return "nua_r_handle_unref"; -+ case nua_r_unref: return "nua_r_unref"; - default: return "NUA_UNKNOWN"; - } - } ---- a/libsofia-sip-ua/nua/nua_stack.c -+++ b/libsofia-sip-ua/nua/nua_stack.c -@@ -514,7 +514,7 @@ int nua_signal(nua_t *nua, nua_handle_t - - assert(tend == t); (void)tend; assert(b == bend); (void)bend; - -- e->e_always = event == nua_r_destroy || event == nua_r_shutdown; -+ e->e_always = event == nua_r_destroy || event == nua_r_shutdown || event == nua_r_handle_unref || event == nua_r_unref; - e->e_event = event; - e->e_nh = nh ? nua_handle_ref(nh) : NULL; - e->e_status = status; -@@ -549,10 +549,10 @@ void nua_stack_signal(nua_t *nua, su_msg - nua_event_data_t *e = ee->ee_data; - nua_handle_t *nh = e->e_nh; - tagi_t *tags = e->e_tags; -- nua_event_t event; -+ nua_event_t event = (enum nua_event_e)e->e_event; - int error = 0; - -- if (nh) { -+ if (nh && event != nua_r_handle_unref) { - if (!nh->nh_prev) - nh_append(nua, nh); - if (!nh->nh_ref_by_stack) { -@@ -575,8 +575,6 @@ void nua_stack_signal(nua_t *nua, su_msg - - su_msg_save(nua->nua_signal, msg); - -- event = (enum nua_event_e)e->e_event; -- - if (nua->nua_shutdown && !e->e_always) { - /* Shutting down */ - nua_stack_event(nua, nh, NULL, event, -- 2.30.2