bug: setup of snaphsot listeners leads to crash on iOS

[BoveFabio]

Plugin(s)

• Analytics
• App
• App Check
• Authentication
• Crashlytics
• Cloud Firestore
• Cloud Functions
• Cloud Messaging
• Cloud Storage
• Performance
• Remote Config

Version

6.3.1

Platform(s)

• Android
• iOS
• Web

Current behavior

When I set up (multiple) snapshot listeners in parallel, the iOS crashes randomly with varying errors. Among them are:

• Thread 4: "-[__NSCFNumber count]: unrecognized selector sent to instance 0x8000000000000000"
• Thread 8: "-[NSTaggedPointerString count]: unrecognized selector sent to instance 0x8000000000000000"
• Thread 17: EXC_BAD_ACCESS (code=1, address=0x10)

XCode shows me an error in L244 in @capacitor-firebase/firestore/ios/Plugin/FirebaseFirestore.swift

self.listenerRegistrationMap[callbackId] = listenerRegistration

Unfortunately, I have no clue about Swift development, so I'm rather helpless with this.

Given that this happens in a very simple setup (see attached repo for a minimal reproduction), I am inclined to say that I am not using the plugin in a way it is not intended to be used.

Any feedback on this would be great!

Expected behavior

Setting up snapshot listeners should not lead to crashes.

Reproduction

https://github.com/BoveFabio/capawesome-firestore-reproduction

Steps to reproduce

Check out the attached repo, then:

1. npm i
2. npm run build
3. Replace [your local IP] inside the capacitor.config.ts
4. npx cap sync
5. Add firebase config as described in the docs: https://github.com/capawesome-team/capacitor-firebase/blob/main/docs/firebase-setup.md#ios
6. npm run dev -- --host
7. Start the app on any emulator or real device
8. Play around with numberOfSnapshotListeners in App.tsx
9. Observe the error in XCode (file @capacitor-firebase/firestore/ios/Plugin/FirebaseFirestore.swift, e.g. )

Other information

No response

Capacitor doctor

❯ npx cap doctor
💊 Capacitor Doctor 💊

Latest Dependencies:

@capacitor/cli: 6.2.0
@capacitor/core: 6.2.0
@capacitor/android: 6.2.0
@capacitor/ios: 6.2.0

Installed Dependencies:

@capacitor/cli: 6.2.0
@capacitor/core: 6.2.0
@capacitor/android: 6.2.0
@capacitor/ios: 6.2.0

[success] iOS looking great! 👌
[success] Android looking great! 👌

Before submitting

• I have read and followed the bug report guidelines.
• I have attached links to possibly related issues and discussions.
• I understand that incomplete issues (e.g. without reproduction) are closed.

[robingenz]

Thank you reaching out. This issue should be a duplicate of #771. Feel free to give our latest pre-release a try:

npm i https://pkg.pr.new/capawesome-team/capacitor-firebase/@capacitor-firebase/firestore@0bffff0

[BoveFabio]

Sorry for that! I swear I searched issues and discussions beforehand 🫣

Thanks, I'll check it out.

[Mortenjoe]

Not to hijack the issue but I had similar issue which also sounds like the one mentioned in #771. My app sometimes crashed while creating multiple snapshot listeners with an EXC_BAD_ACCESS pointing to the snapshot listeners. After installing the suggested prerelease i've had no crashes! Seems to work for me :-)

Best regards

Morten

[robingenz]

Thank you. I will close this issue for now.