| commit | 36eda7cf0e16656fb4fcb7dd5e93b5555b824e56 | [log] [tgz] |
|---|---|---|
| author | wu@webrtc.org <wu@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> | Tue Apr 15 20:37:30 2014 +0000 |
| committer | wu@webrtc.org <wu@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> | Tue Apr 15 20:37:30 2014 +0000 |
| tree | a6433c3cfeddf21d4c2816a18c0b600c7781c665 | |
| parent | ca539bbed0c7819d07bd6828dd1fbbf2e13cdd5c [diff] |
Workaround for https://bugzilla.mozilla.org/show_bug.cgi?id=996329, where the m line from firefox have a space at the end. For example: "m=application 38233 DTLS/SCTP 5000 " BUG=3212 TEST=manually try to use DataChannel between FF 28 and Chrome with rtccopy.com R=jiayl@webrtc.org Review URL: https://webrtc-codereview.appspot.com/12029005 git-svn-id: http://webrtc.googlecode.com/svn/trunk@5915 4adac7df-926f-26a2-2b94-8c16560cd09d
diff --git a/talk/app/webrtc/webrtcsdp.cc b/talk/app/webrtc/webrtcsdp.cc index 1b3e6f5..2a773b6 100644 --- a/talk/app/webrtc/webrtcsdp.cc +++ b/talk/app/webrtc/webrtcsdp.cc
@@ -2115,6 +2115,12 @@ // <fmt> std::vector<int> codec_preference; for (size_t j = 3 ; j < fields.size(); ++j) { + // TODO(wu): Remove when below bug is fixed. + // https://bugzilla.mozilla.org/show_bug.cgi?id=996329 + if (fields[j] == "" && j == fields.size() - 1) { + continue; + } + int pl = 0; if (!GetValueFromString(line, fields[j], &pl, error)) { return false;