On Thu, 2017-10-05 at 11:41 +0200, Borislav Petkov wrote: > On Tue, Oct 03, 2017 at 08:54:09PM -0700, Ricardo Neri wrote: > > > > Even though memory addresses are unsigned, the operands used to compute the > > effective address do have a sign. This is true for ModRM.rm, SIB.base, > > SIB.index as well as the displacement bytes. Thus, signed variables shall > > be used when computing the effective address from these operands. Once the > > signed effective address has been computed, it is casted to an unsigned > > long to determine the linear address. > > > > Variables are renamed to better reflect the type of address being > > computed. > > > > Cc: Borislav Petkov <[email protected]> > > Cc: Andy Lutomirski <[email protected]> > > Cc: Dave Hansen <[email protected]> > > Cc: Adam Buchbinder <[email protected]> > > Cc: Colin Ian King <[email protected]> > > Cc: Lorenzo Stoakes <[email protected]> > > Cc: Qiaowei Ren <[email protected]> > > Cc: Peter Zijlstra <[email protected]> > > Cc: Nathan Howard <[email protected]> > > Cc: Adan Hawthorn <[email protected]> > > Cc: Joe Perches <[email protected]> > > Cc: Ravi V. Shankar <[email protected]> > > Cc: [email protected] > > Signed-off-by: Ricardo Neri <[email protected]> > > --- > > arch/x86/mm/mpx.c | 20 ++++++++++++++------ > > 1 file changed, 14 insertions(+), 6 deletions(-) > Reviewed-by: Borislav Petkov <[email protected]>
Thank you! BR, Ricardo
