[streaming] RTP payloadtype bug in miniSAPserver
Derk-Jan Hartman
hartman at videolan.org
Wed Jul 21 01:04:40 CEST 2004
Since some time now I have been aware of a HUGE bug in miniSAPserver.
It creates announces with m= video port udp 14
This essentially means it is announcing video streams which are RTP
packetized like MP3 streams.... This bug has remained unnoticed to us,
because essentially the SAP parser in VLC media player is very bad.
This parser assumes that any single stream (only one m= line) with UDP
in it must be a MPEG2 TS stream no matter what the RTP payload format
is set to.
In order to correctly announce VLS streams, the attached patch is
required. Unfortunately miniSAPserver is in quite a bad shape ATM, and
many in the VL team have come to recognize that it is essentially a
very bad piece of software. Work may therefore not continue on it,
since it would require quite a major overhaul.
However since this has been annoying me so much, and since I hate
non-compliant stuff, even more so when we ourselves are the guilty
party, I have done some work on it. One thing led to another and there
you go some other changes as well :)
* Upped the version number to 0.2.4
* Fixed a missing newline in the -h help
* Bugfix in RTP payloadformat announce (now uses 33 instead of 14)
* Added a config option to set the SAP delay
* Added a=tool parameter so miniSAPserver can be more easily recognized
(good for bugfixing :)
* Added a=mux:m2t and a=packetformat:RAW parameters (not required, but
some players like these. VBrick player comes to mind)
* Changed a=plgroup to a=x-plgroup because it is a non standardized
parameter (new style will be supported by VLC 0.7.3 and up)
* Updated the manpage
* Clarified that miniSAPserver is only supposed to work for MPEG TS
streams.
ATM, it is not possible to edit using Version Control for
miniSAPserver, because it isn't in Subversion yet, and CVS cannot be
edited anymore (as far as I know).
So the patch is attached and I suggest we make a new release with these
changes ASAP, regardless of wether it is in version control yet.
DJ
---
Universiteit Twente
Derk-Jan Hartman (d.hartman at student.utwente dot nl)
http://home.student.utwente.nl/d.hartman
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mSAPs.patch
Type: application/octet-stream
Size: 8178 bytes
Desc: not available
Url : http://mailman.videolan.org/pipermail/streaming/attachments/20040721/f7a4550b/attachment.obj
More information about the streaming
mailing list