tree cc50a234cc801168896ad93e2ea2f4bea0e8e024
parent 2fa8f07e5cf28f70fda6408f626b08feedb29bcc
author Emeric Brun <ebrun@haproxy.com> 1635518689 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1636552121 +0100

BUG/MINOR: resolvers: throw log message if trash not large enough for query

Before this patch the sent error counter was increased
for each targeted nameserver as soon as we were unable to build
the query message into the trash buffer. But this counter is here
to count sent errors at dns.c transport layer and this error is not
related to a nameserver.

This patch stops to increase those counters and sent a log message
to signal the trash buffer size is not large enough to build the query.

Note: This case should not happen except if trash size buffer was
customized to a very low value.

The function was also re-worked to return -1 in this error case
as it was specified in comment. This function is currently
called at multiple point in resolver.c but return code
is still not yet handled. So to advert the user of the malfunction
the log message was added.

This patch should be backported on all versions including the
layer split between dns.c and resolver.c (v >= 2.4)

(cherry picked from commit 0161d32df2b602cd1571539cb8890a68518832a3)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>