Closed Bug 1432936 Opened 2 years ago Closed Last year

Rust SDP Parser fails to produce an error on NewSdpTest.ParseInvalidSimulcastNotReceiving

Categories

(Core :: WebRTC: Signaling, defect, P3)

58 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: dminor, Assigned: johannes.willbold)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

[ RUN      ] RoundTripSerialize/NewSdpTest.ParseInvalidSimulcastNotReceiving/0
version: 0
origin: -, 4294967296, 2, 127.0.0.1
session: SIP Call
connection: 198.51.100.7
bandwidth: CT, 5000
timing: 0, 0
media: Video, 56436, Rtp/Savpf, [120]
/home/dminor/src/firefox/media/webrtc/signaling/gtest/sdp_unittests.cpp:3798: Failure
Expected: ("") != (GetParseErrors()), actual: "" vs ""
[  FAILED  ] RoundTripSerialize/NewSdpTest.ParseInvalidSimulcastNotReceiving/0, where GetParam() = (false, false) (0 ms)
Assignee: nobody → johannes.willbold
Comment on attachment 8983175 [details]
Bug 1432936: Added sanity check for sendonly attribute.

https://reviewboard.mozilla.org/r/249028/#review255534

::: media/webrtc/signaling/src/sdp/rsdparsa/src/lib.rs:613
(Diff revision 1)
> +    for msection in &session.media {
> +        if msection.get_attribute(SdpAttributeType::Sendonly).is_some() {
> +            if let Some(&SdpAttribute::Simulcast(ref x)) = msection.get_attribute(SdpAttributeType::Simulcast) {
> +                if x.receive.len() > 0 {
> +                    return Err(SdpParserError::Sequence {
> +                        message: "Simulcast can't define send parameters for recvonly".to_string(),

Isn't this the other way around?
Attachment #8983175 - Flags: review?(docfaraday)
Comment on attachment 8983175 [details]
Bug 1432936: Added sanity check for sendonly attribute.

https://reviewboard.mozilla.org/r/249028/#review255534

> Isn't this the other way around?

Yes absolutely
Comment on attachment 8983175 [details]
Bug 1432936: Added sanity check for sendonly attribute.

https://reviewboard.mozilla.org/r/249028/#review255576
Attachment #8983175 - Flags: review?(docfaraday) → review+
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 6ee80e70c3e754cbd834923504861e650e680a62 -d c9ae7e47a6ef: rebasing 467229:6ee80e70c3e7 "Bug 1432936: Added sanity check for sendonly attribute. r=bwc" (tip)
merging media/webrtc/signaling/gtest/sdp_unittests.cpp
merging media/webrtc/signaling/src/sdp/rsdparsa/src/attribute_type.rs
merging media/webrtc/signaling/src/sdp/rsdparsa/src/media_type.rs
merging media/webrtc/signaling/src/sdp/rsdparsa_capi/src/attribute.rs
warning: conflicts while merging media/webrtc/signaling/src/sdp/rsdparsa_capi/src/attribute.rs! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 6ee80e70c3e754cbd834923504861e650e680a62 -d 033f5f2b2582: rebasing 467268:6ee80e70c3e7 "Bug 1432936: Added sanity check for sendonly attribute. r=bwc" (tip)
merging media/webrtc/signaling/gtest/sdp_unittests.cpp
merging media/webrtc/signaling/src/sdp/rsdparsa/src/attribute_type.rs
merging media/webrtc/signaling/src/sdp/rsdparsa/src/media_type.rs
merging media/webrtc/signaling/src/sdp/rsdparsa_capi/src/attribute.rs
warning: conflicts while merging media/webrtc/signaling/src/sdp/rsdparsa_capi/src/attribute.rs! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Pushed by drno@ohlmeier.org:
https://hg.mozilla.org/integration/autoland/rev/e383e893aa70
Added sanity check for sendonly attribute. r=bwc
https://hg.mozilla.org/mozilla-central/rev/e383e893aa70
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.