Bug 618341: Rethrow exceptions we aren't supposed to handle. r=philiKON
authorRichard Newman <rnewman@mozilla.com>
Fri, 10 Dec 2010 15:43:12 -0800
changeset 59092 32a264aa3ff832d672295b48f5c2ab4c5a879dd5
parent 59070 16f9d94814287beb0a9c5ded17c96897f71e6604
child 59093 df5562e2bed7950a71298b55c80f06e84c2585df
push id1
push usershaver@mozilla.com
push dateTue, 04 Jan 2011 17:58:04 +0000
reviewersphiliKON
bugs618341
Bug 618341: Rethrow exceptions we aren't supposed to handle. r=philiKON
services/sync/modules/engines.js
--- a/services/sync/modules/engines.js
+++ b/services/sync/modules/engines.js
@@ -485,16 +485,22 @@ SyncEngine.prototype = {
           this._tracker.ignoreAll = true;
           this._store.applyIncoming(item);
         } else {
           count.reconciled++;
           this._log.trace("Skipping reconciled incoming item " + item.id);
         }
       }
       catch(ex) {
+
+        if (!Utils.isHMACMismatch(ex)) {
+          // Rethrow anything we shouldn't handle.
+          throw ex;
+        }
+
         this._log.warn("Error processing record: " + Utils.exceptionStr(ex));
 
         // Upload a new record to replace the bad one if we have it
         if (this._store.itemExists(item.id))
           this._modified[item.id] = 0;
       }
       this._tracker.ignoreAll = false;
       Sync.sleep(0);