[Gnash-dev] Patch: fix gnash being unable to download anything with newer libcurl versions

[Hans de Goede]

Hi,

While trying gnash under Fedora 14 alpha I noticed that it didn't work
with youtube, even after clearing cookies.

So I did some debugging and found at that there is a bad interaction
between gnash and the latest libcurl, the attached
gnash-0.8.8-new-libcurl.patch fixes this.

The problem is that newer libcurl versions (atleast 7.21.x) return -1
for maxfd from curl_multi_fdset while they are doing dns resolving. This
gets combined with a short (7ms) return value from curl_multi_timeout
indicating that the thread / caller should just sleep a bit and then retry.

However we break out of the cache fill loop when maxfd < 1. We always use
a short timeout, so things work fine once we remove the maxfd < 0 check.

###

While looking into this I also tried using curl_multi_timeout to get
the select timeout as the curl documentation says is mandatory, rather
then hardcoding 10ms. See: gnash-0.8.8-curl-multi_timeout.patch, but
this does not work. After dns resolution curl_multi_timeout returns
a timeout of 30 seconds and despite maxfd being 10 now and thus the
fd_sets presumably being filled, the select now hangs until the entire
30 seconds are used up. I've tried changing the hardcoded timeout
to various values and it seems that for some reason the curl_multi_fdset
returned fd_sets seem to be no good, the select call always times out.

Regards,

Hans

gnash-0.8.8-new-libcurl.patch
Description: Text document

gnash-0.8.8-curl-multi_timeout.patch
Description: Text document