Changeset 3360 for pjproject/trunk/pjmedia/src/pjmedia/endpoint.c

Timestamp:

Nov 3, 2010 6:46:27 AM (13 years ago)

Author:

bennylp

Message:

Fixed #1154 (Run-time option to disable telephone-event in outgoing SDP offer (thanks Marcus Froeschl for the suggestion))

File:

• pjproject/trunk/pjmedia/src/pjmedia/endpoint.c (modified) (4 diffs)

pjproject/trunk/pjmedia/src/pjmedia/endpoint.c

@@ -88 +88 @@
     /** To signal polling thread to quit. */
     pj_bool_t             quit_flag;
+
+    /** Is telephone-event enable */
+    pj_bool_t             has_telephone_event;
 };
 
@@ -121 +124 @@
     endpt->ioqueue = ioqueue;
     endpt->thread_cnt = worker_cnt;
+    endpt->has_telephone_event = PJ_TRUE;
 
     /* Sound */
@@ -222 +226 @@
 }
 
+PJ_DEF(pj_status_t) pjmedia_endpt_set_flag( pjmedia_endpt *endpt,
+                                            pjmedia_endpt_flag flag,
+                                            const void *value)
+{
+    PJ_ASSERT_RETURN(endpt, PJ_EINVAL);
+
+    switch (flag) {
+    case PJMEDIA_ENDPT_HAS_TELEPHONE_EVENT_FLAG:
+        endpt->has_telephone_event = *(pj_bool_t*)value;
+        break;
+    default:
+        return PJ_EINVAL;
+    }
+
+    return PJ_SUCCESS;
+}
+
+PJ_DEF(pj_status_t) pjmedia_endpt_get_flag( pjmedia_endpt *endpt,
+                                            pjmedia_endpt_flag flag,
+                                            void *value)
+{
+    PJ_ASSERT_RETURN(endpt, PJ_EINVAL);
+
+    switch (flag) {
+    case PJMEDIA_ENDPT_HAS_TELEPHONE_EVENT_FLAG:
+        *(pj_bool_t*)value = endpt->has_telephone_event;
+        break;
+    default:
+        return PJ_EINVAL;
+    }
+
+    return PJ_SUCCESS;
+}
 
 /**
@@ -490 +527 @@
 #if defined(PJMEDIA_RTP_PT_TELEPHONE_EVENTS) && \
     PJMEDIA_RTP_PT_TELEPHONE_EVENTS != 0
-
     /*
      * Add support telephony event
      */
-    m->desc.fmt[m->desc.fmt_count++] = 
-        pj_str(PJMEDIA_RTP_PT_TELEPHONE_EVENTS_STR);
-
-    /* Add rtpmap. */
-    attr = PJ_POOL_ZALLOC_T(pool, pjmedia_sdp_attr);
-    attr->name = pj_str("rtpmap");
-    attr->value = pj_str(PJMEDIA_RTP_PT_TELEPHONE_EVENTS_STR 
-                         " telephone-event/8000");
-    m->attr[m->attr_count++] = attr;
-
-    /* Add fmtp */
-    attr = PJ_POOL_ZALLOC_T(pool, pjmedia_sdp_attr);
-    attr->name = pj_str("fmtp");
-    attr->value = pj_str(PJMEDIA_RTP_PT_TELEPHONE_EVENTS_STR " 0-15");
-    m->attr[m->attr_count++] = attr;
+    if (endpt->has_telephone_event) {
+        m->desc.fmt[m->desc.fmt_count++] =
+            pj_str(PJMEDIA_RTP_PT_TELEPHONE_EVENTS_STR);
+
+        /* Add rtpmap. */
+        attr = PJ_POOL_ZALLOC_T(pool, pjmedia_sdp_attr);
+        attr->name = pj_str("rtpmap");
+        attr->value = pj_str(PJMEDIA_RTP_PT_TELEPHONE_EVENTS_STR
+                             " telephone-event/8000");
+        m->attr[m->attr_count++] = attr;
+
+        /* Add fmtp */
+        attr = PJ_POOL_ZALLOC_T(pool, pjmedia_sdp_attr);
+        attr->name = pj_str("fmtp");
+        attr->value = pj_str(PJMEDIA_RTP_PT_TELEPHONE_EVENTS_STR " 0-15");
+        m->attr[m->attr_count++] = attr;
+    }
 #endif