RFR: 8299329: Assertion failure with fastdebug build when trying to use CDS without classpath [v3]

Ashutosh Mehra duke at openjdk.org
Wed Jan 4 17:40:58 UTC 2023


> It appears longest_common_app_classpath_prefix_len() is not returning correct value when there is no file separator in the path being searched backwards. Instead of returning 0 it return 1.
> 
> In case of empty classpath, it can result in assertion failure in check_paths():
> 
> `assert(strlen(rp_array->at(i)) > (size_t)runtime_prefix_len, "sanity");`
> 
> It can also result in incorrectly validating the app classpaths if they only differ by first character. Eg:
> 
> Dump time:
>   -cp hello.jar
> Run time:
>   -cp mello.jar
> 
> This would not result in classpath mismatch!
> 
> This fix updates longest_common_app_classpath_prefix_len() to return 0 if no file separator character is found.
> 
> Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>

Ashutosh Mehra has updated the pull request incrementally with one additional commit since the last revision:

  Fix whitespace
  
  Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11781/files
  - new: https://git.openjdk.org/jdk/pull/11781/files/b0da2921..be0ae2c7

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=11781&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=11781&range=01-02

  Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/11781.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11781/head:pull/11781

PR: https://git.openjdk.org/jdk/pull/11781


More information about the hotspot-runtime-dev mailing list