From 1f5095c1d304b38f014f9da0bf4351218f44a4b8 Mon Sep 17 00:00:00 2001 From: Devreese Jorik Date: Thu, 7 May 2026 14:35:34 +0200 Subject: [PATCH] support/kconfig: fix compiler warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Commit 324612d68e09d0db44c51c451bc6b87466f7f874 fixed several compiler warnings, but actually introduced a new one by increasing the buffer size in confdata.c that gets passed along to file_write_dep in util.c, because buf2's size wasn't increased along with it. ./util.c: In function ‘file_write_dep’: ./util.c:86:26: warning: ‘%s’ directive writing 10 or more bytes into a region of size between 1 and 4097 [-Wformat-overflow=] 86 | sprintf(buf2, "%s%s", dir, name); | ^~ ./util.c:86:9: note: ‘sprintf’ output 11 or more bytes (assuming 4107) into a destination of size 4097 86 | sprintf(buf2, "%s%s", dir, name); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Fix this by increasing the size of buf2 to match the passed buffer size. Signed-off-by: Devreese Jorik Signed-off-by: Thomas Devoogdt Signed-off-by: Peter Korsgaard --- support/kconfig/util.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/support/kconfig/util.c b/support/kconfig/util.c index 8665f5bb89..c2e7979ac1 100644 --- a/support/kconfig/util.c +++ b/support/kconfig/util.c @@ -35,7 +35,7 @@ struct file *file_lookup(const char *name) int file_write_dep(const char *name) { char *str; - char buf[PATH_MAX+20], buf2[PATH_MAX+1], dir[PATH_MAX+1]; + char buf[PATH_MAX+20], buf2[PATH_MAX+20], dir[PATH_MAX+1]; struct symbol *sym, *env_sym; struct expr *e; struct file *file;