From: Remi Tricot-Le Breton <rlebreton@haproxy.com>
Date: Fri, 28 Nov 2025 13:50:36 +0000 (+0100)
Subject: BUG/MINOR: jwt: Missing "case" in switch statement
X-Git-Tag: v3.4-dev1~71
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2b3d13a740d53dc52f8b06c6943d2b5a3cfa11ca;p=thirdparty%2Fhaproxy.git

BUG/MINOR: jwt: Missing "case" in switch statement

Because of missing "case" keyword in front of the values in a switch
case statement, the values were interpreted as goto tags and the switch
statement became useless.

This patch should fix GitHub issue #3200.
The fix should be backported up to 2.8.
---

diff --git a/src/sample.c b/src/sample.c
index 6c658176d..0473dc86c 100644
--- a/src/sample.c
+++ b/src/sample.c
@@ -4519,9 +4519,9 @@ static int sample_conv_jwt_verify_check(struct arg *args, struct sample_conv *co
 
 	if (args[1].type == ARGT_STR) {
 		switch (alg) {
-			JWS_ALG_HS256:
-			JWS_ALG_HS384:
-			JWS_ALG_HS512:
+			case JWS_ALG_HS256:
+			case JWS_ALG_HS384:
+			case JWS_ALG_HS512:
 			/* don't try to load a file with HMAC algorithms */
 				retval = 1;
 				break;