Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Optimize library loading #1013

Merged
merged 2 commits into from
Sep 21, 2018
Merged

Optimize library loading #1013

merged 2 commits into from
Sep 21, 2018
+36 −15

Conversation

matthiasblaesing
Copy link
Member

@matthiasblaesing matthiasblaesing commented Sep 15, 2018 •
edited
Loading

Two problems are addressed here:

  1. it is a long standing issue, that JNA should default to not load the jnidispatch library from the system search path. This was again observed when introducing appveyor builds, as there somehow a jnidispatch library is on the system path
  2. Unix systems (Mac OS X and Linux at least) default to a global system temp directory. This is unsafe, as an attacker could gain control over that directory. This is nor fixed.

This was referenced Sep 15, 2018
Closed
Closed
@matthiasblaesing matthiasblaesing force-pushed the optimize_library_loading branch 2 times, most recently from 6df1c87 to 9fdbc6a Compare September 20, 2018 19:05
@matthiasblaesing
[java-native-access#384] Make jna.nosys=true default
d6289ec 
Closes: java-native-access#384
@matthiasblaesing
[java-native-access#985] Improve handling of dynamicaly extracted nat…
fe02b87 
…ive library.

On Mac OS X systems `~/Library/Caches/JNA/temp` and on
other Unix like systems `$XDG_CACHE_DIR/JNA/temp` (Default value is:
`~/.cache/JNA/temp`) is used.

Behaviour on windows is unchanged, as since Vista the tempdir is by
default inside the user profile directory.

Closes: java-native-access#985
@matthiasblaesing matthiasblaesing merged commit 088ad82 into java-native-access:master Sep 21, 2018
@matthiasblaesing matthiasblaesing deleted the optimize_library_loading branch September 22, 2018 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@matthiasblaesing