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

[BUG] srt-live-transmit removes a socket from epoll twice on connection timeout #1568

Closed
2 tasks done
maxsharabayko opened this issue Sep 22, 2020 · 0 comments · Fixed by #1569
Closed
2 tasks done
Labels
[apps] Area: Test applications related improvements Type: Bug Indicates an unexpected problem or unintended behavior
Milestone

Comments

@maxsharabayko
Copy link
Collaborator

maxsharabayko commented Sep 22, 2020

Extracted from #1566
The issue can be simply reproduced, just trying to connect to some non-existing destination like this:

./srt-live-transmit "udp://:4201?a" srt://192.168.1.237:4200
Media path: 'udp://:4201?a' --> 'srt://192.168.1.237:4200'
13:27:29.628714/T0x106da3dc0*E:SRT.ea: IPE: remove_usock: @287539723 not found as either socket or group. 
Removing only from epoll system.

The IPE message comes from SRT, as srt-live-transmit tries to remove the same socket from epoll twice.

TODO:

  • It is an error when an application tries to delete a socket that does not exist in epoll. But is it an IPE type error?
  • Fix srt-live-transmit
@maxsharabayko maxsharabayko added Type: Bug Indicates an unexpected problem or unintended behavior [apps] Area: Test applications related improvements labels Sep 22, 2020
@maxsharabayko maxsharabayko added this to the v1.5.0 - Sprint 24 milestone Sep 22, 2020
@maxsharabayko maxsharabayko linked a pull request Sep 22, 2020 that will close this issue
@mbakholdina mbakholdina modified the milestones: v1.5.0 - Sprint 24, v1.4.2 Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[apps] Area: Test applications related improvements Type: Bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants