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

App freeze on iOS with "arithmetic overflow" error in sentry swift #1872

Closed
zvikarp opened this issue Feb 7, 2024 · 5 comments · Fixed by #1874
Closed

App freeze on iOS with "arithmetic overflow" error in sentry swift #1872

zvikarp opened this issue Feb 7, 2024 · 5 comments · Fixed by #1874
Assignees

Comments

@zvikarp
Copy link

zvikarp commented Feb 7, 2024

Platform

Dart

Obfuscation

Disabled

Debug Info

Disabled

Doctor

[✓] Flutter (Channel stable, 3.16.2, on macOS 14.2.1 23C71 darwin-arm64, locale
en-IL)
• Flutter version 3.16.2 on channel stable at
/Users/zvikarp/.asdf/installs/flutter/3.16.2-stable
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 9e1c857886 (10 weeks ago), 2023-11-30 11:51:18 -0600
• Engine revision cf7a9d0800
• Dart version 3.2.2
• DevTools version 2.28.3

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
• Android SDK at /Users/zvikarp/Library/Android/sdk
• Platform android-34, build-tools 32.0.0
• Java binary at: /Applications/Android
Studio.app/Contents/jbr/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build
17.0.7+0-17.0.7b1000.6-10550314)
• All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 15C500b
• CocoaPods version 1.15.0

[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build
17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.83.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.82.0

[✓] VS Code (version 1.86.0-insider)
• VS Code at /Applications/Visual Studio Code - Insiders.app/Contents
• Flutter extension can be installed from:
🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)


[✓] Network resources
• All expected network resources are available.

• No issues found!

Version

7.14.0

Steps to Reproduce

I'm not sure, this issue started happening recently, it is hard for me to understand why it is happening, sometimes it works and sometimes I get an error in xcode:

recording_2024-02-07_9 -29-18

To get to this error I run the app and at some stage it just freezes, hot reload stops working and the error above appears in xcode. seems like that the result of the selected line is negative but we are using unit here and this is the cause of the error.

Expected Result

Not to freeze

Actual Result

App freezes

Are you willing to submit a PR?

None

@buenaflor
Copy link
Contributor

buenaflor commented Feb 7, 2024

Hi, do you know if you can reproduce that consistently?

@zvikarp
Copy link
Author

zvikarp commented Feb 11, 2024

No. It often happens in debug mode when navigating from a loading screen to a screen with a lot of data. It happened to me on testflight only the first time I installed it from it. On testflight the app just crashes.

@denrase
Copy link
Collaborator

denrase commented Feb 12, 2024

Looked into this, avoiding this error should be a simple thing, but it does not solve the underlying issue. Judging from a debug session, it looks like begin/end is not called symmetrically when using sentry navigator observer. I will further investigate and see if i can fix the root issue.

Bildschirmfoto 2024-02-12 um 11 25 00

/cc @buenaflor

@zvikarp
Copy link
Author

zvikarp commented Feb 12, 2024

Thank you @denrase I really appreciate it 🙏

@buenaflor
Copy link
Contributor

the fix is included in 7.16.1

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

Successfully merging a pull request may close this issue.

3 participants