Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
home:bmwiedemann:reproducible:distribution:ring1
pesign
pesign-bsc1221694-fix-reversed-calloc-arguments...
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File pesign-bsc1221694-fix-reversed-calloc-arguments.patch of Package pesign
From 1f9e2fa0b4d872fdd01ca3ba81b04dfb1211a187 Mon Sep 17 00:00:00 2001 From: Stephen Gallagher <sgallagh@redhat.com> Date: Fri, 2 Feb 2024 09:32:48 -0500 Subject: [PATCH] Fix reversed calloc() arguments The prototype is "void *calloc(size_t nelem, size_t elsize);" These two instances had them reversed, almost certainly leading to buffer overflow issues. This was detected by -Werror=calloc-transposed-args on gcc. Signed-off-by: Stephen Gallagher <sgallagh@redhat.com> --- src/pesigcheck.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/pesigcheck.c b/src/pesigcheck.c index 6dc67f7..8119cf1 100644 --- a/src/pesigcheck.c +++ b/src/pesigcheck.c @@ -240,7 +240,7 @@ check_signature(pesigcheck_context *ctx, int *nreasons, cert_iter iter; - reasonps = calloc(sizeof(struct reason), 512); + reasonps = calloc(512, sizeof(struct reason)); if (!reasonps) err(1, "check_signature"); @@ -281,7 +281,7 @@ check_signature(pesigcheck_context *ctx, int *nreasons, num_reasons += 16; - new_reasons = calloc(sizeof(struct reason), num_reasons); + new_reasons = calloc(num_reasons, sizeof(struct reason)); if (!new_reasons) err(1, "check_signature"); reasonps = new_reasons; -- 2.35.3
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor