diff options
author | Andrew Burgess <aburgess@redhat.com> | 2022-10-19 15:07:40 +0100 |
---|---|---|
committer | Andrew Burgess <aburgess@redhat.com> | 2022-10-24 17:12:11 +0100 |
commit | 548d634f1b61571f118c3133ce0e8986714c8fd6 (patch) | |
tree | 2ae81644447f769cfade1586f24b21ec17df9b7b /sim/ppc/emul_netbsd.c | |
parent | 368b8c325922ca329ae0edb1a9ce6bc16c9f927f (diff) | |
download | gdb-548d634f1b61571f118c3133ce0e8986714c8fd6.zip gdb-548d634f1b61571f118c3133ce0e8986714c8fd6.tar.gz gdb-548d634f1b61571f118c3133ce0e8986714c8fd6.tar.bz2 |
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.
Diffstat (limited to 'sim/ppc/emul_netbsd.c')
-rw-r--r-- | sim/ppc/emul_netbsd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sim/ppc/emul_netbsd.c b/sim/ppc/emul_netbsd.c index 322b584..a8bfd27 100644 --- a/sim/ppc/emul_netbsd.c +++ b/sim/ppc/emul_netbsd.c @@ -879,7 +879,7 @@ do_fstat(os_emul_data *emul, { int fd = cpu_registers(processor)->gpr[arg0]; unsigned_word stat_buf_addr = cpu_registers(processor)->gpr[arg0+1]; - struct stat buf; + struct stat buf = {}; int status; #ifdef SYS_fstat SYS(fstat); |