32 lines
1.1 KiB
Diff
32 lines
1.1 KiB
Diff
diff --git a/src/tools/opensc-explorer.c b/src/tools/opensc-explorer.c
|
|
index 41e620a..57f8a79 100644
|
|
--- a/src/tools/opensc-explorer.c
|
|
+++ b/src/tools/opensc-explorer.c
|
|
@@ -1839,6 +1839,12 @@ static int do_apdu(int argc, char **argv)
|
|
if (argc < 1)
|
|
return usage(do_apdu);
|
|
|
|
+ /* gcc-11 complains about BUF potentially being used without being
|
|
+ initialized. I can't convince myself that the calls to
|
|
+ parse_string_or_hexdata will fully initialize it, so we just
|
|
+ initialize it here. */
|
|
+ memset (buf, 0, sizeof (buf));
|
|
+
|
|
/* loop over the args and parse them, making sure the result fits into buf[] */
|
|
for (i = 0, len = 0; i < (unsigned) argc && len < sizeof(buf); i++) {
|
|
size_t len0 = sizeof(buf) - len;
|
|
commit 1680b3a1fb15319e41dbe3214ef8c4a4c215d529
|
|
Author: Jakub Jelen <jjelen@redhat.com>
|
|
Date: Tue Feb 23 19:57:02 2021 +0100
|
|
|
|
Fix build on gcc11
|
|
|
|
This made most of the applications crashing in Fedora 34 when
|
|
smart card was plugged in.
|
|
|
|
The suggested patch makes the code path more obvious for gcc to
|
|
handle.
|
|
|
|
https://bugzilla.redhat.com/show_bug.cgi?id=1930652
|
|
|