Message ID | 20231115-qemu-user-dumpable-v1-1-edbe7f0fbb02@t-8ch.de |
---|---|
State | New |
Headers | show |
Series | linux-user: two fixes to coredump generation | expand |
diff --git a/linux-user/elfload.c b/linux-user/elfload.c index 4cd6891d7b6a..799fe8497346 100644 --- a/linux-user/elfload.c +++ b/linux-user/elfload.c @@ -4667,8 +4667,7 @@ static int elf_core_dump(int signr, const CPUArchState *env) init_note_info(&info); errno = 0; - getrlimit(RLIMIT_CORE, &dumpsize); - if (dumpsize.rlim_cur == 0) + if (getrlimit(RLIMIT_CORE, &dumpsize) == 0 && dumpsize.rlim_cur == 0) return 0; corefile = core_dump_filename(ts);
Should getrlimit() fail the value of dumpsize.rlimit_cur may not be initialized. Avoid reading garbage data by checking the return value of getrlimit. Signed-off-by: Thomas Weißschuh <thomas@t-8ch.de> --- linux-user/elfload.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)