Changeset 734 for pjproject/trunk/pjmedia/src/pjmedia/sdp_neg.c

Timestamp:

Sep 22, 2006 4:55:42 PM (16 years ago)

Author:

bennylp

Message:

Fixed few bugs that seem to have been introduced by new dialog
locking algorithm:

• Fixed crash in PJSUA-API when initiating client subscription
• Fixed another crash in PJSUA-API when hanging-up call

Also fixed SDP negotiator:

• add a=inactive when rejecting media line

Also increase maximum log size from 1500 to 2000 since some
SIP packet is quite large. A little bit of Warning:
THIS MAY AFFECT APPLICATION'S STACK USAGE

File:

• pjproject/trunk/pjmedia/src/pjmedia/sdp_neg.c (modified) (2 diffs)

pjproject/trunk/pjmedia/src/pjmedia/sdp_neg.c

@@ -895 +895 @@
              * Reject the offer by setting the port to zero in the answer.
              */
+            pjmedia_sdp_attr *a;
+
             /* For simplicity in the construction of the answer, we'll
              * just clone the media from the offer. Anyway receiver will
@@ -903 +905 @@
             am->desc.port = 0;
 
-            /* Match direction */
-            update_media_direction(pool, om, am);
+            /* Remove direction attribute, and replace with inactive */
+            remove_all_media_directions(am);
+
+            a = pjmedia_sdp_attr_create(pool, "inactive", NULL);
+            pjmedia_sdp_media_add_attr(am, a);
 
         } else {