kumquat-buildroot/package/tpm2-totp/0002-src-fix-compilation-failure-due-to-variable-may-be-u.patch

43 lines
1.3 KiB
Diff
Raw Normal View History

From 194f41635367452a7a3c9a75ebbada531bf4c58d Mon Sep 17 00:00:00 2001
From: Carlos Santos <unixmania@gmail.com>
Date: Sun, 26 May 2019 13:39:44 -0300
Subject: [PATCH] src: fix compilation failure due to "variable may be used
uninitialized"
Some inline declarations of strtok_r (specifically in Sourcery CodeBench
Lite 2016.11-19) contain code where an '__s' local variable can be used
uninitialized.
When GCC expands that declaration in 'parse_pcrs', __s becomes an alias
to the local variable 'saveptr', which in fact is not initialized, but
this is not relevant, since the 'str' argument is knowingly not NULL
when passed to strtok_r because it comes from 'optarg' in parse_opts.
Anyway, initialize saveptr to NULL to prevent the compilation error.
Fixes:
http://autobuild.buildroot.net/results/5693a35e4d6bc76a1f46fe0e217abc49f7188aad/
Change-Id: I03ad3731774c56744f18154ec161c92ba002903d
Signed-off-by: Carlos Santos <unixmania@gmail.com>
---
src/tpm2-totp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/tpm2-totp.c b/src/tpm2-totp.c
index 3f60b4a..f28a4d6 100644
--- a/src/tpm2-totp.c
+++ b/src/tpm2-totp.c
@@ -93,7 +93,7 @@ int
parse_pcrs(char *str, int *pcrs)
{
char *token;
- char *saveptr;
+ char *saveptr = NULL;
char *endptr;
long pcr;
--
2.20.1