Changeset 2869 for pjproject/trunk/pjsip/src/pjsua-lib/pjsua_call.c

Timestamp:

Aug 12, 2009 5:53:47 PM (15 years ago)

Author:

bennylp

Message:

Ticket #877: Memory consumption of the invite session grows indefinitely if call is running for long period of time and with many re-INVITES

• introducing flip-flop pools in the pjsip_inv_session. There are two additional pools created, and one of them will be reset everytime SDP negotiation is done to release memory back to the OS

File:

• pjproject/trunk/pjsip/src/pjsua-lib/pjsua_call.c (modified) (6 diffs)

pjproject/trunk/pjsip/src/pjsua-lib/pjsua_call.c

@@ -1597 +1597 @@
         status = create_sdp_of_call_hold(call, &sdp);
     } else {
-        status = pjsua_media_channel_create_sdp(call->index, call->inv->pool, 
+        status = pjsua_media_channel_create_sdp(call->index, 
+                                                call->inv->pool_prov,
                                                 NULL, &sdp, NULL);
         call->local_hold = PJ_FALSE;
@@ -1656 +1657 @@
 
     /* Create SDP */
-    status = pjsua_media_channel_create_sdp(call->index, call->inv->pool, 
+    status = pjsua_media_channel_create_sdp(call->index, 
+                                            call->inv->pool_prov, 
                                             NULL, &sdp, NULL);
     if (status != PJ_SUCCESS) {
@@ -3201 +3203 @@
     pjmedia_sdp_session *sdp;
 
-    /* Use call's pool */
-    pool = call->inv->pool;
+    /* Use call's provisional pool */
+    pool = call->inv->pool_prov;
 
     /* Create new offer */
@@ -3268 +3270 @@
               call->index));
 
-    status = pjsua_media_channel_create_sdp(call->index, call->inv->pool, 
+    status = pjsua_media_channel_create_sdp(call->index, 
+                                            call->inv->pool_prov, 
                                             offer, &answer, NULL);
     if (status != PJ_SUCCESS) {
@@ -3297 +3300 @@
          * (See RFC 3264 Section 8.4 and RFC 4317 Section 3.1)
          */
-        attr = pjmedia_sdp_attr_create(call->inv->pool, "sendonly", NULL);
+        attr = pjmedia_sdp_attr_create(call->inv->pool_prov, "sendonly", NULL);
         pjmedia_sdp_media_add_attr(answer->media[0], attr);
     }
@@ -3335 +3338 @@
                   call->index));
 
-        status = pjsua_media_channel_create_sdp(call->index, call->inv->pool, 
+        status = pjsua_media_channel_create_sdp(call->index, 
+                                                call->inv->pool_prov, 
                                                 NULL, offer, NULL);
     }