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

LaunchDarkly Crash on Timer #265

Open
res0nance opened this issue Mar 29, 2022 · 4 comments
Open

LaunchDarkly Crash on Timer #265

res0nance opened this issue Mar 29, 2022 · 4 comments

Comments

@res0nance
Copy link

Describe the bug
We get a crash captured with firebase

To reproduce
We unfortunately don't know exactly how this issue occurs because the stack trace does not seem to originate from our code

Expected behavior
LaunchDarkly should not crash

Logs

Crashed: LaunchDarkly.FlagSynchronizer.syncQueue
EXC_BAD_ACCESS KERN_INVALID_ADDRESS 0x00000000d024ee20
Crashed: LaunchDarkly.FlagSynchronizer.syncQueue
0  libobjc.A.dylib                0x33f0 objc_retain + 16
1  libswiftCore.dylib             0x3c83d4 swift_bridgeObjectRetain + 48
2  LaunchDarkly                   0x959c specialized Dictionary.init(dictionaryLiteral:) + 104 (<compiler-generated>:104)
3  LaunchDarkly                   0x10b14 DarklyService.getFeatureFlags(useReport:completion:) + 812 (<compiler-generated>:812)
4  LaunchDarkly                   0x449f4 FlagSynchronizer.makeFlagRequest(isOnline:) + 440 (<compiler-generated>:440)
5  LaunchDarkly                   0x474c0 partial apply for implicit closure #6 in implicit closure #5 in FlagSynchronizer.startPolling(isOnline:) + 100 (<compiler-generated>:100)
6  LaunchDarkly                   0x62444 closure #1 in LDTimer.timerFired() + 128 (<compiler-generated>:128)
7  LaunchDarkly                   0x61fb8 thunk for @escaping @callee_guaranteed () -> () + 20 (<compiler-generated>:20)
8  libdispatch.dylib              0x2924 _dispatch_call_block_and_release + 32
9  libdispatch.dylib              0x4670 _dispatch_client_callout + 20
10 libdispatch.dylib              0xbdf4 _dispatch_lane_serial_drain + 672
11 libdispatch.dylib              0xc968 _dispatch_lane_invoke + 392
12 libdispatch.dylib              0x171b8 _dispatch_workloop_worker_thread + 656
13 libsystem_pthread.dylib        0x10f4 _pthread_wqthread + 288
14 libsystem_pthread.dylib        0xe94 start_wqthread + 8

SDK version
5.4.5

Language version, developer tools

  • Xcode 13.2.1
  • Swift 5

OS/platform
We see this issue on iOS 14 and 15, potentially more.

@louis-launchdarkly
Copy link
Contributor

Hello @res0nance, thank you for reaching out. Does this happen repeatedly and/or frequently? I have also created a task in the team to look at this.

Filed Internally as 147990

@res0nance
Copy link
Author

Hello @res0nance, thank you for reaching out. Does this happen repeatedly and/or frequently? I have also created a task in the team to look at this.

Filed Internally as 147990

It has happened on several of our app releases. It doesn't seem to happen all that frequently

@louis-launchdarkly
Copy link
Contributor

Hello @res0nance, thank you for checking. I have discussed this issue with the team, and we have started and are currently working on a newer version of the iOS SDK to address the issues that relate to the iOS SDK's Dictionary usage. I don't have an ETA yet, but I will give you another update when the new version is close to done.

@res0nance
Copy link
Author

res0nance commented Oct 11, 2022 via email

This was referenced Dec 29, 2023
keelerm84 pushed a commit that referenced this issue Jan 3, 2024
🤖 I have created a release *beep* *boop*
---


##
[8.4.0](8.3.1...8.4.0)
(2023-12-29)


### Features

* Store and use e-tag header between SDK initializations
([#268](#268))
([701aaa8](701aaa8))


### Bug Fixes

* LDContext equality is no longer order dependent
([#265](#265))
([683e0c3](683e0c3))
* Use stable encoding format to increase cache hits
([#267](#267))
([40a5d01](40a5d01))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: LaunchDarklyReleaseBot <LaunchDarklyReleaseBot@launchdarkly.com>
keelerm84 added a commit that referenced this issue Jan 3, 2024
🤖 I have created a release *beep* *boop*
---


##
[9.3.0](9.2.1...9.3.0)
(2024-01-02)


### Features

* Store and use e-tag header between SDK initializations
([#268](#268))
([701aaa8](701aaa8))


### Bug Fixes

* LDContext equality is no longer order dependent
([#265](#265))
([683e0c3](683e0c3))
* Use stable encoding format to increase cache hits
([#267](#267))
([40a5d01](40a5d01))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: LaunchDarklyReleaseBot <LaunchDarklyReleaseBot@launchdarkly.com>
Co-authored-by: Matthew Keeler <mkeeler@launchdarkly.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants