mozilla / webrtc-sdp

Rust SDP parser for WebRTC
Mozilla Public License 2.0
155 stars 30 forks source link

Consider accepting s=<newline> as a valid session name #204

Closed dminor closed 4 years ago

dminor commented 4 years ago

The webrtc.org sdp corpus has a number of files with s= (no space!) which suggests that their sdp parser will accept this as valid input. Example here: https://cs.chromium.org/chromium/src/third_party/webrtc/test/fuzzers/corpora/sdp-corpus/10.sdp. RFC 4566 specifically says this value must not be empty, but if chrome is accepting it, we might want to as well just to avoid compatibility problems.

dminor commented 4 years ago

Now fixed.