RepoMirrors/haproxy
1
0
Fork 0
mirror of http://git.haproxy.org/git/haproxy.git/ synced 2025-01-02 18:22:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

BUG/MINOR: server: do not enable DNS resolution on disabled proxies

Browse Source 
Leonhard Wimmer reported an interesting bug in github issue #1742.
Servers in disabled proxies that are configured for resolution are still
subscribed to DNS resolutions, but the LB algos are not initialized at
all since the proxy is disabled, so when the server state changes,
attempts to update its status cause a crash when the server's weight
is recalculated via a divide by the proxy's total weight which is zero.

This should be backported to all versions. Beware that before 2.5 or
so, there's no PR_FL_DISABLED flag, instead px->disabled should be
used (2.3-2.4) or PR_STSTOPPED for older versions.

Thanks to Leonhard for his report and quick test!
This commit is contained in:
Willy Tarreau 2022-06-10 11:11:44 +02:00
parent 5a0e7ca5d0
commit 9b46fb4cca
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/resolvers.c
View File

@ -2528,6 +2528,13 @@ static int resolvers_finalize_config(void)
for (px = proxies_list; px; px = px->next) {
struct server *srv;
if (px->flags & PR_FL_DISABLED) {
/* must not run and will not work anyway since
* nothing in the proxy is initialized.
*/
continue;
}
for (srv = px->srv; srv; srv = srv->next) {
struct resolvers *resolvers;