Bug 1875891 - Adding MalformedInputException to catch list of handleP…

…lacesExceptions.
mozilla-mobile · Jan 24, 2024 · d10752f · d10752f
1 parent 3872fae
commit d10752f
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/...owser/storage-sync/src/main/java/mozilla/components/browser/storage/sync/PlacesStorage.kt b/...owser/storage-sync/src/main/java/mozilla/components/browser/storage/sync/PlacesStorage.kt
@@ -22,6 +22,7 @@ import mozilla.components.concept.sync.SyncableStore
 import mozilla.components.support.base.log.logger.Logger
 import mozilla.components.support.base.utils.NamedThreadFactory
 import mozilla.components.support.utils.logElapsedTime
+import java.nio.charset.MalformedInputException
 import java.util.concurrent.Executors
 
 /**
@@ -145,6 +146,9 @@ abstract class PlacesStorage(
     protected inline fun handlePlacesExceptions(operation: String, block: () -> Unit) {
         try {
             block()
+        } catch (e: MalformedInputException) {
+            crashReporter?.submitCaughtException(e)
+            logger.debug("Ignoring invalid invalid non utf-8 character when running $operation", e)
         } catch (e: PlacesApiException.OperationInterrupted) {
             logger.debug("Ignoring expected OperationInterrupted exception when running $operation", e)
         } catch (e: PlacesApiException.UrlParseFailed) {