Make NativeLibrary#encoding final, #options and #callFlags private

matthiasblaesing · matthiasblaesing · commit 458adcf3dde6 · 2023-12-04T21:49:10.000+01:00
Reported by: @sebbASF Closes: #1560
diff --git a/src/com/sun/jna/Function.java b/src/com/sun/jna/Function.java
@@ -245,7 +245,7 @@ public static Function getFunction(Pointer p, int callFlags, String encoding) {
         this.library = library;
         this.functionName = functionName;
         this.callFlags = callFlags;
-        this.options = library.options;
+        this.options = library.getOptions();
         this.encoding = encoding != null ? encoding : Native.getDefaultStringEncoding();
         try {
             this.peer = library.getSymbolAddress(functionName);
diff --git a/src/com/sun/jna/NativeLibrary.java b/src/com/sun/jna/NativeLibrary.java
@@ -101,9 +101,9 @@ public long getSymbolAddress(long handle, String name, SymbolProvider parent) {
     private final String libraryPath;
     private final Map<String, Function> functions = new HashMap<>();
     private final SymbolProvider symbolProvider;
-    final int callFlags;
-    private String encoding;
-    final Map<String, ?> options;
+    private final int callFlags;
+    private final String encoding;
+    private final Map<String, ?> options;
 
     private static final Map<String, Reference<NativeLibrary>> libraries = new HashMap<>();
 
@@ -120,6 +120,7 @@ private static String functionKey(String name, int flags, String encoding) {
         return name + "|" + flags + "|" + encoding;
     }
 
+    @SuppressWarnings("LeakingThisInConstructor")
     private NativeLibrary(String libraryName, String libraryPath, long handle, Map<String, ?> options) {
         this.libraryName = getLibraryName(libraryName);
         this.libraryPath = libraryPath;
@@ -129,17 +130,18 @@ private NativeLibrary(String libraryName, String libraryPath, long handle, Map<S
         int callingConvention = option instanceof Number ? ((Number)option).intValue() : Function.C_CONVENTION;
         this.callFlags = callingConvention;
         this.options = options;
-        this.encoding = (String)options.get(Library.OPTION_STRING_ENCODING);
         SymbolProvider optionSymbolProvider = (SymbolProvider)options.get(Library.OPTION_SYMBOL_PROVIDER);
         if (optionSymbolProvider == null) {
             this.symbolProvider = NATIVE_SYMBOL_PROVIDER;
         } else {
             this.symbolProvider = optionSymbolProvider;
         }
 
-        if (this.encoding == null) {
-            this.encoding = Native.getDefaultStringEncoding();
+        String encodingValue = (String) options.get(Library.OPTION_STRING_ENCODING);
+        if (encodingValue == null) {
+            encodingValue = Native.getDefaultStringEncoding();
         }
+        this.encoding = encodingValue;
 
         // Special workaround for w32 kernel32.GetLastError
         // Short-circuit the function to use built-in GetLastError access
