BUG/MINOR: resolvers: fix sent messages were counted twice
The sent messages counter was increased at both resolver.c and dns.c
layers.
This patch let the dns.c layer count the sent messages since this
layer handle a retry if transport layer is not ready (EAGAIN on udp
or tcp session ring buffer full).
This patch should be backported on all versions using a split of those
layers for resolving (v >=2.4)
(cherry picked from commit c37caab21c172ca042d5ef3aca6d1102f479dd53)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
diff --git a/src/resolvers.c b/src/resolvers.c
index db20a07..dac24f7 100644
--- a/src/resolvers.c
+++ b/src/resolvers.c
@@ -383,9 +383,7 @@
continue;
}
- if (dns_send_nameserver(ns, trash.area, len) < 0)
- ns->counters->snd_error++;
- else
+ if (dns_send_nameserver(ns, trash.area, len) >= 0)
resolution->nb_queries++;
}