Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[15.9.0] Crash on iOS with SVG containing feComposite called by SvgXml #2552

Closed
TomCorvus opened this issue Nov 27, 2024 · 5 comments · Fixed by #2557
Closed

[15.9.0] Crash on iOS with SVG containing feComposite called by SvgXml #2552

TomCorvus opened this issue Nov 27, 2024 · 5 comments · Fixed by #2557

Comments

@TomCorvus
Copy link

Description

Hello,

Since 15.9.0, I have a crash on IOS with a SVG called with SvgXml.
I tried some manipulation like remove feComposite tags and the app doesn't crash.
No problem on Android (debug and release)

Thank you

Steps to reproduce

  1. Clone
  2. expo run:ios

Snack or a link to a repository

https://github.com/TomCorvus/RNSVG

SVG version

15.9.0

React Native version

0.76.3

Platforms

iOS

JavaScript runtime

Hermes

Workflow

React Native

Architecture

None

Build type

Debug app & production bundle

Device

iOS simulator

Device model

No response

Acknowledgements

Yes

@jakex7
Copy link
Member

jakex7 commented Nov 28, 2024

Hey @TomCorvus,
Thank you for reporting this, can confirm that there is an issue with FeComposite when not all arguments (k1, k2, k3, k4) are provided. I will release the fix as soon as possible. In the meantime, as a workaround, you can add k1="0" k4="0" to each FeComposite to ensure all values are passed (0 is the default).

@TomCorvus
Copy link
Author

Thank you @jakex7!

@fbp93
Copy link

fbp93 commented Nov 29, 2024

@jakex7 I'm actually experiencing crashes with many SVGXML (not just feComposite) since the 15.0.0.
For example this is working fine with online editor but it's crashing with the latest release. At the same time I cannot use the 14.1.0 because it's crashing with react 0.75 on android for other reasons.
mobile_discount_bf (1)

@fbp93
Copy link

fbp93 commented Nov 29, 2024

@jakex7 LMK if you prefer that I open a new issue, but I thought it could be related to this

@jakex7
Copy link
Member

jakex7 commented Dec 2, 2024

Hey @fbp93, there's no need to create a new issue since it's related to the same crash and requires the same fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants