RoboCup-SSL / ssl-vision

Shared Vision System For The RoboCup Small Size League
GNU General Public License v3.0
92 stars 111 forks source link

Choose a random free port instead of the target port for local bind #82

Closed g3force closed 6 years ago

g3force commented 6 years ago

If some other applications also binds to the same local port and forgets to also set SO_REUSEPORT, the bind will fail. There is no reason for binding to a fixed local port in this case.

joydeep-b commented 6 years ago

This looks fine to me, once tested, can merge to master.

andre-ryll commented 6 years ago

I tested this with a real field and vision setup and can confirm that the vision now uses an ephemeral source port. Target port and functionality is unaffected and works as before.