modifications to open helper for RO/RW db management

Author: n8fr8

src/net/sqlcipher/database/SQLiteOpenHelper.java

@@ -39,7 +39,7 @@ public abstract class SQLiteOpenHelper {
     private final CursorFactory mFactory;
     private final int mNewVersion;
 
-    private SQLiteDatabase mDatabase = null;
+    private SQLiteDatabase mDatabaseRO, mDatabaseRW = null;
     private boolean mIsInitializing = false;
 
     /**
@@ -75,8 +75,8 @@ public SQLiteOpenHelper(Context context, String name, CursorFactory factory, int
      * @return a read/write database object valid until {@link #close} is called
      */
     public synchronized SQLiteDatabase getWritableDatabase(String password) {
-        if (mDatabase != null && mDatabase.isOpen() && !mDatabase.isReadOnly()) {
-            return mDatabase;  // The database is already open for business
+        if (mDatabaseRW != null && mDatabaseRW.isOpen()) {
+            return mDatabaseRW;  // The database is already open for business
         }
 
         if (mIsInitializing) {
@@ -91,7 +91,7 @@ public synchronized SQLiteDatabase getWritableDatabase(String password) {
 
         boolean success = false;
         SQLiteDatabase db = null;
-        if (mDatabase != null) mDatabase.lock();
+        if (mDatabaseRW != null) mDatabaseRW.lock();
         try {
             mIsInitializing = true;
             if (mName == null) {
@@ -135,13 +135,13 @@ public synchronized SQLiteDatabase getWritableDatabase(String password) {
         } finally {
             mIsInitializing = false;
             if (success) {
-                if (mDatabase != null) {
-                    try { mDatabase.close(); } catch (Exception e) { }
-                    mDatabase.unlock();
+                if (mDatabaseRW != null) {
+                    try { mDatabaseRW.close(); } catch (Exception e) { }
+                    mDatabaseRW.unlock();
                 }
-                mDatabase = db;
+                mDatabaseRW = db;
             } else {
-                if (mDatabase != null) mDatabase.unlock();
+                if (mDatabaseRW != null) mDatabaseRW.unlock();
                 if (db != null) db.close();
             }
         }
@@ -161,21 +161,14 @@ public synchronized SQLiteDatabase getWritableDatabase(String password) {
      *     or {@link #close} is called.
      */
     public synchronized SQLiteDatabase getReadableDatabase(String password) {
-        if (mDatabase != null && mDatabase.isOpen()) {
-            return mDatabase;  // The database is already open for business
+        if (mDatabaseRO != null && mDatabaseRO.isOpen()) {        	
+        		return mDatabaseRO;  // The database is already open for business        	
         }
 
         if (mIsInitializing) {
             throw new IllegalStateException("getReadableDatabase called recursively");
         }
 
-        try {
-            return getWritableDatabase(password);
-        } catch (SQLiteException e) {
-            if (mName == null) throw e;  // Can't open a temp database read-only!
-            Log.e(TAG, "Couldn't open " + mName + " for writing (will try read-only):", e);
-        }
-
         SQLiteDatabase db = null;
         try {
             mIsInitializing = true;
@@ -188,11 +181,11 @@ public synchronized SQLiteDatabase getReadableDatabase(String password) {
 
             onOpen(db);
             Log.w(TAG, "Opened " + mName + " in read-only mode");
-            mDatabase = db;
-            return mDatabase;
+            mDatabaseRO = db;
+            return mDatabaseRO;
         } finally {
             mIsInitializing = false;
-            if (db != null && db != mDatabase) db.close();
+            if (db != null && db != mDatabaseRO) db.close();
         }
     }
 
@@ -202,9 +195,14 @@ public synchronized SQLiteDatabase getReadableDatabase(String password) {
     public synchronized void close() {
         if (mIsInitializing) throw new IllegalStateException("Closed during initialization");
 
-        if (mDatabase != null && mDatabase.isOpen()) {
-            mDatabase.close();
-            mDatabase = null;
+        if (mDatabaseRO != null && mDatabaseRO.isOpen()) {
+        	mDatabaseRO.close();
+        	mDatabaseRO = null;
+        }
+        
+        if (mDatabaseRW != null && mDatabaseRW.isOpen()) {
+        	mDatabaseRW.close();
+        	mDatabaseRW = null;
         }
     }