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

[jboileau]

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.

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.

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.

One other thing that could possibly be done in live555.cpp is to ping
the IP before trying to connect. Unfortunatly, I have no clue how to
do this.

Jacques Boileau




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
>>
>