diff options
| author | Petr Štetiar <ynezz@true.cz> | 2022-09-22 11:49:13 +0200 |
|---|---|---|
| committer | Petr Štetiar <ynezz@true.cz> | 2022-09-27 09:51:35 +0200 |
| commit | d2081ec7a1ba302aa3f4414c007e782e08a32f6e (patch) | |
| tree | 01a23b88eb85ad41a69da76e1187eda4ea4eee52 /lib/path.c | |
| parent | 2aa5e706404e014accb2160596b7b47e179fa9dd (diff) | |
| download | util-linux-d2081ec7a1ba302aa3f4414c007e782e08a32f6e.tar.gz | |
lib/path: ul_path_cpuparse: fix parsing of empty sysfs files
Kernel 5.15 returns empty content for topology/thread_siblings on
aarch64 platform, which in conjunction of uninitialized `buf` memory
buffer results in the garbage:
(gdb) p buf
$14 = " @\377\367\177\000\000\000\275\000\347j\032\236"
This garbage is then being later consumed by underlying helper functions
like for example cpumask_parse() and this leads to the following crash
later:
in __libc_free (p=0x7ff7f67c00) at src/malloc/mallocng/free.c:105
in free (p=<optimized out>) at src/malloc/free.c:5
in add_cpuset_to_array (setsize=<optimized out>, set=<optimized out>, items=<optimized out>, ary=<optimized out>) at ../sys-utils/lscpu-topology.c:29
in cputype_read_topology (cxt=cxt@entry=0x7ff7fffe70, ct=0x4298a0) at ../sys-utils/lscpu-topology.c:153
in lscpu_read_topology (cxt=cxt@entry=0x7ff7fffe70) at ../sys-utils/lscpu-topology.c:629
in main (argc=1, argv=0x7ffffffdb8) at ../sys-utils/lscpu.c:1341
It looks like the problem is that current logic expects fgets() to set
errno on failure, but fgets() is not documented to do so and and neither
glibc nor musl set errno. So if errno was set to 0 before fgets() call,
the failure from fgets() is ignored and then invalid buffer is being
parsed.
Fixes: #1810
Suggested-by: Thomas Weißschuh <thomas@t-8ch.de>
Signed-off-by: Petr Štetiar <ynezz@true.cz>
Diffstat (limited to 'lib/path.c')
| -rw-r--r-- | lib/path.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/path.c b/lib/path.c index 42b4eadf16..8da6c06db0 100644 --- a/lib/path.c +++ b/lib/path.c @@ -1028,7 +1028,7 @@ static int ul_path_cpuparse(struct path_cxt *pc, cpu_set_t **set, int maxcpus, i if (!f) return -errno; - rc = fgets(buf, len, f) == NULL ? -errno : 0; + rc = fgets(buf, len, f) == NULL ? -EIO : 0; fclose(f); if (rc) |