Commit 7e61aeb9 authored by Eric S. Raymond's avatar Eric S. Raymond
Browse files

Address GitLab issue Issue #13: DNS lookup broken in obscure cases

This is "A hack good-enough solution would be to make the array bigger."
Before this change the default array sizes were absurdly small, as though
RAM still cost the way it did in 1990 or whenever.

Also added: a note in devel/TODO that this whole mess ought to be replaced
by something like c_ares.
parent 50b0284a
......@@ -91,6 +91,11 @@ Neither is ideal, easy pickings for someone to code on.
(and multiple inbound NTP connections, and a hole in your firewall) even when
it has a known-good local timesource like a GPS. This should be fixed.
* The code used for asynchronous DNS lookup is more general, moere complex,
and more bug-prone than it should be. Either our version should be
rewritten and simplified or (better idea!) it should be replaced by
a well-maintained third-party library like c_ares.
* Hal says "We need a way to inspect/debug bits on the wire. `ntpdate
-du' is the traditional approach." Hal's straw man is a new program
rather than trying to make ntpdig do this. Eric's proposed answer:
......
......@@ -22,8 +22,8 @@
#define CHILD_EXIT_REQ ((blocking_pipe_header *)(intptr_t)-1)
#define CHILD_GONE_RESP CHILD_EXIT_REQ
#define WORKITEMS_ALLOC_INC 16
#define RESPONSES_ALLOC_INC 4
#define WORKITEMS_ALLOC_INC 64
#define RESPONSES_ALLOC_INC 16
#ifdef OVERRIDE_THREAD_MINSTACKSIZE
#define THREAD_MINSTACKSIZE OVERRIDE_THREAD_MINSTACKSIZE
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment