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] Do not bind unused ports #1782

Closed
kiyotsu opened this issue Aug 23, 2023 · 2 comments
Closed

[BUG] Do not bind unused ports #1782

kiyotsu opened this issue Aug 23, 2023 · 2 comments

Comments

@kiyotsu
Copy link
Contributor

kiyotsu commented Aug 23, 2023

Hello.
When I run this application as root, it binds port 132.
Port 132 is opened by the usrsctp library.
However, I do not believe this port is used by WebRTC and should not be open.

I checked usrsctp and found the following pull request.
It looks like I can change the initialization function to not bind this port.
sctplab/usrsctp#532

After changing the initialization function and checking, the unused port is no longer bound.
kiyotsu@300e672

I would like to see the bind of unused ports fixed.

SDK version number
b2b05c8

Environment

  • Ubuntu 22.04
@kiyotsu kiyotsu added bug Something isn't working needs-triage labels Aug 23, 2023
@jdelapla
Copy link
Contributor

jdelapla commented Sep 1, 2023

Hello @kiyotsu ,
Would you like to create a pull request to our develop branch so you can receive credit for this fix? If not I can go ahead and make the change myself.
Thank you for your deep dive.

@niyatim23
Copy link
Contributor

The PR to fix the issue has been merged. Thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants