[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [orbot/master] changes for new securety layer



commit 6b3fc6183eb04c0599f4c8fb282fb697d2430194
Author: arrase <arrase@xxxxxxxxx>
Date:   Thu Nov 17 19:34:08 2016 +0100

    changes for new securety layer
---
 .../java/org/torproject/android/OrbotMainActivity.java    | 15 ++-------------
 .../org/torproject/android/ui/hs/database/HSDatabase.java |  1 -
 .../torproject/android/ui/hs/dialogs/HSDataDialog.java    |  7 ++-----
 app/src/main/res/layout/dialog_hs_data.xml                |  7 -------
 app/src/main/res/values/strings.xml                       |  1 -
 5 files changed, 4 insertions(+), 27 deletions(-)

diff --git a/app/src/main/java/org/torproject/android/OrbotMainActivity.java b/app/src/main/java/org/torproject/android/OrbotMainActivity.java
index f5f6227..f887291 100644
--- a/app/src/main/java/org/torproject/android/OrbotMainActivity.java
+++ b/app/src/main/java/org/torproject/android/OrbotMainActivity.java
@@ -572,7 +572,7 @@ public class OrbotMainActivity extends AppCompatActivity
             stopVpnService();
     }
 	
-	private void enableHiddenServicePort (String hsName, int hsPort, int hsRemotePort, boolean allowBackups) throws RemoteException, InterruptedException
+	private void enableHiddenServicePort (String hsName, int hsPort, int hsRemotePort) throws RemoteException, InterruptedException
 	{
 		String onionHostname = null;
 
@@ -593,15 +593,6 @@ public class OrbotMainActivity extends AppCompatActivity
 		Cursor row = cr.query(HSContentProvider.CONTENT_URI, mProjection, "port=" + mHsPort, null, null);
 
 		if(row == null) {
-			/*
-			 * For security reasons:
-			 *
-			 * Allow managed backups option can only be set to a arbitrary true or false value when a service is created
-			 * Subsequently an application can only change this value to false via Intent service
-			 * Modifying this value to true again can only be performed by the user through the Orbot interface
-			 *
-			 */
-			fields.put("allow_managed_backups", allowBackups);
 			cr.insert(HSContentProvider.CONTENT_URI, fields);
 		} else {
 			onionHostname = row.getString(row.getColumnIndex(HSContentProvider.HiddenService.DOMAIN));
@@ -674,7 +665,6 @@ public class OrbotMainActivity extends AppCompatActivity
         	final int hiddenServicePort = getIntent().getIntExtra("hs_port", -1);
         	final int hiddenServiceRemotePort = getIntent().getIntExtra("hs_onion_port", -1);
         	final String  hiddenServiceName = getIntent().getStringExtra("hs_name");
-        	final boolean hiddenServiceAllowBackups = getIntent().getBooleanExtra("hs_allow_namaged_backup",false);
 
 			DialogInterface.OnClickListener dialogClickListener = new DialogInterface.OnClickListener() {
 			    
@@ -684,8 +674,7 @@ public class OrbotMainActivity extends AppCompatActivity
 			            
 						try {
 							enableHiddenServicePort (
-									hiddenServiceName, hiddenServicePort,
-									hiddenServiceRemotePort, hiddenServiceAllowBackups
+									hiddenServiceName, hiddenServicePort, hiddenServiceRemotePort
 							);
 						} catch (RemoteException e) {
 							// TODO Auto-generated catch block
diff --git a/app/src/main/java/org/torproject/android/ui/hs/database/HSDatabase.java b/app/src/main/java/org/torproject/android/ui/hs/database/HSDatabase.java
index 07e5c1b..cafa7ed 100644
--- a/app/src/main/java/org/torproject/android/ui/hs/database/HSDatabase.java
+++ b/app/src/main/java/org/torproject/android/ui/hs/database/HSDatabase.java
@@ -16,7 +16,6 @@ public class HSDatabase extends SQLiteOpenHelper {
                     "name TEXT, " +
                     "domain TEXT, " +
                     "onion_port INTEGER, " +
-                    "allow_managed_backups INTEGER DEFAULT 0, " +
                     "port INTEGER);";
 
     public HSDatabase(Context context) {
diff --git a/app/src/main/java/org/torproject/android/ui/hs/dialogs/HSDataDialog.java b/app/src/main/java/org/torproject/android/ui/hs/dialogs/HSDataDialog.java
index f5237e3..fb62cb7 100644
--- a/app/src/main/java/org/torproject/android/ui/hs/dialogs/HSDataDialog.java
+++ b/app/src/main/java/org/torproject/android/ui/hs/dialogs/HSDataDialog.java
@@ -45,10 +45,8 @@ public class HSDataDialog extends DialogFragment {
                         ((EditText) dialog_view.findViewById(R.id.hsOnionPort)).getText().toString()
                 );
 
-                Boolean allowBackups = ((CheckBox) dialog_view.findViewById(R.id.allow_managed_backup)).isEnabled();
-
                 if (checkInput(localPort,onionPort)) {
-                    saveData(serverName, localPort,onionPort,allowBackups);
+                    saveData(serverName, localPort,onionPort);
                     serverDataDialog.dismiss();
                 }
             }
@@ -80,12 +78,11 @@ public class HSDataDialog extends DialogFragment {
         return is_ok;
     }
 
-    private void saveData(String name, Integer local, Integer remote, Boolean allowBackups) {
+    private void saveData(String name, Integer local, Integer remote) {
         ContentValues fields = new ContentValues();
         fields.put("name", name);
         fields.put("port", local);
         fields.put("onion_port", remote);
-        fields.put("allow_managed_backups", allowBackups);
 
         ContentResolver cr = getContext().getContentResolver();
 
diff --git a/app/src/main/res/layout/dialog_hs_data.xml b/app/src/main/res/layout/dialog_hs_data.xml
index dd83148..7379f45 100644
--- a/app/src/main/res/layout/dialog_hs_data.xml
+++ b/app/src/main/res/layout/dialog_hs_data.xml
@@ -53,13 +53,6 @@
         android:id="@+id/hsOnionPort"
         android:inputType="number" />
 
-    <CheckBox
-        android:text="@string/allow_managed_backup"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:id="@+id/allow_managed_backup"
-        android:layout_weight="1" />
-
     <LinearLayout
         android:orientation="horizontal"
         android:layout_width="match_parent"
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 7f534fe..ea01787 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -339,5 +339,4 @@
     <string name="onion_port">Onion Port</string>
     <string name="name">Name</string>
     <string name="invalid_port">Invalid Port</string>
-    <string name="allow_managed_backup">Allow managed backups</string>
 </resources>



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits