From 6b3fc6183eb04c0599f4c8fb282fb697d2430194 Mon Sep 17 00:00:00 2001 From: arrase Date: Thu, 17 Nov 2016 19:34:08 +0100 Subject: [PATCH] changes for new securety layer --- .../org/torproject/android/OrbotMainActivity.java | 15 ++------------- .../android/ui/hs/database/HSDatabase.java | 1 - .../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 f5f62276..f887291c 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 07e5c1be..cafa7edd 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 f5237e3d..fb62cb7c 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 dd831488..7379f455 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" /> - - Onion Port Name Invalid Port - Allow managed backups