[jdk11u-dev] Integrated: 8315380: AsyncGetCallTrace crash in frame::safe_for_sender
Jaroslav Bachorik
jbachorik at openjdk.org
Mon May 12 03:41:06 UTC 2025
On Wed, 26 Feb 2025 11:11:12 GMT, Jaroslav Bachorik <jbachorik at openjdk.org> wrote:
> This change is fixing the problem in `frame_aarch64.cpp`, function `safe_for_sender`, where we have this code
>
>
> bool unextended_sp_safe = unextended_sp < thread->stack_base();
>
>
> While this captures one possibility of not being safe, it omits the check for `unextended_sp` falling within the stack space.
>
> The proposed change then is
>
>
> bool unextended_sp_safe = (unextended_sp < thread->stack_base() && \
> sp >= thread->stack_base() - thread->stack_size());
>
>
> This is actually just making sure the behaviour is the same as in JDK 15+ (since [JDK-8238988](https://bugs.openjdk.org/browse/JDK-8238988)) where the `unextended_sp` is checked for being within the stack limits.
>
>
> The change is not accompanied by a JTReg test because I was not able to craft one triggering the issue reliably.
>
> Existing tests from tier1-tier4 were run on a linux-aarch64 system with no new failures observed.
This pull request has now been integrated.
Changeset: d46f769e
Author: Jaroslav Bachorik <jbachorik at openjdk.org>
URL: https://git.openjdk.org/jdk11u-dev/commit/d46f769e70f6c0e8effcb78310cacc391a14fd6f
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
8315380: AsyncGetCallTrace crash in frame::safe_for_sender
Reviewed-by: phh, shade, apangin
-------------
PR: https://git.openjdk.org/jdk11u-dev/pull/3003
More information about the jdk-updates-dev
mailing list