Changeset 4399

Timestamp:

Feb 27, 2013 2:26:03 PM (12 years ago)

Author:

riza

Message:

Re #1609: backported to 1.x

Location:

pjproject/branches/1.x

Files:

• . (modified) (1 prop)
• pjsip/src/pjsip/sip_auth_client.c (modified) (2 diffs)
• pjsip/src/pjsip/sip_tel_uri.c (modified) (2 diffs)

pjproject/branches/1.x/pjsip/src/pjsip/sip_auth_client.c

@@ -921 +921 @@
          */
         unsigned i;
-        char *uri_str;
-        int len;
-
-        uri_str = (char*)pj_pool_alloc(tdata->pool, PJSIP_MAX_URL_SIZE);
-        len = pjsip_uri_print(PJSIP_URI_IN_REQ_URI, tdata->msg->line.req.uri,
-                              uri_str, PJSIP_MAX_URL_SIZE);
-        if (len < 1 || len >= PJSIP_MAX_URL_SIZE)
+        pj_str_t uri;
+
+        uri.ptr = (char*)pj_pool_alloc(tdata->pool, PJSIP_MAX_URL_SIZE);
+        uri.slen = pjsip_uri_print(PJSIP_URI_IN_REQ_URI,
+                                   tdata->msg->line.req.uri,
+                                   uri.ptr, PJSIP_MAX_URL_SIZE);
+        if (uri.slen < 1 || uri.slen >= PJSIP_MAX_URL_SIZE)
             return PJSIP_EURITOOLONG;
 
@@ -947 +947 @@
                 pj_strdup(tdata->pool, &hs->credential.digest.realm,
                           &c->realm);
-                pj_strdup2(tdata->pool, &hs->credential.digest.uri,
-                           uri_str);
+                pj_strdup(tdata->pool, &hs->credential.digest.uri, &uri);
                 pj_strdup(tdata->pool, &hs->credential.digest.algorithm,
                           &sess->pref.algorithm);

pjproject/branches/1.x/pjsip/src/pjsip/sip_tel_uri.c

@@ -183 +183 @@
     int printed;
     char *startbuf = buf;
-    char *endbuf = buf+size;
+    char *endbuf = buf+size-1;
     const pjsip_parser_const_t *pc = pjsip_parser_const();
 
@@ -217 +217 @@
         return -1;
     buf += printed;
+
+    *buf = '\0';
 
     return (buf-startbuf);