Skip to content
This repository has been archived by the owner on Mar 15, 2024. It is now read-only.

Crash at FlurryiOSUtil.m line 126 #200

Closed
2 of 11 tasks
Isuru-Nanayakkara opened this issue May 22, 2020 · 5 comments
Closed
2 of 11 tasks

Crash at FlurryiOSUtil.m line 126 #200

Isuru-Nanayakkara opened this issue May 22, 2020 · 5 comments
Assignees
Labels

Comments

@Isuru-Nanayakkara
Copy link

Hi,

I have an iOS app with Flurry SDK added. I'm using Flurry for Analytics and Firebase for crash reporting. Today I got the following crash report. This has occurred in a user's device, not mine.

These are all the info I got from the crash report.

Crashed: FlurryNetworkStatusSource
EXC_BAD_ACCESS KERN_INVALID_ADDRESS 0x0000000000000008
+[FlurryiOSUtil getCarrierInstance]

Crashed: FlurryNetworkStatusSource
0 CoreFoundation 0x1b12da444 -[__NSDictionaryM objectForKeyedSubscript:] + 144
1 CoreTelephony 0x1b5b9e640 -[CTTelephonyNetworkInfo updateRat:descriptor:] + 140
2 CoreTelephony 0x1b5b9e534 -[CTTelephonyNetworkInfo queryRatForDescriptor:] + 392
3 CoreTelephony 0x1b5b9e2f0 -[CTTelephonyNetworkInfo queryRat] + 236
4 CoreTelephony 0x1b5b9bf90 -[CTTelephonyNetworkInfo initWithClient:] + 928
5 CoreTelephony 0x1b5b9bb90 -[CTTelephonyNetworkInfo init] + 92
6 Flurry_iOS_SDK 0x102bef8c4 +[FlurryiOSUtil getCarrierInstance] + 126 (FlurryiOSUtil.m:126)
7 Flurry_iOS_SDK 0x102bf20e8 -[FlurryNetworkStatusSource onqueue_dataProviderDidUpdate:] + 82 (FlurryNetworkStatusSource.m:82)
8 Flurry_iOS_SDK 0x102bb9ce4 __30-[FlurryActor wrapAsyncBlock:]_block_invoke + 321 (FlurryActor.m:321)
9 libdispatch.dylib 0x1b10ca610 _dispatch_call_block_and_release + 24
10 libdispatch.dylib 0x1b10cb184 _dispatch_client_callout + 16
11 libdispatch.dylib 0x1b1077404 _dispatch_lane_serial_drain$VARIANT$mp + 608
12 libdispatch.dylib 0x1b1077df8 _dispatch_lane_invoke$VARIANT$mp + 420
13 libdispatch.dylib 0x1b1081314 _dispatch_workloop_worker_thread + 588
14 libsystem_pthread.dylib 0x1b111ab88 _pthread_wqthread + 276
15 libsystem_pthread.dylib 0x1b111d760 start_wqthread + 8

crash_info_entry_0
CoreUI: deallocating _CUIInternalLinkRendition 44703 /System/Library/CoreServices/CoreGlyphs.bundle/Assets.car

The crash has occurred in an iPhone SE.

This Bug is About
Please choose the closest item by replacing [ ] with [x].

  • Integration
  • Analytics
  • Ads
  • Messaging/Push
  • IAP
  • Remote Configuration
  • Flurry Website/Dashboard
  • Other

Platform
Please choose the platform(s) that you are having the issue by replacing [ ] with [x].

  • iOS
  • tvOS
  • WatchOS

Environment
Please tell us the versions of Flurry SDK, Xcode, and iOS you are using.

  • Flurry SDK: 10.3.1
  • Xcode: 11.4.1
  • iOS: 13.3.1 (17D50)

Additional context
Are you using Flurry Push/Messaging? No
(iOS) Are you using CocoaPods or manual binary integration? Yes, CocoaPods.

@hantao-flurrydev
Copy link
Contributor

hantao-flurrydev commented May 23, 2020

Hi @Isuru-Nanayakkara, thanks for reporting this issue to Flurry iOS SDK, after investigation, we found that there is an iOS bug that causes instances of the CTTelephonyNetworkInfo class to sometimes get notifications after they have been deallocated. Instead of instantiating, using, and releasing instances, we will instead retain and never release them to work around the bug. Also we have replaced potentially deprecated CoreTelephony API call for iOS 12.0+ devices. All fixes and updates will be out in the next Flurry SDK binary release. Please let me know if you have other concerns.

@Isuru-Nanayakkara
Copy link
Author

Fantastic! Thanks for the quick response. Any idea when the next version will be released?

@hantao-flurrydev
Copy link
Contributor

@Isuru-Nanayakkara it should be out as soon as early next week. We will let you know to update the sdk binary when it has been released.

@rajkaur01
Copy link
Contributor

@Isuru-Nanayakkara Please update to v10.3.3 as this issue has been resolved.

@Isuru-Nanayakkara
Copy link
Author

Thanks, guys.

bguidolim added a commit to marmelroy/PhoneNumberKit that referenced this issue May 4, 2022
…e issues (#521)

* Keeping a single reference of CTTelephonyNetworkInfo to avoid possible issues

This commit fix #444

Resources:
 - #484
 - https://stackoverflow.com/questions/14238586/coretelephony-crash
 - flurry/flurry-ios-sdk#200
 - amplitude/Amplitude-iOS#13

* Keeping only the real changes

Co-authored-by: Bruno Guidolim <bruno@pockid.com>
elijahheineman added a commit to EverlongProject/PhoneNumberKit that referenced this issue May 9, 2023
* bump tvOS support to 10, fix build issues

* Remove duplicate methods

Fix the bug that NSRange may cross the boundary

When working with text that’s presented to the user, use the localizedCaseInsensitiveCompare(_:) method instead.

Remove duplicate calls `setup`.

* remove nil value from carrier cellular provider

* 🔗 Callable URL and call() method to call on a device

* Update RegexManager.swift

Memory leak fixed on Charset fix and tested please add to next release

* Update CountryCodePickerViewController.swift

* Metadata updates for release 8.12.43

* Xcode: Update last upgrade check

* PhoneNumber: remove call method due to build issue when included in app extension

* Regex manager: fix build error with special character set

* Release 3.3.4

* Update flag after pasting/auto-filling a phone number

* Fix Swift 5.6 crash on Linux (#516)

* Replace use of NSMutableCharacterSet with CharacterSet

This change fixes a crash when initializing PhoneNumberKit on Linux

* Use designated constant for non-breaking space

* Create stale.yml

* Updated the compile time check to work with other platforms (#435)

* Metadata update: 8.12.47 (#519)

* Metadata update: 8.12.47

* Updated tests

Co-authored-by: Bruno Guidolim <bruno@pockid.com>

* Release 3.3.5

* Update stale.yml

* Adds accessibility text to country code button, (#439)

Makes Search bar input more explicit

Co-authored-by: Rebecca Chin <rchin@league.com>

* Added modalPresentationStyle Property (#436)

* Added modalPresentationStyle Property

* Fixed optional type to modalPresentationStyle

* Fixed Sample Project

* Fix bundle accessor for SwiftUI previews. (#485)

* Release 3.3.6

* Fix bundle acessor (#523)

* Fix #522

* Let's play safe

Co-authored-by: Bruno Guidolim <bruno@pockid.com>

* Keeping a single reference of CTTelephonyNetworkInfo to avoid possible issues (#521)

* Keeping a single reference of CTTelephonyNetworkInfo to avoid possible issues

This commit fix #444

Resources:
 - marmelroy/PhoneNumberKit#484
 - https://stackoverflow.com/questions/14238586/coretelephony-crash
 - flurry/flurry-ios-sdk#200
 - amplitude/Amplitude-iOS#13

* Keeping only the real changes

Co-authored-by: Bruno Guidolim <bruno@pockid.com>

* Release 3.3.7

* Introduce coding strategies for `PhoneNumber` (#517)

* Release 3.4.0

* Makes RegexManager thread-safe (#529)

* Fix #528

* Apply suggestions from code review

Co-authored-by: Michael Williams <michaeldwilliams@users.noreply.github.com>

Co-authored-by: Michael Williams <michaeldwilliams@users.noreply.github.com>

* Release 3.4.1

* Fix bundle search path (#531)

* Updated metadata and related unit-tests according to the v8.12.50 in Google library. (#534)

Please note: With the new metadata the partial formatter can add extra characters (like `(`) in the beginning of the string detection, if the default region is `US`. The Unit test changes in the `PartialFormatterTests` are highlighting those edge cases.

* Fix #249: Potential thread explosion (#535)

* Fix #249: Potential thread explosion

* Applying suggestion from code review

* Release 3.4.2

* Simplified `defaultRegionCode()` in `PhoneNumberKit` in order to handle better the multiple SIM related issues. (#538)

- The `CoreTelephony` based solution was replaced with a `Contacts` framework based one.
- Added more strict performance tests for parsing both via an array,. and through a non-optimized way (calling the `parse(_:_:)` method in the body of a for loop).

* Updated metadata according to 8.12.51. (#537)

* Release 3.4.3

* Normalizes the way of getting metadata from MetadataManager (#542)

* Release 3.4.4

* Updated metadata according to v8.12.52 https://github.com/google/libphonenumber/releases/tag/v8.12.52 (#545)

* Release 3.4.5

* Fix #451 no flag change for different regionCode (#550)

* Falling back default region to Locale if CNContactsUserDefaults is not available (fix #549) (#551)

* Metadata automation (#554)

* Started script to automate releases

* Added GH action

* Updated metadata to version metadata/8.12.53 (#555)

Co-authored-by: runner <runner@Mac-1660824613252.local>

* Release 3.4.6

* Stop testing metadata since it's supposed to change.

* Updated metadata to version metadata/8.12.54 (#556)

Co-authored-by: runner <runner@Mac-1661155838180.local>

* Release 3.4.7

* Fix #557 (#561)

* Revert #550 due issues with other countries (#562)

* Make sample app use local package

* Revert "Fix #451 no flag change for different regionCode (#550)"

This reverts commit d8c0069.

* Release 3.4.8

* Updated metadata to version metadata/8.12.55 (#568)

Co-authored-by: runner <runner@Mac-1662768069832.local>

* Release 3.4.9

* Updated metadata to version metadata/8.12.56 (#572)

Co-authored-by: runner <runner@Mac-1663934197792.local>

* Change references to ISO 639 language code into ISO 3166 region code (#571)

* Update flags for countries with the same international code (#563)

Fix #451

* Release 3.4.10

* Updated metadata to version metadata/8.12.57 (#576)

Co-authored-by: runner <runner@Mac-1665748767715.local>

* Fix macCatalyst OS bug if language is set to Korean (#578)

* Fix minor "dispaly" typo (#580)

* Updated metadata to version metadata/8.13.0 (#582)

Co-authored-by: runner <runner@Mac-1667735953427.local>

* Make sample app use the local package

* Introducing the new error `ambiguousNumber` (#583)

ParseManager now throws an error with a list of possible phone numbers in case a number was not able to be parsed and the guessing logic provides multiple phone numbers.

* Release 3.5.0

* Updated metadata to version metadata/8.13.1 (#586)

Co-authored-by: runner <runner@Mac-1669635957015.local>

* Release 3.5.1

* Updated metadata to version metadata/8.13.2 (#589)

Co-authored-by: runner <runner@Mac-1670500125558.local>

* Release 3.5.2

* Updated metadata to version metadata/8.13.3 (#590)

Co-authored-by: runner <runner@Mac-1671752828078.local>

* Add Equatable conformance for PhoneNumberError (#591)

* Change references to ISO 639 language code into ISO 3166 region code

* Add documentation for PhoneNumberError.ambiguousNumber

* Add Equatable conformance for PhoneNumberError

* Release 3.5.3

* Add missing text field delegate method (#592)

* Updated metadata to version metadata/8.13.4 (#594)

Co-authored-by: runner <runner@Mac-1673265644046.local>

* Release 3.5.4

* Only change default region if the withFlag is true (#593)

* Fix country picker by id (#601)

* Release 3.5.5

* Revert "Fix country picker by id (#601)" (#603)

This reverts commit eddc377.

Co-authored-by: Bruno Guidolim <bruno@ruuky.com>

* Updated metadata to version metadata/8.13.5 (#604)

Co-authored-by: runner <runner@Mac-1675092228380.local>

* Release 3.5.6

* Updated metadata to version metadata/8.13.6 (#605)

* Increment Swift version (#606)

* Release 3.5.7

* Updated metadata to version metadata/8.13.7 (#608)

* ensure country codes are capitalized per ISO specs (#612)

* Release 3.5.8

* Updated metadata to version metadata/8.13.8 (#617)

* Updated metadata to version metadata/8.13.9 (#624)

* Alternate way to load contents of a file in a Data object (#620)

* Updated metadata to version metadata/8.13.10 (#631)

* Updated metadata to version metadata/8.13.11 (#636)

Co-authored-by: runner <runner@Mac-1682595728106.local>

* Release 3.5.9

---------

Co-authored-by: Nicolas Degen <ndegen@me.com>
Co-authored-by: fuyoufang <fuyoufang@163.com>
Co-authored-by: Kevin CATHALY <alaknorth@gmail.com>
Co-authored-by: Ben Myers <benl.myers01@gmail.com>
Co-authored-by: sachithamh <5784803+sachithamh@users.noreply.github.com>
Co-authored-by: Horevoi Illia <illia.horevoi@materialise.kiev.ua>
Co-authored-by: Roy Marmelstein <marmelroy@gmail.com>
Co-authored-by: Roy Marmelstein <marmelroy@spotify.com>
Co-authored-by: Alessandro Mulloni <ale@mulloni.tech>
Co-authored-by: Bruno Guidolim <bguidolim@gmail.com>
Co-authored-by: David Roman <d@vidroman.dev>
Co-authored-by: Bobby Rohweder <BobbyRohweder@gmail.com>
Co-authored-by: Bruno Guidolim <bruno@pockid.com>
Co-authored-by: Rebecca Chin <rebeccagchin@gmail.com>
Co-authored-by: Rebecca Chin <rchin@league.com>
Co-authored-by: Erkam Kücet <ekucet@gmail.com>
Co-authored-by: Luke Redpath <luke@lukeredpath.co.uk>
Co-authored-by: Michael Williams <michaeldwilliams@users.noreply.github.com>
Co-authored-by: vladislavsosiuk <vladislavsosiuk@gmail.com>
Co-authored-by: Peter Molnar <petermolnar.hu@gmail.com>
Co-authored-by: Max <14940890+MaxZheleznyy@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: runner <runner@Mac-1660824613252.local>
Co-authored-by: runner <runner@Mac-1661155838180.local>
Co-authored-by: runner <runner@Mac-1662768069832.local>
Co-authored-by: runner <runner@Mac-1663934197792.local>
Co-authored-by: Iggy Drougge <idrougge@mac.com>
Co-authored-by: runner <runner@Mac-1665748767715.local>
Co-authored-by: jackyabcde <49716044+jackyabcde@users.noreply.github.com>
Co-authored-by: Greg <greg@vida.com>
Co-authored-by: runner <runner@Mac-1667735953427.local>
Co-authored-by: runner <runner@Mac-1669635957015.local>
Co-authored-by: runner <runner@Mac-1670500125558.local>
Co-authored-by: runner <runner@Mac-1671752828078.local>
Co-authored-by: Anna Strzeżek <astrzezek@gmail.com>
Co-authored-by: runner <runner@Mac-1673265644046.local>
Co-authored-by: Olivier Destrebecq <olivier@mobdesignapps.fr>
Co-authored-by: 4taras4 <4taras4@gmail.com>
Co-authored-by: Bruno Guidolim <bruno@ruuky.com>
Co-authored-by: runner <runner@Mac-1675092228380.local>
Co-authored-by: Juantri94 <55553750+Juantri94@users.noreply.github.com>
Co-authored-by: Morgan Dock <mmdock@users.noreply.github.com>
Co-authored-by: runner <runner@Mac-1682595728106.local>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants