lottie crashed in AnimationContainer.init() on iOS9 device when used Xcode11.4 #1155

EmilLiu · 2020-03-25T08:55:55Z

When I compiled and ran with the latest version of Xcode 11.4, there was a crash anywhere I used Lottie on iOS 9 devices

Check these before submitting:

[✔️] The issue doesn't involve an Unsupported Feature
[✔️] This issue isn't related to another open issue

This issue is a:

[] Non-Crashing Bug (Visual or otherwise)
[✔️] Crashing Bug
[] Feature Request
[] Regression (Something that once worked, but doesn't work anymore)

Which Version of Lottie are you using?

Lottie 3.1.6

What Platform are you on?

[] MacOS
[✔️] iOS

What Language are you in?

[✔️] Swift
[] Objective-C

Expected Behavior

Actual Behavior

Code Example

Animation JSON

robaggio · 2020-03-30T15:28:36Z

Same for me, only iOS 9 devices will crash, the same code on Xcode 11.3 is OK.

Crashed: com.apple.main-thread
0 libobjc.A.dylib 0x1976e8160 objc_release + 16
1 Lottie 0x101d7cdac specialized AnimationContainer.init(animation:imageProvider:textProvider:) + 608 (:608)
2 Lottie 0x101d84358 AnimationView.makeAnimationLayer() + 268 (AnimationContainer.swift:268)
3 Lottie 0x101d83de4 AnimationView.animation.setter + 76 (:76)

robaggio · 2020-03-30T16:44:00Z

The example project in the repo can reproduce the crash, with Xcode 11.4 on an iOS 9 device. Change the deployment target to 9.0 first.

frank-du · 2020-04-01T11:45:49Z

Same for me, only iOS 9 devices will crash, the same code on Xcode 11.3 is OK.

Crashed: com.apple.main-thread
0 libobjc.A.dylib 0x1976e8160 objc_release + 16
1 Lottie 0x101d7cdac specialized AnimationContainer.init(animation:imageProvider:textProvider:) + 608 (:608)
2 Lottie 0x101d84358 AnimationView.makeAnimationLayer() + 268 (AnimationContainer.swift:268)
3 Lottie 0x101d83de4 AnimationView.animation.setter + 76 (:76)

Same for me

hakonk · 2020-04-01T14:11:54Z

Can confirm this problem on iOS 9 with Xcode 11.4 as well. Enabling Zombie Objects in Xcode 11.4 results in the following output before crashing the application: *** -[Lottie.AnimationContainer release]: message sent to deallocated instance 0x<address>

brunsman · 2020-04-01T15:08:31Z

Same problem here. I blame the changed embedding of frameworks in Xcode 11.4. Downgraded to Xcode 11.3 for now and waiting for Apple to fix the embedding of frameworks.

hakonk · 2020-04-02T07:35:40Z

@brunsman Do you have more details on what the change in embedding of frameworks entails?

brunsman · 2020-04-15T06:55:45Z

After digging into the issue more, I can now isolate the issue to something else. Will post the same here for reference, see https://forums.developer.apple.com/thread/131551

The code seems related to Xcode 11.4, ios 9 devices, and custom CALayers with properties.

To see this problem, Create an extension of an CALayer:

class CustomLayer: CALayer {  
   var someProperty: CGFloat = 0  
}

Now programmatically create such a layer, e.g. in a viewDidLoad

let customLayer = CustomLayer()

Compile and run this on a iOS 9 device (my tests were with an iPhone 4S with iOS 9.3.6). The device crashes when initializing the layer. It is deallocated BEFORE it can actually set the property on the customLayer. Enable zombies for a more detailed stack trace.

Since lottie uses custom CALayers with properties, it now crashes on iOS 9 devices when using Xcode 11.4. Hoping for a fix or workaround from Apple. Filed a bug report about it.

hakonk · 2020-04-15T07:15:48Z

Great work @brunsman ! Looking forward to hearing the about the response.

hakonk · 2020-04-15T09:45:23Z

@brunsman I tried to circumvent the initializer of CALayer in Swift by making a simple category for CALayer in Objective-C:

CALayer+Init.h

#import <QuartzCore/QuartzCore.h>

NS_ASSUME_NONNULL_BEGIN

@interface CALayer (Init)
+(instancetype)newInstance;
@end

NS_ASSUME_NONNULL_END

CALayer+Init.m

#import "CALayer+Init.h"

@implementation CALayer (Init)
+(instancetype)newInstance {
    return [[CALayer alloc] init];
}
@end

Making use of the class method in this category seems to work.

class CustomLayer: CALayer {
    var someProperty: CGFloat = 0
}

class ViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()
        let customLayer = CustomLayer.newInstance() // this seems to work
        let customLayer2 = CustomLayer() // <- this still crashes
        view.layer.addSublayer(customLayer)
        view.layer.addSublayer(customLayer)
    }
}

Edit:

Unfortunately, trying to access the property on CustomLayer after making use of the class method in the category mentioned above, I get a runtime error: "Thread 1: EXC_BAD_ACCESS (code=2, address=0x.......)"

Interestingly, changing the property to be a let instead of var, does not lead to a crash. I.e.:

class CustomLayer: CALayer {
    let someProperty: CGFloat = 0
}

class ViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()
        let customLayer = CustomLayer()
        view.layer.addSublayer(customLayer)
    }
}

brunsman · 2020-04-15T10:10:57Z

@hakonk If you find a workaround, that would be great! So far, no dice.

Edit: XCode 11.4.1 also does not fix this issue. Still waiting for a response from Apple.

hakonk · 2020-04-16T09:48:53Z

@brunsman The approach above with the Objective-C category is flawed for sure. My colleagues have pointed me in the direction of reading up on CALayer and the magic that happens under the hood, so I'll do that when I have time.

I attempted to make use of Objective-C's runtime capabilities (associated objects) to implement a property on a subclass of CALayer:

class CustomLayer: CALayer {
    private static var somePropertyHandle = 0
    var someProperty: CGFloat {
        get {
            guard let val = objc_getAssociatedObject(
                self,
                &CustomLayer.somePropertyHandle
            ) as? CGFloat else { fatalError("No obj found") }
            return val
        }
        set {
            objc_setAssociatedObject(
                self,
                &CustomLayer.somePropertyHandle,
                newValue,
                objc_AssociationPolicy.OBJC_ASSOCIATION_RETAIN_NONATOMIC
            )
        }
    }
}

class ViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()
        let customLayer = CustomLayer()
        // needs to be assigned, or else the computed property logic will crash
        customLayer.someProperty = 123
        print(customLayer.someProperty)
        view.layer.addSublayer(customLayer)
    }
}

While it works, it's a cumbersome and somewhat hacky approach for just implementing a simple property.

brunsman · 2020-05-07T11:54:56Z

Reported the bug to apple on the 15th of april. Got an update from Apple:

Custom CALayer with properties crashes on iOS 9 devices
15 april 2020, 8:36 AM — FB7664233
Recent Similar Reports: Less than 10
Resolution: Potential fix identified - For a future OS update

Hoping for this best and that the new xCode will fix the problem...

falcon283 · 2020-05-18T23:21:23Z

XCode 11.5 GM is out and the release notes state:

Fixed an issue with incorrect code generation when targeting armv7 devices. (61901594)

Is it the bug we are hunting?
Anyone have chance to test it?

Many thanks

hakonk · 2020-05-19T07:46:05Z

@falcon283 @brunsman
I tried compiling and running with Xcode Version 11.5 (11E608c) and I'm still getting -[Lottie.AnimationContainer release]: message sent to deallocated instance (Zombie objects enabled).

falcon283 · 2020-05-19T11:05:09Z

@falcon283 @brunsman

I tried compiling and running with Xcode Version 11.5 (11E608c) and I'm still getting -[Lottie.AnimationContainer release]: message sent to deallocated instance (Zombie objects enabled).

Thanks a lot for the feedback @hakonk

rizerco · 2020-06-16T15:58:24Z

It looks like the issue is still present in the latest Xcode 11.6 beta (11N700h).

falcon283 · 2020-06-16T18:22:47Z

It looks like the issue is still present in the latest Xcode 11.6 beta (11N700h).

This is so bad :(
Thanks for keeping us posted.

manuhook · 2020-06-27T14:00:11Z

Hii, For the moment the only solution to fix the issue is to rollback to Xcode 11.3 ? Thank you

hakonk · 2020-06-27T14:06:23Z

I have tried downgrading to Lottie 2.5.3, which is written in Objective-C. This works without crashes with Xcode 11.5. As @brunsman pointed out, the issue seems to be related to properties on subclasses of CALayer written in Swift. While I guess it’s not ideal to downgrade, I found rewriting code to work with Lottie 2.5.3 to be quite simple, and definitely worth it to keep up with newer Xcode versions.

ShiQiangMu · 2020-06-30T08:59:08Z

Hii, For the moment the only solution to fix the issue is to rollback to Xcode 11.3 ? Thank you

Yes,It's work.

caioremedio · 2020-08-04T17:37:51Z

Did anyone try testing with Xcode 12 beta?

hakonk · 2020-10-06T10:53:29Z

@caioremedio When testing on iOS 9 with a 32 bit device, I'm still getting the crashes, even with Xcode 12. I've sent them a report with Feedback assistant, but I haven't gotten a reply. @brunsman Did you get a reply from Apple?

brunsman · 2020-10-06T11:21:46Z

@caioremedio When testing on iOS 9 with a 32 bit device, I'm still getting the crashes, even with Xcode 12. I've sent them a report with Feedback assistant, but I haven't gotten a reply. @brunsman Did you get a reply from Apple?

No.... and Apple even reverted the resolution from "Potential fix identified - For a future OS update" to "Open". I doubt they will ever fix this... I stopped using Lottie on iOS < 11.

hakonk · 2020-10-06T11:55:22Z

Wow, that's a bummer, @brunsman! Anyway, it's interesting to know that the issue is still open.

zhiliang729 · 2020-10-20T03:58:07Z

xcode 12.0.1 still crash !!!!!!!!!!

sonnt612 · 2020-10-30T04:09:54Z

How to fix this?

sonnt612 · 2020-10-30T04:10:24Z

xcode 12.1 still crash

robaggio · 2020-10-30T11:32:41Z

Maybe it is time to consider dropping support for iOS9 in your app.

Mervin1024 · 2020-11-25T09:35:25Z

same to me. xcode 12.0

sweet4018 · 2021-02-09T09:17:17Z

Xcode 12.3 still crash. 😥

FunnyerFeng · 2021-02-26T03:29:41Z

Version 12.4 (12D4e)

xperiatrix · 2021-02-27T01:27:09Z

I switched to a gif animation in Kingfisher's AnimatedImageView only on iOS9 instead of lottie's way .

brunsman · 2021-05-20T06:58:39Z

From Apple:

Thanks for your update. We have additional information.

The compiler can shrink lifetimes of objects in case an object is not used at a certain point.
This is an optimization and it can definitely be that this optimization is not performed in an older compiler version, but it is performed in a newer compiler version. This would explain why the crash does not show up when compiling with an older Xcode version.

If this is the case with customLayer, a fix would be to use withExtendedLifetime to ensure that customLayer is not deleted until the end of the specified lifetime.
For details see: https://developer.apple.com/documentation/swift/1541033-withextendedlifetime

I haven't tested whether this works since we stopped using iOS 9. Perhaps it can be used by others.

yaochenfeng · 2021-07-30T08:20:01Z

也许用OC实现CALayer的子类，Swift中使用，iOS10以上就不用这样了

Viveco · 2022-01-26T02:23:57Z

use xib crah， need code xcode 12.3 ios9 lottie 3.2.1

Feng999 · 2022-02-28T03:59:47Z

anything new?
Xcode 13.2.1, the problem is still there

calda · 2022-07-13T16:42:32Z

Closing since Lottie no longer supports iOS 9 (the minimum deployment version is now iOS 11)

robaggio mentioned this issue Jul 10, 2020

Crash in iOS 9 with Xcode 11.4 #1221

Closed

caioremedio mentioned this issue Sep 8, 2020

Lottie framework not worked with IOS 9 and IOS 10 with latest xcode(above xcode 11.3) #1249

Closed

calda closed this as completed Jul 13, 2022

lottie crashed in AnimationContainer.init() on iOS9 device when used Xcode11.4 #1155

lottie crashed in AnimationContainer.init() on iOS9 device when used Xcode11.4 #1155

Comments

EmilLiu commented Mar 25, 2020

Check these before submitting:

This issue is a:

Which Version of Lottie are you using?

What Platform are you on?

What Language are you in?

Expected Behavior

Actual Behavior

Code Example

Animation JSON

robaggio commented Mar 30, 2020

robaggio commented Mar 30, 2020

frank-du commented Apr 1, 2020

hakonk commented Apr 1, 2020

brunsman commented Apr 1, 2020

hakonk commented Apr 2, 2020 • edited

brunsman commented Apr 15, 2020

hakonk commented Apr 15, 2020

hakonk commented Apr 15, 2020 • edited

brunsman commented Apr 15, 2020 • edited

hakonk commented Apr 16, 2020 • edited

brunsman commented May 7, 2020

falcon283 commented May 18, 2020

hakonk commented May 19, 2020

falcon283 commented May 19, 2020

rizerco commented Jun 16, 2020

falcon283 commented Jun 16, 2020

manuhook commented Jun 27, 2020

hakonk commented Jun 27, 2020

ShiQiangMu commented Jun 30, 2020

caioremedio commented Aug 4, 2020 • edited

hakonk commented Oct 6, 2020

brunsman commented Oct 6, 2020

hakonk commented Oct 6, 2020

zhiliang729 commented Oct 20, 2020

sonnt612 commented Oct 30, 2020

sonnt612 commented Oct 30, 2020

robaggio commented Oct 30, 2020

Mervin1024 commented Nov 25, 2020

sweet4018 commented Feb 9, 2021

FunnyerFeng commented Feb 26, 2021

xperiatrix commented Feb 27, 2021

brunsman commented May 20, 2021

yaochenfeng commented Jul 30, 2021

Viveco commented Jan 26, 2022

Feng999 commented Feb 28, 2022

calda commented Jul 13, 2022

hakonk commented Apr 2, 2020 •

edited

hakonk commented Apr 15, 2020 •

edited

brunsman commented Apr 15, 2020 •

edited

hakonk commented Apr 16, 2020 •

edited

caioremedio commented Aug 4, 2020 •

edited