Framework to Local Blacklist Synchronization Fix

Added db.beginTransaction()/db.endTransaction() before and after
a sequence of db accesses that have to be performed atomically

Change-Id: I912230afe8a7f58841306ec8ad64819ba00d3405

1 files changed, 12 insertions, 6 deletions

diff --git a/src/com/android/messaging/util/BlacklistSync.java b/src/com/android/messaging/util/BlacklistSync.java
index 9cc611e..44253c4 100644
--- a/src/com/android/messaging/util/BlacklistSync.java
+++ b/src/com/android/messaging/util/BlacklistSync.java
@@ -72,12 +72,18 @@ public class BlacklistSync extends AsyncTask<Void, Void, Void> {
                     // there was no phone number in the local participants database that was
                     // blacklisted in the framework blacklist database, create a new participant
                     // and insert him into the local participants database
-                    ParticipantData participant = ParticipantData
-                            .getFromRawPhoneBySystemLocale(number);
-                    BugleDatabaseOperations.getOrCreateParticipantInTransaction(db,
-                            participant);
-                    BugleDatabaseOperations.updateDestination(db, number,
-                            isBlocked, false);
+                    db.beginTransaction();
+                    try {
+                        ParticipantData participant = ParticipantData
+                                .getFromRawPhoneBySystemLocale(number);
+                        BugleDatabaseOperations.getOrCreateParticipantInTransaction(db,
+                                participant);
+                        BugleDatabaseOperations.updateDestination(db, number,
+                                isBlocked, false);
+                        db.setTransactionSuccessful();
+                    } finally {
+                        db.endTransaction();
+                    }
                 }
             }
         }