RepoMirrors/haproxy
1
0
Fork 0
mirror of http://git.haproxy.org/git/haproxy.git/ synced 2025-02-28 00:20:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

BUG/MINOR: filters: Skip disabled proxies during startup only

Browse Source 
This partially reverts the patch 400829cd2 ("BUG/MEDIUM: filters: Don't try to
init filters for disabled proxies"). Disabled proxies must not be skipped in
flt_deinit() and flt_deinit_all_per_thread() when HAProxy is stopped because,
obvioulsy, at this step, all proxies appear as disabled (or stopped, it is the
same state). It is safe to do so because, during startup, filters declared on
disabled proxies are removed. Thus they don't exist anymore during shutdown.

This patch must be backported in all versions where the patch above is.
This commit is contained in:
Christopher Faulet 2020-11-03 16:40:37 +01:00
parent 9dd7f4fb4b
commit 743bd6adc8
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/filters.c
View File

@ -356,7 +356,7 @@ flt_deinit(struct proxy *proxy)
struct flt_conf *fconf, *back;
list_for_each_entry_safe(fconf, back, &proxy->filter_configs, list) {
if (!proxy->disabled && fconf->ops->deinit)
if (fconf->ops->deinit)
fconf->ops->deinit(proxy, fconf);
LIST_DEL(&fconf->list);
free(fconf);
@ -385,10 +385,8 @@ flt_deinit_all_per_thread()
{
struct proxy *px;
for (px = proxies_list; px; px = px->next) {
if (!px->disabled)
flt_deinit_per_thread(px);
}
for (px = proxies_list; px; px = px->next)
flt_deinit_per_thread(px);
}
/* Attaches a filter to a stream. Returns -1 if an error occurs, 0 otherwise. */