| commit | 1f976d6e543aca95f2fdfb64f970446e94ad5796 | |
| author | Max Kellermann <max@duempel.org> | |
| Tue, 20 Jul 2010 20:49:29 +0000 (20 22:49 +0200) | ||
| committer | Max Kellermann <max@duempel.org> | |
| Tue, 20 Jul 2010 20:55:13 +0000 (20 22:55 +0200) | ||
| tree | d83cf65a2eb1957b95609457ba554041dfbd0f6e | treesnapshot (tar.gz zip) |
| parent | a4908dca428dda7a3e83bfaf0d7e96d2b6242b73 | commitdiff |
input/curl: remove assertion after curl_multi_fdset()
Some users reported that MPD crashes when using a new CURL version
with the threaded DNS resolver enabled. It seems that
curl_multi_fdset() returns no file descriptor when the DNS resolver
runs in another thread, so MPD does not have any event to wait for.
On the CURL mailing list, somebody suggested to sleep for a fixed
amount of time. This is not an elegant solution, because daemons
should never have to sleep without waiting for an event. I hope the
CURL developers will review the API and remove the threaded DNS
resolver.
Meanwhile, I'm removing the assertion in question, to allow those
unfortunate users running the latest CURL version to continue using
MPD.
Some users reported that MPD crashes when using a new CURL version
with the threaded DNS resolver enabled. It seems that
curl_multi_fdset() returns no file descriptor when the DNS resolver
runs in another thread, so MPD does not have any event to wait for.
On the CURL mailing list, somebody suggested to sleep for a fixed
amount of time. This is not an elegant solution, because daemons
should never have to sleep without waiting for an event. I hope the
CURL developers will review the API and remove the threaded DNS
resolver.
Meanwhile, I'm removing the assertion in question, to allow those
unfortunate users running the latest CURL version to continue using
MPD.
| NEWS | diffblobblamehistory | |
| src/input/curl_input_plugin.c | diffblobblamehistory |