-
Notifications
You must be signed in to change notification settings - Fork 117
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
Crash after closing video call #304
Comments
Hello @ve-sdk-ios, can you provide a detailed stack trace? Just using |
There is full console output with |
I could not reproduce it. I tried your code and went through the steps 3 X 5+ times (publish only). Anyway one suggestion would be to un-publish when the call is closed and do a dismiss on - (IBAction)closeAction:(id)sender {
[_session unpublish:_publisher error:nil];
}
- (void)publisher:(OTPublisherKit*)publisher
streamDestroyed:(OTStream *)stream {
[self dismissViewControllerAnimated:YES completion:nil];
}
Hopefully that will free up the camera in time for the next sequence of Start. Let me know if that helps. Also from the logs I see thread #11
frame #0: 0x00000001d76cef0c libsystem_kernel.dylib`kevent + 8
frame #1: 0x0000000102245edc Video-Transformers`-[OTSubscriber initWithStream:delegate:] + 969564
frame #2: 0x0000000102241f34 Video-Transformers`-[OTSubscriber initWithStream:delegate:] + 953268
frame #3: 0x00000001021cadb4 Video-Transformers`-[OTSubscriber initWithStream:delegate:] + 465460
frame #4: 0x000000010217f174 Video-Transformers`-[OTSubscriber initWithStream:delegate:] + 155124
frame #5: 0x00000001f9f744d4 libsystem_pthread.dylib`_pthread_start + 136 Are there subscribers involved? I will keep on trying to repo. I tested on iPhone 12 Pro/ 17.1.2 btw. |
I tested without subscribers. I have updated the our fork with your recommendation here. Still crashed but something new. In unpublish I have got sometimes I got error UPD: Looks like I close the screen before publishing is started |
It's become difficult to reproduce but it still possible. I see this error in console |
In your logs what is
Will it be possible to try a Lastly will it be possible to just not use the modal controller and just do the publisher on the main view like it was initially in that sample. That will sort of narrow down the cause of the crash (big if...) |
I refactored fork with your instructions and it still crashes. It is our use case. User can start the live stream and finish them. And after that start a live stream again. We are expected that all will work without crashes. |
I wish I can reproduce it so we don't go back and forth so much. Thanks for your patience. Anyway the It is strange we are seeing this. Just for experimentation can you turn off all transformers code. Just a plain vanilla publish - unpublish - disconnect (start and close in modal) and check if it is reproducible. |
@IGitGotIt @juliobecerragomez Did you fix something on your side? I can't see anymore logs
And the crash is not reproducible now. |
That's good. No we did not do anything. Did you restart the phone ? |
@IGitGotIt how does restarting phone impact on the crash? |
I was just asking "was the reason you are not seeing any crashes NOW, was because you restarted the phone or not". |
Is the issue fixed now, since you are not seeing any more crashes. |
@IGitGotIt , we are still experiencing the same crash in our build and cannot release the app. |
I am confused by conflicting messages about crashes happening and not happening here and I have not been able to repo the crash with the original sample code. Anyway did you try our previous "experimental" recommendation #304 (comment) . What was the result? Thanks |
@ve-sdk-ios @GlebPBanuba meanwhile we were able to reproduce your issue. Sorry for the late update, but the issue was already solved on 2.27.2 release. Please check release docs If you still are facing the issue in the latest version please re-open the ticket again. |
Describe the bug
Crash appears after several starting and closing the call in our app. Crash reproduces on iPhone 12 iOS 17.2.1(several devices) and not reproduced on iPhone X iOS 16.7.1.
To Reproduce
I reproduce this crash in our fork from this repo: https://github.com/ve-sdk-ios/opentok-ios-sdk-samples. There is additional start screen with button "Start call" and "Close" button at existing ViewController screen. Action at "Start call" opens yours controller in Video-Transformers sample
Steps to reproduce the behavior:
After several times it can be like 3-5 or more the crash appears.
Expected behavior
Crash should not be exists
Screenshots
Device (please compete the following information):
Additional context
In our app we have got similar issue but stack trace is different. We have tried to disconnect session before closing the view but it wouldn't help
The text was updated successfully, but these errors were encountered: