sim/ppc: initialize a memory buffer in all cases
authorAndrew Burgess <aburgess@redhat.com>
Wed, 19 Oct 2022 14:07:40 +0000 (15:07 +0100)
committerAndrew Burgess <aburgess@redhat.com>
Mon, 24 Oct 2022 16:12:11 +0000 (17:12 +0100)
commit548d634f1b61571f118c3133ce0e8986714c8fd6
tree2ae81644447f769cfade1586f24b21ec17df9b7b
parent368b8c325922ca329ae0edb1a9ce6bc16c9f927f
sim/ppc: initialize a memory buffer in all cases

In the ppc simulator's do_fstat function, which provides the fstat
call for the simulator, if the fstat is going to fail then we
currently write an uninitialized buffer into the simulated target.

In theory, I think this is fine, we also write the error status into
the simulated target, so, given that the fstat has failed, the target
shouldn't be relying on the buffer contents.

However, writing an uninitialized buffer means we might leak simulator
private data into the simulated target, which is probably a bad thing.
Plus it probably makes life easier if something consistent, like all
zeros, is written rather than random junk, which might look like a
successful call (except for the error code).

So, in this commit, I initialize the stat buffer to zero before
it is potentially used.  If the stat call is not made then the buffer
will be left initialized as all zeros.
sim/ppc/emul_netbsd.c