Backport a minor fix.
Signed-off-by: Peter Jones <pjones@redhat.com>
This commit is contained in:
parent
f4cb5bfd7e
commit
6a21c3cf8a
49
0031-pesigcheck-Fix-a-wrong-assignment.patch
Normal file
49
0031-pesigcheck-Fix-a-wrong-assignment.patch
Normal file
@ -0,0 +1,49 @@
|
|||||||
|
From c555fd74c009242c3864576bd5f17a1f8f4fdffd Mon Sep 17 00:00:00 2001
|
||||||
|
From: Peter Jones <pjones@redhat.com>
|
||||||
|
Date: Tue, 18 Feb 2020 16:28:56 -0500
|
||||||
|
Subject: [PATCH] pesigcheck: Fix a wrong assignment
|
||||||
|
|
||||||
|
gcc says:
|
||||||
|
|
||||||
|
pesigcheck.c: In function 'check_signature':
|
||||||
|
pesigcheck.c:321:17: error: implicit conversion from 'enum <anonymous>' to 'enum <anonymous>' [-Werror=enum-conversion]
|
||||||
|
321 | reason->type = siBuffer;
|
||||||
|
| ^
|
||||||
|
pesigcheck.c:333:17: error: implicit conversion from 'enum <anonymous>' to 'enum <anonymous>' [-Werror=enum-conversion]
|
||||||
|
333 | reason->type = siBuffer;
|
||||||
|
| ^
|
||||||
|
cc1: all warnings being treated as errors
|
||||||
|
|
||||||
|
And indeed, that line of code makes no sense at all - it was supposed to
|
||||||
|
be reason->sig.type.
|
||||||
|
|
||||||
|
Signed-off-by: Peter Jones <pjones@redhat.com>
|
||||||
|
---
|
||||||
|
src/pesigcheck.c | 4 ++--
|
||||||
|
1 file changed, 2 insertions(+), 2 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/pesigcheck.c b/src/pesigcheck.c
|
||||||
|
index 524cce307bf..8fa0f1ad03d 100644
|
||||||
|
--- a/src/pesigcheck.c
|
||||||
|
+++ b/src/pesigcheck.c
|
||||||
|
@@ -318,7 +318,7 @@ check_signature(pesigcheck_context *ctx, int *nreasons,
|
||||||
|
reason->type = SIGNATURE;
|
||||||
|
reason->sig.data = data;
|
||||||
|
reason->sig.len = datalen;
|
||||||
|
- reason->type = siBuffer;
|
||||||
|
+ reason->sig.type = siBuffer;
|
||||||
|
nreason += 1;
|
||||||
|
is_invalid = true;
|
||||||
|
}
|
||||||
|
@@ -330,7 +330,7 @@ check_signature(pesigcheck_context *ctx, int *nreasons,
|
||||||
|
reason->type = SIGNATURE;
|
||||||
|
reason->sig.data = data;
|
||||||
|
reason->sig.len = datalen;
|
||||||
|
- reason->type = siBuffer;
|
||||||
|
+ reason->sig.type = siBuffer;
|
||||||
|
nreason += 1;
|
||||||
|
has_valid_cert = true;
|
||||||
|
}
|
||||||
|
--
|
||||||
|
2.24.1
|
||||||
|
|
@ -71,6 +71,7 @@ Patch0027: 0027-Make-pesign-users-groups-static-in-the-repo.patch
|
|||||||
Patch0028: 0028-rpm-Make-the-client-signer-use-the-fedora-values-unl.patch
|
Patch0028: 0028-rpm-Make-the-client-signer-use-the-fedora-values-unl.patch
|
||||||
Patch0029: 0029-Make-macros.pesign-error-in-kojibuilder-if-we-don-t-.patch
|
Patch0029: 0029-Make-macros.pesign-error-in-kojibuilder-if-we-don-t-.patch
|
||||||
Patch0030: 0030-efikeygen-Fix-the-build-with-nss-3.44.patch
|
Patch0030: 0030-efikeygen-Fix-the-build-with-nss-3.44.patch
|
||||||
|
Patch0031: 0031-pesigcheck-Fix-a-wrong-assignment.patch
|
||||||
|
|
||||||
%description
|
%description
|
||||||
This package contains the pesign utility for signing UEFI binaries as
|
This package contains the pesign utility for signing UEFI binaries as
|
||||||
|
Loading…
Reference in New Issue
Block a user