Bug 1256750: Remove unnecessary sscanf_s parameter on windows, and fix format string everywhere else. r?jesup draft
authorByron Campen [:bwc] <docfaraday@gmail.com>
Tue, 15 Mar 2016 14:10:28 -0500
changeset 340635 0c4ff55103ea212d4e170b4d29694868b681d457
parent 339110 dd1abe874252e507b825a0a4e1063b0e13578288
child 516237 bdd5bd02424a95922ac70c213f507f7ebc57e61f
push id13022
push userbcampen@mozilla.com
push dateTue, 15 Mar 2016 19:23:37 +0000
reviewersjesup
bugs1256750
milestone48.0a1
Bug 1256750: Remove unnecessary sscanf_s parameter on windows, and fix format string everywhere else. r?jesup MozReview-Commit-ID: 3YFhTbGg5ec
media/webrtc/signaling/src/sdp/SipccSdpAttributeList.cpp
--- a/media/webrtc/signaling/src/sdp/SipccSdpAttributeList.cpp
+++ b/media/webrtc/signaling/src/sdp/SipccSdpAttributeList.cpp
@@ -679,24 +679,17 @@ SipccSdpAttributeList::LoadFmtp(sdp_t* s
 
         sstrncpy(h264Parameters->sprop_parameter_sets, fmtp->parameter_sets,
                  sizeof(h264Parameters->sprop_parameter_sets));
 
         h264Parameters->level_asymmetry_allowed =
             !!(fmtp->level_asymmetry_allowed);
 
         h264Parameters->packetization_mode = fmtp->packetization_mode;
-// Copied from VcmSIPCCBinding
-#ifdef _WIN32
-        sscanf_s(fmtp->profile_level_id, "%x",
-                 &h264Parameters->profile_level_id, sizeof(unsigned*));
-#else
-        sscanf(fmtp->profile_level_id, "%xu",
-               &h264Parameters->profile_level_id);
-#endif
+        sscanf(fmtp->profile_level_id, "%x", &h264Parameters->profile_level_id);
         h264Parameters->max_mbps = fmtp->max_mbps;
         h264Parameters->max_fs = fmtp->max_fs;
         h264Parameters->max_cpb = fmtp->max_cpb;
         h264Parameters->max_dpb = fmtp->max_dpb;
         h264Parameters->max_br = fmtp->max_br;
 
         parameters.reset(h264Parameters);
       } break;