Bug 1042791: H.264 profile-level-id's in SDP have no leading '0x', breaking interop r=ehugg a=sylvestre
authorRandell Jesup <rjesup@jesup.org>
Wed, 23 Jul 2014 15:43:38 -0400
changeset 217304 9730317edd1a4947ee67f059815a2e2691a43ea3
parent 217303 e2dd9299b843e46fee276a186f3d9c076f039f8d
child 217305 debfa7a18dff47f37570de6cd239ccec7030ecaa
push id515
push userraliiev@mozilla.com
push dateMon, 06 Oct 2014 12:51:51 +0000
treeherdermozilla-release@267c7a481bef [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersehugg, sylvestre
bugs1042791
milestone33.0a2
Bug 1042791: H.264 profile-level-id's in SDP have no leading '0x', breaking interop r=ehugg a=sylvestre
media/webrtc/signaling/src/sipcc/core/gsm/gsm_sdp.c
--- a/media/webrtc/signaling/src/sipcc/core/gsm/gsm_sdp.c
+++ b/media/webrtc/signaling/src/sipcc/core/gsm/gsm_sdp.c
@@ -1178,17 +1178,17 @@ gsmsdp_set_video_media_attributes (uint3
                 return;
             }
             added_fmtp = 1;
             (void) sdp_attr_set_fmtp_payload_type(sdp_p, level, 0, a_inst,
                                                   payload_number);
             {
                 char buffer[32];
                 uint32_t profile_level_id = vcmGetVideoH264ProfileLevelID();
-                snprintf(buffer, sizeof(buffer), "0x%x", profile_level_id);
+                snprintf(buffer, sizeof(buffer), "%06x", profile_level_id);
                 (void) sdp_attr_set_fmtp_profile_level_id(sdp_p, level, 0, a_inst,
                                                           buffer);
             }
             if (media_type == RTP_H264_P1) {
                 (void) sdp_attr_set_fmtp_pack_mode(sdp_p, level, 0, a_inst,
                                                    1);
             }
             // TODO: other parameters we may want/need to set for H.264