diff options
| author | Thialfihar <thialfihar@gmail.com> | 2010-05-27 20:27:21 +0000 |
|---|---|---|
| committer | Thialfihar <thialfihar@gmail.com> | 2010-05-27 20:27:21 +0000 |
| commit | b42f02ba922e26eedfbd37a3f1d15b66ed14e9b9 (patch) | |
| tree | a8a8177936c3a9678f987aaa4da09463a750a29a /src/org/thialfihar/android/apg/provider/Database.java | |
| parent | 570b7a6d8eaeeede6421070578485444e8b632ca (diff) | |
| download | open-keychain-b42f02ba922e26eedfbd37a3f1d15b66ed14e9b9.tar.gz open-keychain-b42f02ba922e26eedfbd37a3f1d15b66ed14e9b9.tar.bz2 open-keychain-b42f02ba922e26eedfbd37a3f1d15b66ed14e9b9.zip | |
some coding practice and potential bug fixes
Diffstat (limited to 'src/org/thialfihar/android/apg/provider/Database.java')
| -rw-r--r-- | src/org/thialfihar/android/apg/provider/Database.java | 37 |
1 files changed, 29 insertions, 8 deletions
diff --git a/src/org/thialfihar/android/apg/provider/Database.java b/src/org/thialfihar/android/apg/provider/Database.java index 55736e8b4..6fee3888b 100644 --- a/src/org/thialfihar/android/apg/provider/Database.java +++ b/src/org/thialfihar/android/apg/provider/Database.java @@ -200,7 +200,10 @@ public class Database extends SQLiteOpenHelper { }
} while (cursor.moveToNext());
}
- cursor.close();
+
+ if (cursor != null) {
+ cursor.close();
+ }
break;
}
@@ -427,7 +430,10 @@ public class Database extends SQLiteOpenHelper { rowId = mDb.insert(KeyRings.TABLE_NAME, KeyRings.WHO_ID, values);
mStatus = Id.return_value.ok;
}
- c.close();
+
+ if (c != null) {
+ c.close();
+ }
return rowId;
}
@@ -465,11 +471,14 @@ public class Database extends SQLiteOpenHelper { if (c != null && c.moveToFirst()) {
rowId = c.getLong(0);
mDb.update(UserIds.TABLE_NAME, values,
- UserIds._ID + " = ?", new String[] { "" + rowId });
+ UserIds._ID + " = ?", new String[] { "" + rowId });
} else {
rowId = mDb.insert(UserIds.TABLE_NAME, UserIds.USER_ID, values);
}
- c.close();
+
+ if (c != null) {
+ c.close();
+ }
return rowId;
}
@@ -500,7 +509,10 @@ public class Database extends SQLiteOpenHelper { }
}
}
- c.close();
+
+ if (c != null) {
+ c.close();
+ }
return keyRing;
}
@@ -522,7 +534,10 @@ public class Database extends SQLiteOpenHelper { if (c != null && c.moveToFirst()) {
data = c.getBlob(0);
}
- c.close();
+
+ if (c != null) {
+ c.close();
+ }
return data;
}
@@ -539,7 +554,10 @@ public class Database extends SQLiteOpenHelper { if (c != null && c.moveToFirst()) {
data = c.getBlob(0);
}
- c.close();
+
+ if (c != null) {
+ c.close();
+ }
return data;
}
@@ -561,7 +579,10 @@ public class Database extends SQLiteOpenHelper { deleteKey(keyId);
} while (c.moveToNext());
}
- c.close();
+
+ if (c != null) {
+ c.close();
+ }
mDb.setTransactionSuccessful();
mDb.endTransaction();
|