Move native functions into own class.

This way the Gui will not always load the native libraries when showing profiles and allows some GUI modifications without compiling ndk code

3 files changed, 15 insertions, 13 deletions

diff --git a/src/de/blinkt/openvpn/VpnProfile.java b/src/de/blinkt/openvpn/VpnProfile.java
index 67a2a142..9d183897 100644
--- a/src/de/blinkt/openvpn/VpnProfile.java
+++ b/src/de/blinkt/openvpn/VpnProfile.java
@@ -42,6 +42,7 @@ import android.security.KeyChain;
 import android.security.KeyChainException;
 import android.util.Base64;
 import de.blinkt.openvpn.R;
+import de.blinkt.openvpn.core.NativeUtils;
 import de.blinkt.openvpn.core.OpenVPN;
 import de.blinkt.openvpn.core.OpenVpnService;
 
@@ -140,12 +141,6 @@ public class VpnProfile implements  Serializable{
 
 	public static final String MINIVPN = "miniopenvpn";
 
-
-	static private native byte[] rsasign(byte[] input,int pkey) throws InvalidKeyException;
-	static {
-		System.loadLibrary("opvpnutil");
-	}
-
 	public void clearDefaults() {
 		mServerName="unkown";
 		mUsePull=false;
@@ -868,7 +863,7 @@ public class VpnProfile implements  Serializable{
 			int pkey = (Integer) getPkeyContext.invoke(opensslkey);
 			getPkeyContext.setAccessible(false);
 
-			byte[] signed_bytes = rsasign(data, pkey); 
+			byte[] signed_bytes = NativeUtils.rsasign(data, pkey); 
 			return Base64.encodeToString(signed_bytes, Base64.NO_WRAP);
 
 		} catch (NoSuchMethodException e) {
diff --git a/src/de/blinkt/openvpn/core/NativeUtils.java b/src/de/blinkt/openvpn/core/NativeUtils.java
new file mode 100644
index 00000000..42d0b583
--- /dev/null
+++ b/src/de/blinkt/openvpn/core/NativeUtils.java
@@ -0,0 +1,12 @@
+package de.blinkt.openvpn.core;
+
+import java.security.InvalidKeyException;
+
+public class NativeUtils {
+	public static native byte[] rsasign(byte[] input,int pkey) throws InvalidKeyException;
+	static native void jniclose(int fdint);
+
+	static {
+		System.loadLibrary("opvpnutil");
+	}
+}
diff --git a/src/de/blinkt/openvpn/core/OpenVpnManagementThread.java b/src/de/blinkt/openvpn/core/OpenVpnManagementThread.java
index a44f744e..5b465973 100644
--- a/src/de/blinkt/openvpn/core/OpenVpnManagementThread.java
+++ b/src/de/blinkt/openvpn/core/OpenVpnManagementThread.java
@@ -35,8 +35,6 @@ public class OpenVpnManagementThread implements Runnable, OpenVPNMangement {
 

 	private static Vector<OpenVpnManagementThread> active=new Vector<OpenVpnManagementThread>();

 

-	static private native void jniclose(int fdint);

-

 	public OpenVpnManagementThread(VpnProfile profile, LocalServerSocket mgmtsocket, OpenVpnService openVpnService) {

 		mProfile = profile;

 		mServerSocket = mgmtsocket;

@@ -128,7 +126,7 @@ public class OpenVpnManagementThread implements Runnable, OpenVPNMangement {
 

 			//ParcelFileDescriptor pfd = ParcelFileDescriptor.fromFd(fdint);

 			//pfd.close();

-			jniclose(fdint);

+			NativeUtils.jniclose(fdint);

 			return;

 		} catch (NoSuchMethodException e) {

 			exp =e;

@@ -426,9 +424,6 @@ public class OpenVpnManagementThread implements Runnable, OpenVPNMangement {
 	private void proccessPWFailed(String needed, String args) {

 		OpenVPN.updateStateString("AUTH_FAILED", needed + args,R.string.state_auth_failed,ConnectionStatus.LEVEL_AUTH_FAILED);

 	}

-	private void logStatusMessage(String command) {

-		OpenVPN.logMessage(0,"MGMT:", command);

-	}

 

 

 	private static boolean stopOpenVPN() {