Ios Crash Fix

[rezq2009]

A crash occurs when playing GIF images on a ios
this issue fixed in version SDWebImage : 5.13.4

SDWebImage/SDWebImage#3273

https://github.com/SDWebImage/SDWebImage/releases/tag/5.13.4
https://github.com/SDWebImage/SDWebImage/releases/tag/5.13.3

[mohitarora777]

@rezq2009 Just wanted to confirm using WebP also cause this crash or just GIF images ?

[fabio-nettis]

@DylanVann, Sorry for pinging you directly, but can this be merged please? Library is currently unusable on iOS.

Information on Error

From what I've read so far the issue is caused by the SDWebImage pod that is currently on version 5.11.1 in the projects podspec. Updating it to the latest version would fix this issue.

- (void) setOnFastImageError: (RCTDirectEventBlock)onFastImageError {
    _onFastImageError = onFastImageError;
    if (self.hasErrored) {
        _onFastImageError(@{}); // <-- Thread 1: EXC_BAD_ACCESS (code=1, address=0x10)
    }
}

Related issues

Following issues are containing references to this issue or related issues caused by the outdated version of SDWebImage.

• App Crash on IOS #943
• Fast Image Stopped working ios #914
• unable to see images react-native-fast-image ^7.0.2 #928
• iOS SDWebImage Crash on SDGraphicsGetImageFromCurrentImageContext #911
• iOS memory leaks when render WebP images #738

[econnerty]

Is there anyway for me to patch this manually? I added your changes, but the project will not build anymore? @rezq2009

[rezq2009]

@econnerty

need to :
pod update
in ios dictionary after apply this patch

OR
remove all react-native-fast-image && make sure is unlinked
then install it again and apply this patch before pod install

[ghost]

@DylanVann Do you have any chance to merge this. This is required to increase the performance of the iOS. We need to use newer versions as below:

  s.dependency 'SDWebImage', '~> 5.15.1'
  s.dependency 'SDWebImageWebPCoder', '~> 0.10.1'

[amerllica]

@rezq2009, During these 6 months, the SDWebImage and SDWebImageWebPCoder got updated and even your PR needs to be updated.

  s.dependency 'SDWebImage', '~> 5.15.4'
  s.dependency 'SDWebImageWebPCoder', '~> 0.10.1'

[rezq2009]

So far I'm using the verified versions on all iOS versions without any crash, I'll check for new versions when I have time

…

Sent from my iPhone On 2 Mar 2023, at 12:11 AM, Amer Lotfi Orimi ***@***.***> wrote:  @rezq2009<https://github.com/rezq2009>, During these 6 months, the SDWebImage and SDWebImageWebPCoder got updated and even your PR needs to be updated. s.dependency 'SDWebImage', '~> 5.15.4' s.dependency 'SDWebImageWebPCoder', '~> 0.10.1' — Reply to this email directly, view it on GitHub<#932 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AYLHNYYAGDVJUJNIZK6XJ73WZ63QRANCNFSM6AAAAAAQU3RX54>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[ghost]

Is it possible to do this version upgrade on my project without merging this PR?

[steve-godlewski]

Seeing a large number of these crashes in our production app, would be great to get this PR merged in. Any update here?

[amerllica]

@ensar-initialcode

I think you need to know patch-package