Bug 1251253 - prevent null pointer dereference of |aContext| in CacheStorageService::DoomStorageEntries. r=mayhemer

MozReview-Commit-ID: BmXXg4eW25n

--HG--
extra : rebase_source : 35ca2824ebdd618fef7b6de4858de8214571f466

netwerk/cache2/CacheStorageService.cpp

@@ -1799,15 +1799,19 @@ CacheStorageService::DoomStorageEntries(nsCSubstring const& aContextKey,
   {
     mozilla::MutexAutoLock lock(mForcedValidEntriesLock);
 
-    for (auto iter = mForcedValidEntries.Iter(); !iter.Done(); iter.Next()) {
+    if (aContext) {
-      bool matches;
+      for (auto iter = mForcedValidEntries.Iter(); !iter.Done(); iter.Next()) {
-      DebugOnly<nsresult> rv = CacheFileUtils::KeyMatchesLoadContextInfo(
+        bool matches;
-        iter.Key(), aContext, &matches);
+        DebugOnly<nsresult> rv = CacheFileUtils::KeyMatchesLoadContextInfo(
-      MOZ_ASSERT(NS_SUCCEEDED(rv));
+          iter.Key(), aContext, &matches);
+        MOZ_ASSERT(NS_SUCCEEDED(rv));
 
-      if (matches) {
+        if (matches) {
-        iter.Remove();
+          iter.Remove();
+        }
       }
+    } else {
+      mForcedValidEntries.Clear();
     }
   }