kmodloader: Increase path array size to make it always fit
authorHauke Mehrtens <hauke@hauke-m.de>
Sat, 15 Jun 2019 21:49:28 +0000 (23:49 +0200)
committerHauke Mehrtens <hauke@hauke-m.de>
Sun, 16 Jun 2019 13:42:32 +0000 (15:42 +0200)
When DEF_MOD_PATH (10 bytes), s (max 255 bytes) and ver.release (max 64
bytes) are all written into path, 256 bytes will not fit, increase the
available size. All given sizes are given without terminating NULL byte
and we add one byte to store the NULL byte.

GCC 9.1 warns about this condition and we treat warnings as errors in
ubox.

Reported-by: Joseph Benden <joe@benden.us>
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
kmodloader.c

index 6a7d7c1f52bc8ad9b4f4da052974d6dd56616083..422c895f1f40daf19d8e9aacfc44db5964496ae6 100644 (file)
@@ -75,7 +75,7 @@ static int init_module_folders(void)
        int n = 0;
        struct stat st;
        struct utsname ver;
-       char *s, *e, *p, path[256], ldpath[256];
+       char *s, *e, *p, path[330], ldpath[256];
 
        e = ldpath;
        s = getenv("LD_LIBRARY_PATH");