Re: [Live-devel] [PATCH] overflow in getResponse1()

2010-02-09 Thread Ross Finlayson
Thanks. I have just released a new version (2010.02.10) of the software, that includes this fix. -- Ross Finlayson Live Networks, Inc. http://www.live555.com/ ___ live-devel mailing list live-devel@lists.live555.com http://lists.live555.com/mailman/li

Re: [Live-devel] how to create custom sink

2010-02-09 Thread Ross Finlayson
The question now is: how to createRTPSink ? The "H264VideoRTPSink" class seems to be an abstruct and doesn't have implemetation for createNew Yes it does - just call "H264VideoRTPSink::createNew()" -- Ross Finlayson Live Networks, Inc. http://www.live555.com/ _

Re: [Live-devel] H.264 RTP Packetizer

2010-02-09 Thread Ross Finlayson
anyone have a h.264 rtp packetizer? Like the packetizer that vlc use to join h.264 NAL units.. Of course we have a H.264 RTP packetizer: "H264VideoRTPSink". We also have a H.264 RTP depacketizer: "H264VideoRTPSource". (The latter is used in VLC.) -- Ross Finlayson Live Networks, Inc. http:

Re: [Live-devel] how to create custom sink

2010-02-09 Thread Victor lyamtsev
Any idea if i can find any specific example for H.264 ? As i couldn't find base class for H264MediaSubsession, I am trying to create H264 subsession with PassiveServerMediaSubsession::createNew(RTPSink& rtpSink, RTCPInstance* rtcpInstance) The question now

[Live-devel] [PATCH] overflow in getResponse1()

2010-02-09 Thread Martin Bonnin
Hello all, getResponse1() can overflow by one byte when called from setupMediaSubsession(). The reason is that the 'responseBufferSize' does not include the ending '\0' (see responseBuffer[bytesRead] = '\0' in RTSPClient.cpp around line 2195) The attached patch allocates one byte more, l

[Live-devel] H.264 RTP Packetizer

2010-02-09 Thread Nuno Cardoso
Hi all, anyone have a h.264 rtp packetizer? Like the packetizer that vlc use to join h.264 NAL units.. Thanks, Nuno Cardoso. ___ live-devel mailing list live-devel@lists.live555.com http://lists.live555.com/mailman/listinfo/live-devel

Re: [Live-devel] [PATCH] Fix parseCLine()

2010-02-09 Thread Aurélien Nephtali
On Tue, Feb 9, 2010 at 3:57 PM, Ross Finlayson wrote: >> The attached patch fixes 'c=' lines looking like this : >> >> c=IN IP4 224.0.0.0 > > That's actually not legal SDP (because 224.0.0.0 is a multicast address, and > so the line should have a trailing "/").  However, in the spirit of "be > lib

Re: [Live-devel] [PATCH] Fix parseCLine()

2010-02-09 Thread Ross Finlayson
The attached patch fixes 'c=' lines looking like this : c=IN IP4 224.0.0.0 That's actually not legal SDP (because 224.0.0.0 is a multicast address, and so the line should have a trailing "/"). However, in the spirit of "be liberal in what you accept", I'll make this change in the next relea

[Live-devel] [PATCH] Fix parseCLine()

2010-02-09 Thread Aurélien Nephtali
Hello, The attached patch fixes 'c=' lines looking like this : c=IN IP4 224.0.0.0 Regards, -- Aurélien Nephtali --- a/liveMedia/MediaSession.cpp 2010-01-25 15:55:06.0 + +++ b/liveMedia/MediaSession.cpp 2010-01-25 15:57:31.0 + @@ -287,7 +287,7 @@ Boolean MediaSession::pa