BUG/MINOR: proxy: Use .disabled field as a bitfield as documented
authorChristopher Faulet <cfaulet@haproxy.com>
Thu, 30 Sep 2021 16:11:54 +0000 (18:11 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Fri, 15 Oct 2021 12:12:19 +0000 (14:12 +0200)
.disabled field in the proxy structure is documented to be a bitfield. So
use it as a bitfield. This change was introduced to the 2.5, by commit
8e765b86f ("MINOR: proxy: disabled takes a stopping and a disabled state").

No backport is needed except if the above commit is backported.

src/cfgparse-listen.c
src/cfgparse.c
src/mworker.c
src/proxy.c

index 98e31ec..6f526d4 100644 (file)
@@ -595,7 +595,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm)
        else if (strcmp(args[0], "disabled") == 0) {  /* disables this proxy */
                if (alertif_too_many_args(0, file, linenum, args, &err_code))
                        goto out;
-               curproxy->disabled = PR_DISABLED;
+               curproxy->disabled |= PR_DISABLED;
        }
        else if (strcmp(args[0], "enabled") == 0) {  /* enables this proxy (used to revert a disabled default) */
                if (alertif_too_many_args(0, file, linenum, args, &err_code))
index 173dcdb..e606a47 100644 (file)
@@ -1019,7 +1019,7 @@ int cfg_parse_peers(const char *file, int linenum, char **args, int kwm)
                stktables_list = t;
        }
        else if (strcmp(args[0], "disabled") == 0) {  /* disables this peers section */
-               curpeers->disabled = PR_DISABLED;
+               curpeers->disabled |= PR_DISABLED;
        }
        else if (strcmp(args[0], "enabled") == 0) {  /* enables this peers section (used to revert a disabled default) */
                curpeers->disabled = 0;
index ca1cc74..7c73780 100644 (file)
@@ -443,7 +443,7 @@ void mworker_cleanlisteners()
                }
                /* if the proxy shouldn't be in the master, we stop it */
                if (!listen_in_master)
-                       curproxy->disabled = PR_DISABLED;
+                       curproxy->disabled |= PR_DISABLED;
        }
 }
 
index 6868eff..4dbf671 100644 (file)
@@ -1823,7 +1823,7 @@ void proxy_cond_disable(struct proxy *p)
        if (p->li_ready + p->li_paused > 0)
                return;
 
-       p->disabled = PR_STOPPED;
+       p->disabled |= PR_STOPPED;
 
        /* Note: syslog proxies use their own loggers so while it's somewhat OK
         * to report them being stopped as a warning, we must not spam their log