From be8931078443e94400c995917c43977410b1fe6d Mon Sep 17 00:00:00 2001
From: Tobias Brunner <tobias@strongswan.org>
Date: Tue, 12 Jun 2018 17:05:00 +0200
Subject: [PATCH] android: Allow reconnecting without confirmation in case of
 an error

---
 .../android/ui/VpnProfileControlActivity.java | 22 +++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/src/frontends/android/app/src/main/java/org/strongswan/android/ui/VpnProfileControlActivity.java b/src/frontends/android/app/src/main/java/org/strongswan/android/ui/VpnProfileControlActivity.java
index 9bc0cf032..6f5cddd95 100644
--- a/src/frontends/android/app/src/main/java/org/strongswan/android/ui/VpnProfileControlActivity.java
+++ b/src/frontends/android/app/src/main/java/org/strongswan/android/ui/VpnProfileControlActivity.java
@@ -191,6 +191,24 @@ public class VpnProfileControlActivity extends AppCompatActivity
 		}
 	}
 
+	/**
+	 * Check if we are currently connected to a VPN connection
+	 *
+	 * @return true if currently connected
+	 */
+	private boolean isConnected()
+	{
+		if (mService == null)
+		{
+			return false;
+		}
+		if (mService.getErrorState() != VpnStateService.ErrorState.NO_ERROR)
+		{	/* allow reconnecting (even to a different profile) without confirmation if there is an error */
+			return false;
+		}
+		return (mService.getState() == State.CONNECTED || mService.getState() == State.CONNECTING);
+	}
+
 	/**
 	 * Start the given VPN profile
 	 *
@@ -207,9 +225,9 @@ public class VpnProfileControlActivity extends AppCompatActivity
 
 		removeFragmentByTag(DIALOG_TAG);
 
-		if (mService != null && (mService.getState() == State.CONNECTED || mService.getState() == State.CONNECTING))
+		if (isConnected())
 		{
-			profileInfo.putBoolean(PROFILE_RECONNECT, mService.getProfile().getId() == profile.getId());
+			profileInfo.putBoolean(PROFILE_RECONNECT, mService.getProfile().getUUID().equals(profile.getUUID()));
 
 			ConfirmationDialog dialog = new ConfirmationDialog();
 			dialog.setArguments(profileInfo);