BUG/MINOR: jwt: fix variable initialisation
authorWilliam Lallemand <wlallemand@haproxy.com>
Mon, 8 Jul 2024 12:23:14 +0000 (14:23 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 11 Jul 2024 13:44:58 +0000 (15:44 +0200)
Set the alg variable from sample_conv_jwt_verify_check() to
JWT_ALG_DEFAULT.

This was reported by coverity in #2630, but since you need to use the
first argument to use the 2nd, this has no real impact.

Mut be backported with 883f1bd (as far as 2.6).

(cherry picked from commit 0a1b251c1a2ac55e135db0f5ac3d241e218308b4)
Signed-off-by: Willy Tarreau <w@1wt.eu>

src/sample.c

index 1756c0e..61d51fa 100644 (file)
@@ -4262,7 +4262,8 @@ static int sample_conv_json_query(const struct arg *args, struct sample *smp, vo
 static int sample_conv_jwt_verify_check(struct arg *args, struct sample_conv *conv,
                                        const char *file, int line, char **err)
 {
-       enum jwt_alg alg;
+       enum jwt_alg alg = JWT_ALG_DEFAULT;
+
        vars_check_arg(&args[0], NULL);
        vars_check_arg(&args[1], NULL);