From: Christopher Faulet <cfaulet@haproxy.com>
Date: Wed, 27 Jan 2021 12:32:25 +0000 (+0100)
Subject: BUG/MINOR: stats: Add a break after filling ST_F_MODE field for servers
X-Git-Tag: v2.4-dev7~124
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3888b8cd7bd3d90ad60b413b0a1983ab4f56abdc;p=thirdparty%2Fhaproxy.git

BUG/MINOR: stats: Add a break after filling ST_F_MODE field for servers

The previous patch was pushed too quickly (399bf72f6 "BUG/MINOR: stats:
Remove a break preventing ST_F_QCUR to be set for servers"). It was not an
extra break but a misplaced break statement. Thus, now a break statement
must be added after filling the ST_F_MODE field in stats_fill_sv_stats().

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

diff --git a/src/stats.c b/src/stats.c
index 1d5aa841f6..be8bc64131 100644
--- a/src/stats.c
+++ b/src/stats.c
@@ -2069,6 +2069,7 @@ int stats_fill_sv_stats(struct proxy *px, struct server *sv, int flags,
 				break;
 			case ST_F_MODE:
 				metric = mkf_str(FO_CONFIG|FS_SERVICE, proxy_mode_str(px->mode));
+				break;
 			case ST_F_QCUR:
 				metric = mkf_u32(0, sv->nbpend);
 				break;