summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko <vda.linux@googlemail.com>2009-12-02 22:46:25 (GMT)
committer Denys Vlasenko <vda.linux@googlemail.com>2009-12-02 22:46:25 (GMT)
commit7cb1d6ccf36eccb294bd26e1c6da6866e7fc0ab7 (patch)
tree6fb1df49aac9e14f24acc86c66a2d2b47f63a908
parent907647fb1540cb5303d90f2316dbfe89120db39c (diff)
downloadbusybox-7cb1d6ccf36eccb294bd26e1c6da6866e7fc0ab7.tar.gz
busybox-7cb1d6ccf36eccb294bd26e1c6da6866e7fc0ab7.tar.bz2
ntpd: try to avoid DNS requests *during* NTP roundtrip. +23 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--networking/ntpd.c14
1 files changed, 11 insertions, 3 deletions
diff --git a/networking/ntpd.c b/networking/ntpd.c
index 3bca69a..375f009 100644
--- a/networking/ntpd.c
+++ b/networking/ntpd.c
@@ -166,6 +166,15 @@ set_next(ntp_peer_t *p, unsigned t)
p->next_action_time = time(NULL) + t;
}
+static len_and_sockaddr*
+resolve_hostname(ntp_peer_t *p)
+{
+ p->lsa = host2sockaddr(p->hostname, 123);
+ if (p->lsa)
+ p->dotted = xmalloc_sockaddr2dotted_noport(&p->lsa->u.sa);
+ return p->lsa;
+}
+
static void
add_peers(char *s)
{
@@ -174,6 +183,7 @@ add_peers(char *s)
p = xzalloc(sizeof(*p));
p->hostname = s;
p->dotted = s;
+ resolve_hostname(p);
p->fd = -1;
p->msg.m_status = MODE_CLIENT | (NTP_VERSION << 3);
p->trustlevel = TRUSTLEVEL_PATHETIC;
@@ -300,12 +310,10 @@ send_query_to_peer(ntp_peer_t *p)
//TODO: big ntpd uses all IPs, not just 1st, do we need to mimic that?
//TODO: periodically re-resolve DNS names?
if (!p->lsa) {
- p->lsa = host2sockaddr(p->hostname, 123);
- if (!p->lsa) {
+ if (!resolve_hostname(p)) {
set_next(p, INTERVAL_QUERY_PATHETIC);
return -1;
}
- p->dotted = xmalloc_sockaddr2dotted_noport(&p->lsa->u.sa);
}
family = p->lsa->u.sa.sa_family;