clang is unhappy about this. While I can't see how s_len could be used uninitialized, there's value in silencing these warnings, so the real issues are not as easy to overlook. Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> --- commands/keystore.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/commands/keystore.c b/commands/keystore.c index 4922cf1bebe9..40bcb7105d75 100644 --- a/commands/keystore.c +++ b/commands/keystore.c @@ -17,7 +17,7 @@ static int do_keystore(int argc, char *argv[]) const char *file = NULL; char *secret_str = NULL; void *secret; - size_t s_len; + size_t s_len = 0; while ((opt = getopt(argc, argv, "rs:f:")) > 0) { switch (opt) { -- 2.39.2