[Freeswitch-svn] [commit] r13337 - in freeswitch/trunk/libs/sofia-sip: . libsofia-sip-ua/iptsec

[FreeSWITCH SVN]

Author: mikej
Date: Fri May 15 11:05:59 2009
New Revision: 13337

Log:

Wed May 13 11:00:40 CDT 2009  Pekka Pessi <first.last at nokia.com>
  * auth_client.c: removed leak in auc_digest_authorization()
  Ignore-this: 9b0297083b6c6ce1bf9ef3d723b01f3b
  
  Coverity issue.


Modified:
   freeswitch/trunk/libs/sofia-sip/.update
   freeswitch/trunk/libs/sofia-sip/libsofia-sip-ua/iptsec/auth_client.c

Modified: freeswitch/trunk/libs/sofia-sip/.update
==============================================================================
--- freeswitch/trunk/libs/sofia-sip/.update	(original)
+++ freeswitch/trunk/libs/sofia-sip/.update	Fri May 15 11:05:59 2009
@@ -1 +1 @@
-Fri May 15 11:04:52 CDT 2009
+Fri May 15 11:05:48 CDT 2009

Modified: freeswitch/trunk/libs/sofia-sip/libsofia-sip-ua/iptsec/auth_client.c
==============================================================================
--- freeswitch/trunk/libs/sofia-sip/libsofia-sip-ua/iptsec/auth_client.c	(original)
+++ freeswitch/trunk/libs/sofia-sip/libsofia-sip-ua/iptsec/auth_client.c	Fri May 15 11:05:59 2009
@@ -919,9 +919,9 @@
   auth_challenge_t const *ac = cda->cda_ac;
   char const *cnonce = cda->cda_cnonce;
   unsigned nc = ++cda->cda_ncount;
-  char *uri = url_as_string(home, url);
   void const *data = body ? body->pl_data : "";
   usize_t dlen = body ? body->pl_len : 0;
+  char *uri;
 
   msg_header_t *h;
   auth_hexmd5_t sessionkey, response;
@@ -942,7 +942,10 @@
   ar->ar_qop = NULL;
   ar->ar_auth = ac->ac_auth;
   ar->ar_auth_int = ac->ac_auth_int;
-  ar->ar_uri = uri;
+  ar->ar_uri = uri = url_as_string(home, url);
+
+  if (ar->ar_uri == NULL)
+    return -1;
 
   /* If there is no qop, we MUST NOT include cnonce or nc */
   if (!ar->ar_auth && !ar->ar_auth_int)