From: William King <william.king@quentustech.com>
Date: Sun, 19 May 2013 02:13:18 +0000 (-0700)
Subject: full_contact must be freed by sofia. There isn't a chance of this having blown anythi... 
X-Git-Tag: v1.5.1~44
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9ace496fe305c428fbfeca8c9b340e6aeb7b387f;p=thirdparty%2Ffreeswitch.git

full_contact must be freed by sofia. There isn't a chance of this having blown anything up, but Coverity is right that this should be initialized with NULL to be safe against future refactors.
---

diff --git a/src/mod/endpoints/mod_sofia/sofia.c b/src/mod/endpoints/mod_sofia/sofia.c
index 88b014c9be..696579b1b9 100644
--- a/src/mod/endpoints/mod_sofia/sofia.c
+++ b/src/mod/endpoints/mod_sofia/sofia.c
@@ -5462,7 +5462,7 @@ static void sofia_handle_sip_r_invite(switch_core_session_t *session, int status
 				if (profile->pres_type) {
 					const char *presence_data = switch_channel_get_variable(channel, "presence_data");
 					const char *presence_id = switch_channel_get_variable(channel, "presence_id");
-					char *full_contact = "";
+					char *full_contact = NULL;
 					char *p = NULL;
 					time_t now;