]> git.ipfire.org Git - thirdparty/haproxy.git/commit
Revert "BUG/MINOR: dns: SRV records ignores duplicated AR records"
authorChristopher Faulet <cfaulet@haproxy.com>
Tue, 12 Jan 2021 09:27:26 +0000 (10:27 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Tue, 12 Jan 2021 15:37:54 +0000 (16:37 +0100)
commita1eea3bbb1973f3becc4603c3bbad4e96af3361f
treea2ca6c9b0fab9c59f72dbf24251cf038a97f7df7
parent76603f2552fafc23871a914600a18c3616959693
Revert "BUG/MINOR: dns: SRV records ignores duplicated AR records"

This reverts commit 949a7f64591458eb06c998acf409093ea991dc3a.

The first part of the patch introduces a bug. When a dns answer item is
allocated, its <ar_item> is only initialized at the end of the parsing, when
the item is added in the answer list. Thus, we must not try to release it
during the parsing.

The second part is also probably buggy. It fixes the issue #971 but reverts
a fix for the issue #841 (see commit fb0884c8297 "BUG/MEDIUM: dns: Don't
store additional records in a linked-list"). So it must be at least
revalidated.

This revert fixes a segfault reported in a comment of the issue #971. It
must be backported as far as 2.2.
src/dns.c