-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Crash on app reload on Android in reanimated::NativeProxy::~NativeProxy
#3351
Comments
Hey! 👋 It looks like you've omitted a few important sections from the issue template. Please complete Snack or minimal code example section. |
Hey! 👋 The issue doesn't seem to contain a minimal reproduction. Could you provide a snippet of code, a snack or a link to a GitHub repository that reproduces the problem? |
same crash occurred with RNReanimated v2.9.1 (with RN v0.69.x, hermes) also |
@janicduplessis What device are you using to reproduce this issue? We are seeing the same thing when installing codepush updates but only on certain Android devices. |
This happens on a Pixel 4 api 31 emulator |
Thanks, I was able to repro using a real device earlier today though. Worked around it by downgrading to 2.2.4, since we don't use layout animations this will work for now. I'm trying to validate the codepush stuff now. |
Same on android in release
|
Oh hello, can confim, I started to experience the same crash but on app launch. It's a trap.
Sorry for complaining but to be honest, we are at our company having so many issues with v2, it's one of the most unstable libraries I've ever seen. Litterally every Github issue here occurs in our app, we would remove it altogether if we had a latest react-navigation version which no longer depends on it... Interestingly though the package maintainers admit it's unstable but they dropped support for v1. 🤷♂️ |
Hello @janicduplessis! Thanks for submitting the issue. I was able to reproduce this issue on Samsung SM-A105FN with Android 9, API 28 by clicking Android logs
|
reanimated::NativeProxy::~NativeProxy
Description
When reloading an app in dev mode on Android it causes the following native crash:
Seems to be caused by https://github.com/software-mansion/react-native-reanimated/blob/main/android/src/main/cpp/NativeProxy.cpp#L70. Removing those line prevents the crash.
Not sure if there is anything special needed to repro the crash, note this is using hermes.
Expected behavior
No crash
Actual behavior & steps to reproduce
Reload the app in dev mode.
Snack or minimal code example
N/A
Package versions
Affected platforms
The text was updated successfully, but these errors were encountered: