RFR: 8268129: LibraryLookup::ofDefault leaks symbols from loaded libraries [v2]
Maurizio Cimadamore
mcimadamore at openjdk.java.net
Wed Jun 2 20:17:20 UTC 2021
> This patch overhauls the library loading mechanism used by the Foreign Linker API. We realized that, while handy, the *default* lookup abstraction (`LibraryLookup::ofDefault`) was behaving inconsistentlt across platforms.
>
> This patch replaces `LibraryLookup` with a simpler `SymbolLookup` abstraction, a functional interface. Crucially, `SymbolLookup` does not concern with library loading, only symbol lookup. For this reason, two factories are added:
>
> * `SymbolLookup::loaderLookup` - which obtains a lookup that can be used to lookup symbols in libraries loaded by current loader
> * `CLinker::systemLookup` - a more stable replacement for the *default* lookup, which looks for symbols in libc.so (or its equivalent in other platforms). The contents of this lookup are unspecified.
>
> Both factories are *restricted*, so they can only be called when `--enable-native-access` is set.
Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
Update test/micro/org/openjdk/bench/jdk/incubator/foreign/points/support/PanamaPoint.java
Co-authored-by: Jorn Vernee <JornVernee at users.noreply.github.com>
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/4316/files
- new: https://git.openjdk.java.net/jdk/pull/4316/files/01d9c198..2b668f7c
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4316&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4316&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/4316.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/4316/head:pull/4316
PR: https://git.openjdk.java.net/jdk/pull/4316
More information about the security-dev
mailing list