[vlc-devel] live555 connection timeout (patch included)

Derk-Jan Hartman hartman at videolan.org
Tue Oct 7 23:42:34 CEST 2008


On 7 okt 2008, at 22:52, jboileau wrote:
> BTW although I am not very knowledgeable RTSP wise, There are 2 things
> (well maybe 3) that might be done to minimize this RTSP problem in
> live555.cpp. Since I have very little knowledge of this kind of code,
> I have no idea what side effect these might have.
>
> 1. change i_lefttries from 3 to 1. As Sebastien remarqued, I don't
> know why we want to retry 3 times and take 3 times as much time to
> return. The user can retry manually if he wants to. I am sure he would
> rather retry himself than wait longer.

This is for an initial authentication failure basically, and for  
rollover to TCP i think.
>
> 2. In the Connect member, if p_sys->rtsp->sendOptionsCmd fails
> p_sys->rtsp->describeURL is called anyway. both of these call
> openConnectionFromURL. It would seem that if
> p_sys->rtsp->sendOptionsCmd fails, p_sys->rtsp->describeURL could be
> skipped. This saves waiting for another openConnectionFromURL to fail.

Possibly, but i'm a tad weary on this.
>
> As I said I don't know if these changes have bad effects in certain
> situations, but when trying to connect to an RTSP client that isn't
> there, it reduces the timeout to around 20 seconds. This without any
> changes to the live555 contrib.

The live555 code should be fixed, or better, we should dump live555  
and implement RTSP in VLC. (so we also get ipv6 support).
We can discuss this to the moon, but ultimately, those are the best  
two options. There is one question to ask, and that is who's is gonna  
do the coding work.

>
> On Tue, Oct 7, 2008 at 4:22 PM, jboileau <jboileau at gmail.com> wrote:
>> I tried many approaches but none of them seem to work. I would think,
>> because of the unresolved messages I get, that I need an older  
>> version
>> of live555 than the current to go with VLC.
>>
>> I tried replacing usr\win32\live.com dir with the new and patched
>> version: I get unresolved messages.
>>
>> I tried your suggestion Sebastien, and made a seperate dir for the  
>> new
>> patched live555 and changed --with-live555-tree to point to this new
>> loccation, same thing.
>>
>> Anybody can point me to the right live555 sources I can use?
>>
>> Here are the messages I get:
>>
>> Warning: resolving _select by linking to _select at 20
>> Use --enable-stdcall-fixup to disable these warnings
>> Use --disable-stdcall-fixup to disable these fixups
>> Warning: resolving _getsockname by linking to _getsockname at 12
>> Warning: resolving _gethostname by linking to _gethostname at 8
>> /live555/liveMedia/libliveMedia.a:ByteStreamFileSource.cpp:(.text 
>> +0xedb): undefi
>> ned reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:MP3HTTPSource.cpp:(.text+0xd0d0):  
>> undefined re
>> ference to `_connect'
>> /live555/liveMedia/libliveMedia.a:MP3StreamState.cpp:(.text 
>> +0x10419): undefined
>> reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:MP3StreamState.cpp:(.text 
>> +0x10505): undefined
>> reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:MP3StreamState.cpp:(.text 
>> +0x105f1): undefined
>> reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:MP3StreamState.cpp:(.text 
>> +0x10b89): undefined
>> reference to `_send'
>> /live555/liveMedia/libliveMedia.a:MP3StreamState.cpp:(.text 
>> +0x10fdf): undefined
>> reference to `_recv'
>> /live555/liveMedia/libliveMedia.a:H263plusVideoStreamParser.cpp: 
>> (.text+0x17e7d):
>> undefined reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:InputFile.cpp:(.text+0x198d2):  
>> undefined refer
>> ence to `___getreent'
>> /live555/liveMedia/libliveMedia.a:InputFile.cpp:(.text+0x1992f):  
>> undefined refer
>> ence to `___getreent'
>> /live555/liveMedia/libliveMedia.a:InputFile.cpp:(.text+0x19966):  
>> undefined refer
>> ence to `___getreent'
>> /live555/liveMedia/libliveMedia.a:InputFile.cpp:(.text+0x19a5a):  
>> undefined refer
>> ence to `_fseeko'
>> /live555/liveMedia/libliveMedia.a:InputFile.cpp:(.text+0x19a87):  
>> undefined refer
>> ence to `_ftello'
>> /live555/liveMedia/libliveMedia.a:HTTPSink.cpp:(.text+0x1b5df):  
>> undefined refere
>> nce to `_listen'
>> /live555/liveMedia/libliveMedia.a:HTTPSink.cpp:(.text+0x1b969):  
>> undefined refere
>> nce to `_accept'
>> /live555/liveMedia/libliveMedia.a:HTTPSink.cpp:(.text+0x1b9fa):  
>> undefined refere
>> nce to `_send'
>> /live555/liveMedia/libliveMedia.a:HTTPSink.cpp:(.text+0x1bae9):  
>> undefined refere
>> nce to `_send'
>> /live555/liveMedia/libliveMedia.a:OutputFile.cpp:(.text+0x1f9e2):  
>> undefined refe
>> rence to `___getreent'
>> /live555/liveMedia/libliveMedia.a:OutputFile.cpp:(.text+0x1fa05):  
>> undefined refe
>> rence to `___getreent'
>> /live555/liveMedia/libliveMedia.a:OutputFile.cpp:(.text+0x1fa61):  
>> undefined refe
>> rence to `___getreent'
>> /live555/liveMedia/libliveMedia.a:OutputFile.cpp:(.text+0x1fa6b):  
>> undefined refe
>> rence to `___getreent'
>> /live555/liveMedia/libliveMedia.a:RTPInterface.cpp:(.text+0x2a0a7):  
>> undefined re
>> ference to `_send'
>> /live555/liveMedia/libliveMedia.a:RTPInterface.cpp:(.text+0x2a0cb):  
>> undefined re
>> ference to `_send'
>> /live555/liveMedia/libliveMedia.a:RTPInterface.cpp:(.text+0x2a0fb):  
>> undefined re
>> ference to `_send'
>> /live555/liveMedia/libliveMedia.a:RTPInterface.cpp:(.text+0x2a11e):  
>> undefined re
>> ference to `_send'
>> /live555/liveMedia/libliveMedia.a:RTSPServer.cpp:(.text+0x2cf21):  
>> undefined refe
>> rence to `_listen'
>> /live555/liveMedia/libliveMedia.a:RTSPServer.cpp:(.text+0x2d56d):  
>> undefined refe
>> rence to `_accept'
>> /live555/liveMedia/libliveMedia.a:RTSPServer.cpp:(.text+0x2e0f4):  
>> undefined refe
>> rence to `_send'
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x30562): undefi
>> ned reference to `_listen'
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x30976): undefi
>> ned reference to `_accept'
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x309f1): undefi
>> ned reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x30c6b): undefi
>> ned reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x30ccb): undefi
>> ned reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x30db4): undefi
>> ned reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x30dd6): undefi
>> ned reference to `___getreent'
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x30e16): more u
>> ndefined references to `___getreent' follow
>> /live555/liveMedia/libliveMedia.a:RTSPOverHTTPServer.cpp:(.text 
>> +0x30e69): undefi
>> ned reference to `_send'
>> /live555/liveMedia/libliveMedia.a:RTSPClient.cpp:(.text+0x35566):  
>> undefined refe
>> rence to `_connect'
>> /live555/liveMedia/libliveMedia.a:RTSPClient.cpp:(.text+0x35573):  
>> undefined refe
>> rence to `___errno'
>> /live555/liveMedia/libliveMedia.a:RTSPClient.cpp:(.text+0x35f92):  
>> undefined refe
>> rence to `_send'
>> /live555/liveMedia/libliveMedia.a:RTSPClient.cpp:(.text+0x36cc8):  
>> undefined refe
>> rence to `_connect'
>> /live555/liveMedia/libliveMedia.a:RTSPClient.cpp:(.text+0x36fcf):  
>> undefined refe
>> rence to `_send'
>> /live555/liveMedia/libliveMedia.a:SIPClient.cpp:(.text+0x38beb):  
>> undefined refer
>> ence to `___getreent'
>>
>>
>> Jacques Boileau
>>
>>
>>
>>
>> On Tue, Oct 7, 2008 at 2:54 AM, Sébastien Escudier
>> <sebastien-devel at celeos.eu> wrote:
>>> Quoting jboileau <jboileau at gmail.com>:
>>>> configure-vlc.sh and built VLC. But my changes to live555 are not  
>>>> in
>>>> the new VLC I create. What step am I missing? Are there  
>>>> instructions
>>>> on adding a build of live555 to VLC somewhere?
>>>
>>> did you specify the path of your modified live555 library ?
>>> like : '--with-live555-tree=your_path' when you run configure
>>> But without this vlc souldnt have compiled. Did you apply the  
>>> patch to
>>> live555.cpp in vlc ?
>>> _______________________________________________
>>> vlc-devel mailing list
>>> To unsubscribe or modify your subscription options:
>>> http://mailman.videolan.org/listinfo/vlc-devel
>>>
>>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>




More information about the vlc-devel mailing list