mirror/buildroot
1
0
Fork 0
mirror of https://gitlab.com/buildroot.org/buildroot.git synced 2026-08-08 08:30:47 -09:00
Code Issues Packages Projects Releases Wiki Activity

package/safeclib: fix build with gcc >= 14.x

Browse Source 
Backport two upstream patches to fix build errors introduced by the
bump of gcc to 14.x.

Fixes:
https://autobuild.buildroot.net/results/af6/af65e6386439098ddf706ca43e99320cf5e9fd80/

Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
This commit is contained in:
Bernd Kuhls
2026-02-03 12:47:25 +01:00
committed by Thomas Petazzoni
parent c15b507838
commit 34749a2d3e
2 changed files with 100 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

51
package/safeclib/0002-vsnprintf_s-Increase-Buffer-Size-by-1.patch Normal file
View File

@@ -0,0 +1,51 @@
From 62051f9761f92dc99c8ce0552239ad10e2062168 Mon Sep 17 00:00:00 2001
From: Bernd Kuhls <bernd@kuhls.net>
Date: Sat, 10 Jan 2026 12:16:22 +0100
Subject: [PATCH] vsnprintf_s: Increase Buffer Size by 1
Another fix similar to https://github.com/rurban/safeclib/commit/f59a0c8c1b5cf19cd0ed7f9bfb3a1e85f54113d0
In function 'safec_ntoa_format',
inlined from 'safec_ntoa_long' at str/vsnprintf_s.c:331:12:
str/vsnprintf_s.c:256:24: error: writing 32 bytes into a region of size 0 [-Werror=stringop-overflow=]
256 | buf[len++] = '0';
| ~~~~~~~~~~~^~~~~
str/vsnprintf_s.c: In function 'safec_ntoa_long':
str/vsnprintf_s.c:312:10: note: at offset 32 into destination object 'buf' of size 32
312 | char buf[PRINTF_NTOA_BUFFER_SIZE];
| ^~~
In function 'safec_ntoa_format',
inlined from 'safec_ntoa_long' at str/vsnprintf_s.c:331:12:
str/vsnprintf_s.c:260:24: error: writing 32 bytes into a region of size 0 [-Werror=stringop-overflow=]
260 | buf[len++] = '0';
| ~~~~~~~~~~~^~~~~
str/vsnprintf_s.c: In function 'safec_ntoa_long':
str/vsnprintf_s.c:312:10: note: at offset 32 into destination object 'buf' of size 32
312 | char buf[PRINTF_NTOA_BUFFER_SIZE];
| ^~~
cc1: all warnings being treated as errors
Upstream: https://github.com/rurban/safeclib/pull/150
[backported to version 3.7.1]
Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
src/str/vsnprintf_s.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/str/vsnprintf_s.c b/src/str/vsnprintf_s.c
index 59dbda94..ca838df1 100644
--- a/src/str/vsnprintf_s.c
+++ b/src/str/vsnprintf_s.c
@@ -296,7 +296,7 @@ static size_t safec_ntoa_long(out_fct_type out, const char *funcname,
unsigned long base, unsigned int prec,
unsigned int width, unsigned int flags)
{
- char buf[PRINTF_NTOA_BUFFER_SIZE];
+ char buf[PRINTF_NTOA_BUFFER_SIZE + 1];
size_t len = 0U;
// no hash for 0 values
--
2.47.3

49
package/safeclib/0003-vsnprintf_s-Increase-Buffer-Size-by-1.patch Normal file
View File

@@ -0,0 +1,49 @@
From dea3c2e1aa1b775baa690b9ef40239f881c5f068 Mon Sep 17 00:00:00 2001
From: Khem Raj <raj.khem@gmail.com>
Date: Wed, 13 Aug 2025 20:23:48 -0700
Subject: [PATCH] vsnprintf_s: Increase Buffer Size by 1
It is a buffer overflow warning that GCC 15.2 is catching.
The issue is that it's trying to write to `buf[len++]` when len could
potentially be 31, which would write to buf[31] in a buffer of size 32
(valid indices 0-31), but the len++ post-increment means it could
theoretically write beyond the buffer bounds.
Fixes
../../sources/safec-3.9.1/src/str/vsnprintf_s.c: In function 'safec_ftoa.isra':
../../sources/safec-3.9.1/src/str/vsnprintf_s.c:523:24: error: writing 32 bytes into a region of size 31 [-Werror=stringop-overflow=]
523 | buf[len++] = '0';
| ~~~~~~~~~~~^~~~~
../../sources/safec-3.9.1/src/str/vsnprintf_s.c:394:10: note: at offset [1, 32] into destination object 'buf' of size 32
394 | char buf[PRINTF_FTOA_BUFFER_SIZE];
| ^~~
cc1: all warnings being treated as errors
Upstream-Status: Submitted [https://github.com/rurban/safeclib/pull/148]
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Upstream: https://github.com/rurban/safeclib/commit/f59a0c8c1b5cf19cd0ed7f9bfb3a1e85f54113d0
[backported to version 3.7.1]
Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
---
src/str/vsnprintf_s.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/str/vsnprintf_s.c b/src/str/vsnprintf_s.c
index ca838df1..8ef6989a 100644
--- a/src/str/vsnprintf_s.c
+++ b/src/str/vsnprintf_s.c
@@ -369,7 +369,7 @@ static size_t safec_ftoa(out_fct_type out, const char *funcname,
double value, unsigned int prec, unsigned int width,
unsigned int flags)
{
- char buf[PRINTF_FTOA_BUFFER_SIZE];
+ char buf[PRINTF_FTOA_BUFFER_SIZE + 1];
size_t len = 0U;
double tmp;
double diff = 0.0;
--
2.47.3