the enclosed patch makes sems to accept RTP/SAVPF media transport when
it validates sdp.  if i don't hear objections, i'll commit it during the
next couple of  days.

-- juha

diff --git a/core/AmSdp.cpp b/core/AmSdp.cpp
index a3311d0..1097c9a 100644
--- a/core/AmSdp.cpp
+++ b/core/AmSdp.cpp
@@ -97,6 +97,7 @@ inline string transport_p_2_str(int tp)
   case TP_RTPAVP: return "RTP/AVP";
   case TP_UDP: return "udp";
   case TP_RTPSAVP: return "RTP/SAVP";
+  case TP_RTPSAVPF: return "RTP/SAVPF";
   case TP_UDPTL: return "udptl";
   default: return "<unknown media type>";
   }
@@ -352,7 +353,7 @@ void AmSdp::print(string& body) const
 
       string options;
 
-      if (media_it->transport == TP_RTPAVP || media_it->transport == 
TP_RTPSAVP) {
+      if (media_it->transport == TP_RTPAVP || media_it->transport == 
TP_RTPSAVP || media_it->transport == TP_RTPSAVPF) {
        for(std::vector<SdpPayload>::const_iterator pl_it = 
media_it->payloads.begin();
            pl_it != media_it->payloads.end(); pl_it++) {
 
@@ -875,7 +876,7 @@ static void parse_sdp_media(AmSdp* sdp_msg, char* s)
       }
     case FMT:
       {
-       if (m.transport == TP_RTPAVP || m.transport == TP_RTPSAVP) {
+       if (m.transport == TP_RTPAVP || m.transport == TP_RTPSAVP || 
m.transport == TP_RTPSAVPF) {
          if (contains(media_line, line_end, ' ')) {
            next = parse_until(media_line, ' ');
            string value;
@@ -1389,6 +1390,8 @@ static TransProt transport_type(string transport)
     return TP_UDP;
   else if(transport_uc == "RTP/SAVP")
     return TP_RTPSAVP;
+  else if(transport_uc == "RTP/SAVPF")
+    return TP_RTPSAVPF;
   else if(transport_uc == "UDPTL")
     return TP_UDPTL;
   else 
diff --git a/core/AmSdp.h b/core/AmSdp.h
index fdb5681..1e20e12 100644
--- a/core/AmSdp.h
+++ b/core/AmSdp.h
@@ -55,7 +55,7 @@ enum AddressType { AT_NONE=0, AT_V4, AT_V6 };
 /** media type */
 enum MediaType { MT_NONE=0, MT_AUDIO, MT_VIDEO, MT_APPLICATION, MT_TEXT, 
MT_MESSAGE, MT_IMAGE };
 /** transport protocol */
-enum TransProt { TP_NONE=0, TP_RTPAVP, TP_UDP, TP_RTPSAVP, TP_UDPTL };
+enum TransProt { TP_NONE=0, TP_RTPAVP, TP_UDP, TP_RTPSAVP, TP_UDPTL, 
TP_RTPSAVPF };
 
 /** \brief c=... line in SDP*/
 struct SdpConnection
_______________________________________________
Semsdev mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/semsdev

Reply via email to