diff --git a/app/src/main/java/com/wireguard/android/Application.java b/app/src/main/java/com/wireguard/android/Application.java index 74cb07cf..eaf2d75c 100644 --- a/app/src/main/java/com/wireguard/android/Application.java +++ b/app/src/main/java/com/wireguard/android/Application.java @@ -67,6 +67,8 @@ public class Application extends android.app.Application { ToolsInstaller getToolsInstaller(); TunnelManager getTunnelManager(); + + RootShell getRootShell(); } @Qualifier diff --git a/app/src/main/java/com/wireguard/android/preference/VersionPreference.java b/app/src/main/java/com/wireguard/android/preference/VersionPreference.java index b1a1cf49..2cce769d 100644 --- a/app/src/main/java/com/wireguard/android/preference/VersionPreference.java +++ b/app/src/main/java/com/wireguard/android/preference/VersionPreference.java @@ -16,6 +16,14 @@ import com.wireguard.android.BuildConfig; import com.wireguard.android.R; import com.wireguard.android.backend.GoBackend; import com.wireguard.android.backend.WgQuickBackend; +import com.wireguard.android.util.RootShell; + +import java.util.ArrayList; +import java.util.List; + +import javax.inject.Inject; + +import java9.util.concurrent.CompletionStage; public class VersionPreference extends Preference { private String versionSummary; @@ -26,10 +34,17 @@ public class VersionPreference extends Preference { if (Application.getComponent().getBackendType() == GoBackend.class) { versionSummary = getContext().getString(R.string.version_userspace_summary, GoBackend.getVersion()); } else if (Application.getComponent().getBackendType() == WgQuickBackend.class) { - Application.getComponent().getToolsInstaller().getVersion().whenComplete((version, exception) -> { + versionSummary = getContext().getString(R.string.version_kernel_summary_checking); + Application.getComponent().getAsyncWorker().supplyAsync(() -> { + final List output = new ArrayList<>(); + if (Application.getComponent().getRootShell() + .run(output, "cat /sys/module/wireguard/version") != 0 || output.isEmpty()) + throw new RuntimeException("Unable to determine kernel module version"); + return output.get(0); + }).whenComplete((version, exception) -> { versionSummary = exception == null ? getContext().getString(R.string.version_kernel_summary, version) - : getContext().getString(R.string.version_kernel_unknown_summary); + : getContext().getString(R.string.version_kernel_summary_unknown); notifyChanged(); }); } diff --git a/app/src/main/java/com/wireguard/android/util/ToolsInstaller.java b/app/src/main/java/com/wireguard/android/util/ToolsInstaller.java index 2b8280cc..d4a96f84 100644 --- a/app/src/main/java/com/wireguard/android/util/ToolsInstaller.java +++ b/app/src/main/java/com/wireguard/android/util/ToolsInstaller.java @@ -58,15 +58,6 @@ public final class ToolsInstaller { this.rootShell = rootShell; } - public CompletionStage getVersion() { - return Application.getComponent().getAsyncWorker().supplyAsync(() -> { - final List output = new ArrayList<>(); - if (rootShell.run(output, "cat /sys/module/wireguard/version") != 0 || output.isEmpty()) - throw new RuntimeException("Unable to determine kernel module version"); - return output.get(0); - }); - } - private static File getInstallDir() { final String path = System.getenv("PATH"); if (path == null) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index a9158b37..1a7d8afd 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -83,7 +83,8 @@ Successfully renamed tunnel to ā€œ%sā€ WireGuard for Android v%s" Kernel module backend v%s - Unknown kernel module version + Checking kernel module backend version + Unknown kernel module version Go userspace backend v%s Export tunnels to zip file Unable to export tunnels: %s