tree 7a80bd21c92a22f88d2b2456f8cd0a9ac92a8325
parent 5f5c94617e33382621d43c3ccf0af7049c80d9c6
author Christopher Faulet <cfaulet@haproxy.com> 1680184170 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1680677825 +0200

BUG/MEDIUM: dns: Properly handle error when a response consumed

When a response is consumed, result for co_getblk() is never checked. It
seems ok because amount of output data is always checked first. But There is
an issue when we try to get the first 2 bytes to read the message length. If
there is only one byte followed by a shutdown, the applet ignore the
shutdown and loop till the timeout to get more data.

So to avoid any issue and improve shutdown detection, the co_getblk() return
value is always tested. In addition, if there is not enough data, the applet
explicitly ask for more data by calling applet_need_more_data().

This patch relies on the previous one:

   * BUG/MEDIUM: channel: Improve reports for shut in co_getblk()

Both should be backported as far as 2.4. On 2.5 and 2.4,
applet_need_more_data() must be replaced by si_rx_endp_more().