On 11/07/2017 07:25 PM, Peter Maydell wrote: > In commit 7c4ee5bcc82e643 we changed the order in which we construct > the AUXV, but forgot to adjust the calculation of the length. The > result is that we set info->auxv_len to a bogus and negative value, > and then later on the code in open_self_auxv() gets confused and > ends up presenting the guest with an empty file. > > Since we now have to calculate the auxv length up-front as part > of figuring out how much we're going to put on the stack, set > info->auxv_len then; this allows us to assert that we put the > same number of entries into auxv as we pre-calculated, rather > than merely having a comment saying we need to do that. > > Fixes: https://bugs.launchpad.net/qemu/+bug/1728116 > > Signed-off-by: Peter Maydell <[email protected]> > --- > linux-user/elfload.c | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-)
Reviewed-by: Richard Henderson <[email protected]> r~
