On Thu, 22 Oct 2026 18:38:37 GMT, Chris Plummer <[email protected]> wrote:

> In the linux pathmap_open() you can see a check to make sure that SA_ALTROOT 
> is not too big. This check is missing on macosx, leading to the possibility 
> of a crash if SA_ALTROOT is too long. 
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Marked as reviewed by kevinw (Reviewer).

test/hotspot/jtreg/serviceability/sa/TestLongAltRootPath.java line 61:

> 59: 
> 60:             // Build a long path name to use for SA_ALTROOT that will 
> cause
> 61:             // a crasg if proper length checks are not made.

nit "crash" typo

-------------

PR Review: https://git.openjdk.org/jdk/pull/33520#pullrequestreview-5625815609
PR Review Comment: https://git.openjdk.org/jdk/pull/33520#discussion_r4357622318

Reply via email to