x86/a.out: Clear the dump structure initially
authorBorislav Petkov <bp@suse.de>
Tue, 12 Feb 2019 13:28:03 +0000 (14:28 +0100)
committerBorislav Petkov <bp@suse.de>
Wed, 13 Feb 2019 11:10:51 +0000 (12:10 +0100)
commit10970e1b4be9c74fce8ab6e3c34a7d718f063f2c
tree679b860fb9f0bb90068d414aca73005dc007cc9e
parent20e55bc17dd01f13cec0eb17e76e9511b23963ef
x86/a.out: Clear the dump structure initially

dump_thread32() in aout_core_dump() does not clear the user32 structure
allocated on the stack as the first thing on function entry.

As a result, the dump.u_comm, dump.u_ar0 and dump.signal which get
assigned before the clearing, get overwritten.

Rename that function to fill_dump() to make it clear what it does and
call it first thing.

This was caught while staring at a patch by Derek Robson
<robsonde@gmail.com>.

Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: Derek Robson <robsonde@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Michael Matz <matz@suse.de>
Cc: x86@kernel.org
Cc: <stable@vger.kernel.org>
Link: https://lkml.kernel.org/r/20190202005512.3144-1-robsonde@gmail.com
arch/x86/ia32/ia32_aout.c