[lworld] Integrated: 8271486: [lworld] Memory corruption due to out of bound access in MacroAssembler::move_helper
thartmann at openjdk.java.net
Thu Jul 29 17:47:46 UTC 2021
On Thu, 29 Jul 2021 16:04:59 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
> While debugging weird crashes that only showed up when merging current mainline with lworld, I've noticed that we are writing outside of the `reg_state` array in `MacroAssembler::move_helper` because `from->value()` is `-1` (`OptoReg::BAD_REG`):
> The register is invalid because it belongs to the second half of a `T_LONG` or `T_DOUBLE` argument in the calling convention and should simply be ignored. I've also added asserts to catch similar issues in the future.
This pull request has now been integrated.
Author: Tobias Hartmann <thartmann at openjdk.org>
Stats: 14 lines in 3 files changed: 11 ins; 0 del; 3 mod
8271486: [lworld] Memory corruption due to out of bound access in MacroAssembler::move_helper
More information about the valhalla-dev