Avoid ClassCastExceptions in DefaultBitmapAnimationDrawableFactory

Differential Revision: D64261390
facebook · Oct 11, 2024 · cdd2e61 · cdd2e61
1 parent defe02d
commit cdd2e61
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/...ain/java/com/facebook/fresco/animation/factory/DefaultBitmapAnimationDrawableFactory.java b/...ain/java/com/facebook/fresco/animation/factory/DefaultBitmapAnimationDrawableFactory.java
@@ -118,7 +118,11 @@ public boolean supportsImageType(CloseableImage image) {
   }
 
   @Override
+  @Nullable
   public Drawable createDrawable(CloseableImage image) {
+    if (!supportsImageType(closeableImage)) {
+      return null;
+    }
     CloseableAnimatedImage closeable = ((CloseableAnimatedImage) image);
     AnimatedImage animatedImage = closeable.getImage();
     AnimationBackend animationBackend =
@@ -134,8 +138,12 @@ public Drawable createDrawable(CloseableImage image) {
   }
 
   @Override
+  @Nullable
   public Drawable createDrawable(
       Resources resources, CloseableImage closeableImage, ImageOptions imageOptions) {
+    if (!supportsImageType(closeableImage)) {
+      return null;
+    }
     CloseableAnimatedImage closeable = ((CloseableAnimatedImage) closeableImage);
     AnimatedImage animatedImage = closeable.getImage();
     AnimationBackend animationBackend = null;