From: Thomas Weißschuh <tho...@t-8ch.de> Should getrlimit() fail the value of dumpsize.rlimit_cur may not be initialized. Avoid reading garbage data by checking the return value of getrlimit.
Reviewed-by: Richard Henderson <richard.hender...@linaro.org> Signed-off-by: Thomas Weißschuh <tho...@t-8ch.de> Message-Id: <20240120-qemu-user-dumpable-v3-1-6aa410c93...@t-8ch.de> Signed-off-by: Richard Henderson <richard.hender...@linaro.org> --- linux-user/elfload.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/linux-user/elfload.c b/linux-user/elfload.c index cf9e74468b..c596871938 100644 --- a/linux-user/elfload.c +++ b/linux-user/elfload.c @@ -4667,9 +4667,9 @@ 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); -- 2.34.1