diff --git a/src/frontends/android/app/src/main/java/org/strongswan/android/data/VpnProfileSqlDataSource.java b/src/frontends/android/app/src/main/java/org/strongswan/android/data/VpnProfileSqlDataSource.java index cee26a457..63d2deabe 100644 --- a/src/frontends/android/app/src/main/java/org/strongswan/android/data/VpnProfileSqlDataSource.java +++ b/src/frontends/android/app/src/main/java/org/strongswan/android/data/VpnProfileSqlDataSource.java @@ -54,6 +54,16 @@ public class VpnProfileSqlDataSource implements VpnProfileDataSource return this; } + /** + * A profile keeps the data source it was loaded from, which may be closed + * by the time the profile is updated or deleted, so reopen on demand. + */ + private SQLiteDatabase database() + { + open(); + return mDatabase; + } + @Override public void close() { @@ -67,7 +77,7 @@ public class VpnProfileSqlDataSource implements VpnProfileDataSource public VpnProfile insertProfile(VpnProfile profile) { ContentValues values = ContentValuesFromVpnProfile(profile); - long insertId = mDatabase.insert(DatabaseHelper.TABLE_VPN_PROFILE.Name, null, values); + long insertId = database().insert(DatabaseHelper.TABLE_VPN_PROFILE.Name, null, values); if (insertId == -1) { return null; @@ -82,14 +92,14 @@ public class VpnProfileSqlDataSource implements VpnProfileDataSource { final UUID uuid = profile.getUUID(); ContentValues values = ContentValuesFromVpnProfile(profile); - return mDatabase.update(DatabaseHelper.TABLE_VPN_PROFILE.Name, values, KEY_UUID + " = ?", new String[]{uuid.toString()}) > 0; + return database().update(DatabaseHelper.TABLE_VPN_PROFILE.Name, values, KEY_UUID + " = ?", new String[]{uuid.toString()}) > 0; } @Override public boolean deleteVpnProfile(VpnProfile profile) { final UUID uuid = profile.getUUID(); - return mDatabase.delete(DatabaseHelper.TABLE_VPN_PROFILE.Name, KEY_UUID + " = ?", new String[]{uuid.toString()}) > 0; + return database().delete(DatabaseHelper.TABLE_VPN_PROFILE.Name, KEY_UUID + " = ?", new String[]{uuid.toString()}) > 0; } @Override