Wrap lazy composition tasks with a null check (#1711)
I can't figure out why this would happen but if it is happening, it's better to potentially have an undesirable animation missing some lazy composition tasks rather than crash.
Fixes #1702
diff --git a/lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java b/lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java
index a274ab3..d846b69 100644
--- a/lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java
+++ b/lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java
@@ -226,7 +226,11 @@
Iterator<LazyCompositionTask> it = new ArrayList<>(lazyCompositionTasks).iterator();
while (it.hasNext()) {
LazyCompositionTask t = it.next();
- t.run(composition);
+ // The task should never be null but it appears to happen in rare cases. Maybe it's an oem-specific or ART bug.
+ // https://github.com/airbnb/lottie-android/issues/1702
+ if (t != null) {
+ t.run(composition);
+ }
it.remove();
}
lazyCompositionTasks.clear();
