From ecc3f313730dccf19756dabfc6507eebff292648 Mon Sep 17 00:00:00 2001 From: Stefan Knoblich Date: Mon, 14 Jan 2013 15:01:39 +0100 Subject: [PATCH] mod_sofia: Follow-up fix for FS-5009 to avoid NULL-ptr dereference Commit f913670e changed switch_core_session_sprintf() args from sip->sip_to to sip->sip_from, but didn't update the enclosing if() clause protecting the code from dereferencing NULL pointers. Signed-off-by: Stefan Knoblich --- src/mod/endpoints/mod_sofia/sofia.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mod/endpoints/mod_sofia/sofia.c b/src/mod/endpoints/mod_sofia/sofia.c index 68b9a38e38..cdaafff755 100644 --- a/src/mod/endpoints/mod_sofia/sofia.c +++ b/src/mod/endpoints/mod_sofia/sofia.c @@ -7753,7 +7753,7 @@ void sofia_handle_sip_i_invite(switch_core_session_t *session, nua_t *nua, sofia calling_myself++; } - if (sip && sip->sip_to) { + if (sip && sip->sip_from) { user = switch_core_session_sprintf(session, "%s@%s", sip->sip_from->a_url->url_user, sip->sip_from->a_url->url_host); switch_ivr_set_user(session, user); }