RFR: 8347143: [aix] Fix strdup use in os::dll_load [v3]

Joachim Kern jkern at openjdk.org
Mon Jan 13 17:12:00 UTC 2025


> The use of strdup (which is os::strdup because of context) in aix's os::dll_load is odd. The result is nearly unused (it's passed to strrchr, but the original filename could be used instead), so the duplication doesn't seem needed. And the duplicate is free'd using FREE_C_HEAP_ARRAY, which seems wrong.

Joachim Kern has updated the pull request incrementally with one additional commit since the last revision:

  Cosmetic changes

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/23078/files
  - new: https://git.openjdk.org/jdk/pull/23078/files/ef6b252f..a7cd928a

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

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

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


More information about the hotspot-runtime-dev mailing list