Branch: refs/heads/bolo/x86_immed
Home: https://github.com/dyninst/dyninst
Commit: 070666f480303bab037006a04a014201c464c839
https://github.com/dyninst/dyninst/commit/070666f480303bab037006a04a014201c464c839
Author: Bolo <b...@cs.wisc.edu>
Date: 2023-02-17 (Fri, 17 Feb 2023)
Changed paths:
M common/src/arch-x86.C
M common/src/arch-x86.h
M instructionAPI/h/Result.h
Log Message:
-----------
x86 8-bit immediate values were interpreted incorrectly cross-platform.
Correct interpretation of Intel instructions requires
sign extension. On intel it works because chars are
signed on that platform. On other platforms (arm64, powerpc64)
intel code was not correctly analyzed because incorrect
values were generated from 8 bit immediate constants;
which were interpreted as unsigned on those systems.
These changes ensure intel code will be correctly interpreted
on any system, regardless of the signedness of chars.
A n-way validation was used to validate this change, to
ensure that correct code interpretation of (intel, arm,
power) code happens on all three platforms. No other
issues of this nature were located -- at this time.
Jim Kupsch contributed additional correctness changes to
common/src/arch-x86.C that were added to this
branch/PR to fix yet more problems of this ilk.
_______________________________________________
Dyninst-api mailing list
Dyninst-api@cs.wisc.edu
https://lists.cs.wisc.edu/mailman/listinfo/dyninst-api
