Closed
Bug 818640
Opened 12 years ago
Closed 10 years ago
WebRTC should accept dynamic payload values < 96
Categories
(Core :: WebRTC: Signaling, defect, P3)
Core
WebRTC: Signaling
Tracking
()
RESOLVED
FIXED
mozilla41
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: abr, Assigned: bwc)
References
Details
(Whiteboard: [WebRTC] [blocking-webrtc-])
Attachments
(1 file, 1 obsolete file)
Presently, the WebRTC code will reject any offer that attempts to use a dynamic payload type less than 96. While values in the range of 96 through 127 are preferred for dynamic use, the AVT/RTP profile (and those derived from it) do allow such types to be sent in SDP. By rejecting them, we are out of compliance with the specification.
RFC 3551, section 3:
> "This profile reserves payload type numbers in the range 96-127
> exclusively for dynamic assignment. Applications SHOULD first use
> values in this range for dynamic payload types. Those applications
> which need to define more than 32 dynamic payload types MAY bind
> codes below 96, in which case it is RECOMMENDED that unassigned
> payload type numbers be used first. However, the statically assigned
> payload types are default bindings and MAY be dynamically bound to
> new encodings if needed."
(With particular emphasis on the final quoted sentence)
Updated•12 years ago
|
Priority: -- → P3
Whiteboard: [WebRTC] [blocking-webrtc-]
Reporter | ||
Updated•12 years ago
|
Assignee: nobody → adam
Assignee | ||
Comment 1•10 years ago
|
||
I think this should work, but I can check.
Assignee: adam → docfaraday
Assignee | ||
Comment 2•10 years ago
|
||
/r/8779 - Bug 818640: Test that using dynamic payload types < 96 works.
Pull down this commit:
hg pull -r 5a6b89a790658884e7d0e262180660690a50a701 https://reviewboard-hg.mozilla.org/gecko/
Assignee | ||
Updated•10 years ago
|
Attachment #8606023 -
Flags: review?(martin.thomson)
Updated•10 years ago
|
Attachment #8606023 -
Flags: review?(martin.thomson) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8606023 [details]
MozReview Request: bz://818640/bwc
/r/8779 - Bug 818640: Test that using dynamic payload types < 96 works. r=mt
Pull down this commit:
hg pull -r b0445decb23daca43b0be17eac1096abf94d83a1 https://reviewboard-hg.mozilla.org/gecko/
Attachment #8606023 -
Flags: review+
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8606023 [details]
MozReview Request: bz://818640/bwc
Carry forward r=mt
Attachment #8606023 -
Flags: review+
Assignee | ||
Comment 5•10 years ago
|
||
Check back on retriggers for infra failures.
Flags: needinfo?(docfaraday)
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(docfaraday)
Keywords: checkin-needed
Keywords: checkin-needed
Comment 7•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
Assignee | ||
Comment 8•9 years ago
|
||
Attachment #8606023 -
Attachment is obsolete: true
Attachment #8618003 -
Flags: review+
Assignee | ||
Comment 9•9 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•