Crash: CaptureSDK CSktXmlTag::~CSktXmlTag()

ChernyshenkoTaras commented 10 months ago

Hi, I encountered a crash while using the iOS application, and it appears to be related to the CaptureSDK 1.6. I would greatly appreciate your assistance in resolving this issue. Please let me know if you require any additional information.

iOS version: 14.6 Device: iPhone 7 SDK: CaptureSDK 1.6

Stack trace:

Crashed: Thread #1
SIGABRT ABORT 0x00000001d20817b0

Crashed: Thread
0  libsystem_kernel.dylib         0x277b0 __pthread_kill + 8
1  libsystem_pthread.dylib        0xa9c0 pthread_kill + 212
2  libsystem_c.dylib              0x73a44 abort + 100
3  libsystem_malloc.dylib         0x1d0a8 _malloc_put + 554
4  libsystem_malloc.dylib         0x1d260 malloc_report + 60
5  libsystem_malloc.dylib         0x13448 free + 444
6  CaptureSDK                     0x81bbc CSktXmlTag::~CSktXmlTag() + 52
7  CaptureSDK                     0x81c8c CSktXmlTag::~CSktXmlTag() + 20
8  CaptureSDK                     0x82b5c CSktXmlParser::Parse(char const*, unsigned long) + 88
9  CaptureSDK                     0x6f6b8 CSktConfigXml::Load(char const*, char const*, char const*) + 488
10 CaptureSDK                     0x676e8 CSktCaptureConfiguration::RetrieveConfiguration(CSktConfigXml*) + 108
11 CaptureSDK                     0x64960 CSktCaptureConfiguration::ReadSerialPorts(char*, unsigned int) + 96
12 CaptureSDK                     0x68f9c CSktCapture::ReadConfiguration(char const*, TSktCaptureObject**) + 336
13 CaptureSDK                     0x13efc SktCaptureListenerThread(void*) + 296
14 libsystem_pthread.dylib        0x9c74 _pthread_start + 288
15 libsystem_pthread.dylib        0xe878 thread_start + 8

cyrille-socket commented 10 months ago

Hi @ChernyshenkoTaras

please update to the latest CaptureSDK version 1.8.34. We have fixe this crash.

Also please use https://github.com/SocketMobile/swift-package-capturesdk. We will deprecate the Cocoapods in a year or so.

ChernyshenkoTaras commented 10 months ago

Hi there,

The same issue is still reproduces with the CaptureSDK version 1.8.34. This time it happened on iOS 15.8.0 and iPhone 7 Plus

cyrille-socket commented 9 months ago

Hi @ChernyshenkoTaras

would you be able to send us the XML file that our CaptureSDK creates in your app?

Do you reproduce when running your app on a device by yourself or is it only from crashes from some of your customers?

Do you set a config with a setProperty and the property ID SKTCapturePropertyIDConfiguration in your code to do something in particular?

Thank you for your answers

Cyrille

Pranit-Harekar commented 9 months ago

Hi @cyrille-socket - may I know the reason behind deprecating the Cocoapods library and instead switching to SPM?

ios-dev-socket commented 9 months ago

Hi @Pranit-Harekar,

since some time now, publishing with Cocoapods slows our release process down. Now SPM is the new sheriff in town and a lot of developers have migrated and really simplifies the integration of external dependencies in Xcode project without having additional code or script that sometimes fail, it's really a game changer.

Also an Xcode project can have both SPM and Cocoapods dependencies. So developers who don't want to remove all their Cocoapods can at least use SPM for our SDK in the future.

We will deprecate our Cocoapods in 1 year (1/1/2025).

cyrille-socket commented 8 months ago

Hi @ChernyshenkoTaras ,

can you send us your xml file please?

ChernyshenkoTaras commented 8 months ago

Hi @cyrille-socket ! Could u explain please where I can locate this xml file? I'm still using cocoapods in my project for installation of CaptureSDK if that's matter

cyrille-socket commented 8 months ago

It's in the following folder of the app: Library/Caches/Capture.xml

cyrille-socket commented 8 months ago

Hi @ChernyshenkoTaras . Any update on this?

cyrille-socket commented 8 months ago

Hi @ChernyshenkoTaras,

I investigated with your file and I can't reproduce any crash. The format, the keys and the values are ok.

Do you reproduce on your end on development simulators or development devices at least?

ChernyshenkoTaras commented 7 months ago

Hi @cyrille-socket,

The crash was reproduced only once on our side, and we were not able to reproduce it again. This seems to be a quite rare issue, as in the last 30 days only 35 people encountered it, according to Crashlytics. However, it appears that for most of them the crash occurred more than once, since we have recorded 51 crashes in the last 30 days.

I suspect the crash reproduces more often on older devices. Most crashes occurred on iPhone 6 and 7 models. Here are the details:

cyrille-socket commented 5 months ago

This crash has been fixed in the new release 1.9.18 in our Developer Portal

Pranit-Harekar commented 3 months ago

@cyrille-socket @ios-dev-socket Regarding the previous comment, it looks like the SPM package is also deprecated. Are you going to deprecate both Cocoapods and SPM? Please let me know, thanks!

cyrille-socket commented 3 months ago

Hi @Pranit-Harekar. Yes we did deprecate Cocoapods and SPM on our Github page. They're now available through our Developer Portal for technical reasons.

SocketMobile / cocoapods-capturesdk

Crash: CaptureSDK CSktXmlTag::~CSktXmlTag() #9