google-oss-bot
commented
1 year ago I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.
Closed timobaehr closed 1 year ago
google-oss-bot
commented
1 year ago I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.
milaGGL
commented
1 year ago Hi @timobaehr, thank you for reporting this issue. Could you please provide a minimal reproducible app for us to debug it?
Besides, you have mentioned that Firestore integration in your app is unchanged since years. I assume you are using crashlytics. Can we pin point the date that error started based on crashlytics? How often it is happening? Are only iOS 12 users are affected?
timobaehr
commented
1 year ago Hi @milaGGL, unfortunately I cannot. I cannot reproduce the bug without iPhone 6 with 12.5.7. The bug was reported via Crashlytics only from my users with iOS 12.5.7. Nine users reported this bug 51 times. Crashes are reported between June, 20 and July, 5. Yesterday no user reported this issue but can happen that they didn't retry.
milaGGL
commented
1 year ago @timobaehr, you have mentioned Firestore integration is unchanged since years, but the SDK version in the issue description is a very recent version(10.10.0). Have you upgraded the SDK in your app recently?
timobaehr
commented
1 year ago @milaGGL I upgrade to latest versions from time to time. I meant the source code, not the version. June, 25 I updated the Flutter plugin from 4.4.5 to 4.8.1.
ios/Podfile.lock snippetpubspec.yaml snippetpodspec.lock snippetmilaGGL
commented
1 year ago @timobaehr, You have updated Flutter on June 25th, but the problem began on June 20th, so Flutter update might not be the cause.
With the given information, it is hard to debug the issue. My guess is, since there is no code change, a newly upgraded product might contain a bug, or maybe different products (Firestore, Flutter, iOS 12) have version discrepancies. If we can pin-point which product updates introduced the bug, we might be able to compare the version differences to identify the cause.
timobaehr
commented
1 year ago I guess problem was introduced by EHW+ app version 1.9.53 (app was in sale since June 20, 08:35 am).
Unfortunately Podfile.lock is not checked in so it's hard to say, what version was inside the release before.
I checked Crashlytics, the crash still appears with the latest version, starting with the app version 1.9.53.
But I was able to reproduce the bug with a cloud device. It crashes at the point of the first access to FireBase. The onboarding of the app is without Firebase, then no crash happens.
iPhone 7, iOS 12.1.4
Crashed: com.google.firebase.firestore
0 grpcpp 0x58e8 grpc_core::ExecCtx::ExecCtx() + 112
1 grpcpp 0x2bb78 grpc::SecureChannelCredentials::~SecureChannelCredentials() + 28
2 grpcpp 0x2bcbc std::__1::__shared_ptr_pointer<grpc::SecureChannelCredentials*, std::__1::shared_ptr<grpc::ChannelCredentials>::__shared_ptr_default_delete<grpc::ChannelCredentials, grpc::SecureChannelCredentials>, std::__1::allocator<grpc::SecureChannelCredentials> >::__on_zero_shared() + 20
3 FirebaseFirestore 0x90354 firebase::firestore::remote::GrpcConnection::CreateChannel() const + 424
4 FirebaseFirestore 0x900bc firebase::firestore::remote::GrpcConnection::EnsureActiveStub() + 128
5 FirebaseFirestore 0x90800 firebase::firestore::remote::GrpcConnection::CreateStream(absl::lts_20220623::string_view, firebase::firestore::credentials::AuthToken const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, firebase::firestore::remote::GrpcStreamObserver*) + 60
6 FirebaseFirestore 0x134a30 firebase::firestore::remote::Stream::ResumeStartWithCredentials(firebase::firestore::util::StatusOr<firebase::firestore::credentials::AuthToken> const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) + 132
7 FirebaseFirestore 0x1360f4 std::__1::__function::__func<firebase::firestore::remote::Stream::RequestCredentials()::$_0::operator()(absl::lts_20220623::optional<firebase::firestore::util::StatusOr<firebase::firestore::credentials::AuthToken> > const&, absl::lts_20220623::optional<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > const&) const::'lambda'(), std::__1::allocator<firebase::firestore::remote::Stream::RequestCredentials()::$_0::operator()(absl::lts_20220623::optional<firebase::firestore::util::StatusOr<firebase::firestore::credentials::AuthToken> > const&, absl::lts_20220623::optional<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > const&) const::'lambda'()>, void ()>::operator()() + 72
8 FirebaseFirestore 0x925c firebase::firestore::util::AsyncQueue::ExecuteBlocking(std::__1::function<void ()> const&) + 68
9 FirebaseFirestore 0x1463bc firebase::firestore::util::Task::ExecuteAndRelease() + 184
10 libdispatch.dylib 0x61484 _dispatch_client_callout + 16
11 libdispatch.dylib 0x8bd0 _dispatch_lane_serial_drain$VARIANT$mp + 592
12 libdispatch.dylib 0x9718 _dispatch_lane_invoke$VARIANT$mp + 432
13 libdispatch.dylib 0x11eb8 _dispatch_workloop_worker_thread + 600
14 libsystem_pthread.dylib 0xc0dc _pthread_wqthread + 312
15 libsystem_pthread.dylib 0xecec start_wqthread + 4Nice! Would you be able to provide us with a minimal reproducible app with repro steps?
timobaehr
commented
1 year ago Here some code to reproduce the bug. I tested it on iOS 16 - no crash. On iPhone 7 with iOS 12.1 this example app crashes.
Example is nearly similar to this one.
main.dartmodel.dart
kiritokatklian
commented
1 year ago Here to confirm that it's indeed a Firebase issue and not a Flutter issue. This started happening to an app I'm working on which is written in Swift. It's also affecting only iOS 12.
hermuxo
commented
1 year ago Here for confirmation of the issue: https://github.com/firebase/flutterfire/issues/11216
hermuxo
commented
1 year ago I bought a old iPhone 6 to debug. Strangely this is not happening in debug mode only in release (install via Testflight)
DahanSimon
commented
1 year ago Using SPM instead of cocoapods fixed the issue for me
paulb777
commented
1 year ago @timobaehr or someone else seeing the issue on CocoaPods: Would you share the Podfile and full Podfile.lock including the grpc versions?
timobaehr
commented
1 year ago @paulb777 For sure.
Here the X versions separately:
- BoringSSL-GRPC (0.0.24):
- BoringSSL-GRPC/Implementation (= 0.0.24)
- BoringSSL-GRPC/Interface (= 0.0.24)
- BoringSSL-GRPC/Implementation (0.0.24):
- BoringSSL-GRPC/Interface (= 0.0.24)
- BoringSSL-GRPC/Interface (0.0.24)ios/Podfile snippetPodfile.lock snippetpaulb777
commented
1 year ago Thanks @timobaehr. that all looks right to me and should match the SPM distribution. The only difference I can think of is that the SPM binary distribution is built with an earlier Xcode version than 14.3.1.
Does it make a difference to build with an Xcode 14.2 or earlier - or to use the Invertase binary Firestore distribution from https://github.com/invertase/firestore-ios-sdk-frameworks?
timobaehr
commented
1 year ago Does it make a difference to build with an Xcode 14.2 or earlier - or to use the Invertase binary Firestore distribution from https://github.com/invertase/firestore-ios-sdk-frameworks?
@paulb777 Sorry, I'm in vacation with reduced development environment. The earliest I can check this is at the end of the week.
milaGGL
commented
1 year ago Hi @timobaehr, I am having a hard time locating an iOS12 device and reproducing the error by myself.
Based on the GrpcConnection::EnsureActiveStub() function in the stack, we are suspecting that this issue might be related to this one: https://github.com/firebase/firebase-ios-sdk/issues/11568.
It would be greatly appreciated if you could help reproduce the crash with debug logging enabled. This may give some insight into what the grpc channel to enter the GRPC_CHANNEL_SHUTDOWN state, so that we could reproduce it with any iOS version (not just iOS 12).
victors1681
commented
1 year ago Same issue: only happening on iOS 12.x
Version:12.5.7
Model:iPad Mini 2
Crashed: com.google.firebase.firestore
SIGSEGV 0x000000000000cc67
grpcpp
grpc_core::ExecCtx::ExecCtx() + 112
grpcpp
grpc::SecureChannelCredentials::~SecureChannelCredentials() + 28
grpcpp
std::__1::__shared_ptr_pointer<grpc::SecureChannelCredentials*, std::__1::shared_ptr<grpc::ChannelCredentials>::__shared_ptr_default_delete<grpc::ChannelCredentials, grpc::SecureChannelCredentials>, std::__1::allocator<grpc::SecureChannelCredentials> >::__on_zero_shared() + 20
FirebaseFirestore
firebase::firestore::remote::GrpcConnection::CreateChannel() const + 424
I had to downgrade version by version to version 10.9.0 and is working!
OleksiiSB
commented
1 year ago Looks like the same issue we are experiencing on iOS 12.x after updating to latest Unity SDK. Any chance to have this fix soon or it is better to just downgrade?
Crashed: com.google.firebase.firestore
0 grpcpp 0x7bd0 grpc_core::ExecCtx::ExecCtx() + 156 (time.h:156)
1 grpcpp 0x2f11c grpc::SecureChannelCredentials::~SecureChannelCredentials() + 54 (secure_credentials.h:54)
2 grpcpp 0x2f320 std::__1::__shared_ptr_pointer<grpc::SecureChannelCredentials*, std::__1::shared_ptr<grpc::ChannelCredentials>::__shared_ptr_default_delete<grpc::ChannelCredentials, grpc::SecureChannelCredentials>, std::__1::allocator<grpc::SecureChannelCredentials> >::__on_zero_shared() + 57 (unique_ptr.h:57)
3 FirebaseFirestore 0x94368 firebase::firestore::remote::GrpcConnection::CreateChannel() const + 220 (shared_ptr.h:220)
4 FirebaseFirestore 0x94098 firebase::firestore::remote::GrpcConnection::EnsureActiveStub() + 869 (shared_ptr.h:869)
5 FirebaseFirestore 0x94888 firebase::firestore::remote::GrpcConnection::CreateStream(absl::lts_20220623::string_view, firebase::firestore::credentials::AuthToken const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, firebase::firestore::remote::GrpcStreamObserver*) + 343 (grpc_connection.cc:343)
6 FirebaseFirestore 0x1416e4 firebase::firestore::remote::Stream::ResumeStartWithCredentials(firebase::firestore::util::StatusOr<firebase::firestore::credentials::AuthToken> const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&) + 308 (unique_ptr.h:308)
7 FirebaseFirestore 0x142c14 std::__1::__function::__func<firebase::firestore::remote::Stream::RequestCredentials()::$_0::operator()(absl::lts_20220623::optional<firebase::firestore::util::StatusOr<firebase::firestore::credentials::AuthToken> > const&, absl::lts_20220623::optional<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > const&) const::'lambda'(), std::__1::allocator<firebase::firestore::remote::Stream::RequestCredentials()::$_0::operator()(absl::lts_20220623::optional<firebase::firestore::util::StatusOr<firebase::firestore::credentials::AuthToken> > const&, absl::lts_20220623::optional<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > > const&) const::'lambda'()>, void ()>::operator()() + 176 (shared_ptr.h:176)
8 FirebaseFirestore 0xab60 firebase::firestore::util::AsyncQueue::ExecuteBlocking(std::__1::function<void ()> const&) + 997 (atomic:997)
9 FirebaseFirestore 0x15416c firebase::firestore::util::Task::ExecuteAndRelease() + 491 (function.h:491)
10 libdispatch.dylib 0x607d4 _dispatch_client_callout + 16
11 libdispatch.dylib 0x9324 _dispatch_lane_serial_drain$VARIANT$mp + 592
12 libdispatch.dylib 0x9e40 _dispatch_lane_invoke$VARIANT$mp + 428
13 libdispatch.dylib 0x124ac _dispatch_workloop_worker_thread + 596
14 libsystem_pthread.dylib 0xc114 _pthread_wqthread + 304
15 libsystem_pthread.dylib 0xecd4 start_wqthread + 4
brettwillis
commented
1 year ago Confirming for me also it only happens when building in Release configuration. Does not occur in Debug configuration.
FirebaseFirestore v10.12.0, gRPC-C++ v1.50.1
mgrms
HannahShiSFB
dconeybe
TheStalwart
Description
Users with old iPhones/iPads running under iOS 12 (v12.5.7) see a full app crash of the app with Firestore. Affected devices are e.g. iPhone 6, iPad Air
Reproducing the issue
Firebase SDK Version
10.10.0
Xcode Version
14.3.1
Installation Method
CocoaPods
Firebase Product(s)
Firestore
Targeted Platforms
iOS
Relevant Log Output
If using CocoaPods, the project's Podfile.lock
Expand
Podfile.locksnippet```yml PODS: - Firebase/Analytics (10.10.0): - Firebase/Core - Firebase/Auth (10.10.0): - Firebase/CoreOnly - FirebaseAuth (~> 10.10.0) - Firebase/Core (10.10.0): - Firebase/CoreOnly - FirebaseAnalytics (~> 10.10.0) - Firebase/CoreOnly (10.10.0): - FirebaseCore (= 10.10.0) - Firebase/Crashlytics (10.10.0): - Firebase/CoreOnly - FirebaseCrashlytics (~> 10.10.0) - Firebase/Firestore (10.10.0): - Firebase/CoreOnly - FirebaseFirestore (~> 10.10.0) - Firebase/Messaging (10.10.0): - Firebase/CoreOnly - FirebaseMessaging (~> 10.10.0) - firebase_analytics (10.4.3): - Firebase/Analytics (= 10.10.0) - firebase_core - Flutter - firebase_auth (4.6.3): - Firebase/Auth (= 10.10.0) - firebase_core - Flutter - firebase_core (2.14.0): - Firebase/CoreOnly (= 10.10.0) - Flutter - firebase_crashlytics (3.3.3): - Firebase/Crashlytics (= 10.10.0) - firebase_core - Flutter - firebase_messaging (14.6.3): - Firebase/Messaging (= 10.10.0) - firebase_core - Flutter - FirebaseAnalytics (10.10.0): - FirebaseAnalytics/AdIdSupport (= 10.10.0) - FirebaseCore (~> 10.0) - FirebaseInstallations (~> 10.0) - GoogleUtilities/AppDelegateSwizzler (~> 7.11) - GoogleUtilities/MethodSwizzler (~> 7.11) - GoogleUtilities/Network (~> 7.11) - "GoogleUtilities/NSData+zlib (~> 7.11)" - nanopb (< 2.30910.0, >= 2.30908.0) - FirebaseAnalytics/AdIdSupport (10.10.0): - FirebaseCore (~> 10.0) - FirebaseInstallations (~> 10.0) - GoogleAppMeasurement (= 10.10.0) - GoogleUtilities/AppDelegateSwizzler (~> 7.11) - GoogleUtilities/MethodSwizzler (~> 7.11) - GoogleUtilities/Network (~> 7.11) - "GoogleUtilities/NSData+zlib (~> 7.11)" - nanopb (< 2.30910.0, >= 2.30908.0) - FirebaseAppCheckInterop (10.11.0) - FirebaseAuth (10.10.0): - FirebaseAppCheckInterop (~> 10.0) - FirebaseCore (~> 10.0) - GoogleUtilities/AppDelegateSwizzler (~> 7.8) - GoogleUtilities/Environment (~> 7.8) - GTMSessionFetcher/Core (< 4.0, >= 2.1) - FirebaseCore (10.10.0): - FirebaseCoreInternal (~> 10.0) - GoogleUtilities/Environment (~> 7.8) - GoogleUtilities/Logger (~> 7.8) - FirebaseCoreExtension (10.11.0): - FirebaseCore (~> 10.0) - FirebaseCoreInternal (10.11.0): - "GoogleUtilities/NSData+zlib (~> 7.8)" - FirebaseCrashlytics (10.10.0): - FirebaseCore (~> 10.5) - FirebaseInstallations (~> 10.0) - FirebaseSessions (~> 10.5) - GoogleDataTransport (~> 9.2) - GoogleUtilities/Environment (~> 7.8) - nanopb (< 2.30910.0, >= 2.30908.0) - PromisesObjC (~> 2.1) - FirebaseFirestore (10.10.0): - abseil/algorithm (~> 1.20220623.0) - abseil/base (~> 1.20220623.0) - abseil/container/flat_hash_map (~> 1.20220623.0) - abseil/memory (~> 1.20220623.0) - abseil/meta (~> 1.20220623.0) - abseil/strings/strings (~> 1.20220623.0) - abseil/time (~> 1.20220623.0) - abseil/types (~> 1.20220623.0) - FirebaseCore (~> 10.0) - "gRPC-C++ (~> 1.50.1)" - leveldb-library (~> 1.22) - nanopb (< 2.30910.0, >= 2.30908.0) - FirebaseInstallations (10.11.0): - FirebaseCore (~> 10.0) - GoogleUtilities/Environment (~> 7.8) - GoogleUtilities/UserDefaults (~> 7.8) - PromisesObjC (~> 2.1) - FirebaseMessaging (10.10.0): - FirebaseCore (~> 10.0) - FirebaseInstallations (~> 10.0) - GoogleDataTransport (~> 9.2) - GoogleUtilities/AppDelegateSwizzler (~> 7.8) - GoogleUtilities/Environment (~> 7.8) - GoogleUtilities/Reachability (~> 7.8) - GoogleUtilities/UserDefaults (~> 7.8) - nanopb (< 2.30910.0, >= 2.30908.0) - FirebaseSessions (10.11.0): - FirebaseCore (~> 10.5) - FirebaseCoreExtension (~> 10.0) - FirebaseInstallations (~> 10.0) - GoogleDataTransport (~> 9.2) - GoogleUtilities/Environment (~> 7.10) - nanopb (< 2.30910.0, >= 2.30908.0) - PromisesSwift (~> 2.1) - ... DEPENDENCIES: - firebase_analytics (from `.symlinks/plugins/firebase_analytics/ios`) - firebase_auth (from `.symlinks/plugins/firebase_auth/ios`) - firebase_core (from `.symlinks/plugins/firebase_core/ios`) - firebase_crashlytics (from `.symlinks/plugins/firebase_crashlytics/ios`) - firebase_messaging (from `.symlinks/plugins/firebase_messaging/ios`) - ... ```