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

1. rtp_stream can not bind on mac. 2. \r\n in injection file #728

Closed
exploman opened this issue May 11, 2024 · 1 comment
Closed

1. rtp_stream can not bind on mac. 2. \r\n in injection file #728

exploman opened this issue May 11, 2024 · 1 comment

Comments

@exploman
Copy link

exploman commented May 11, 2024

Device information

system: macOS 14.4 (23E214)
Device: MacBook Pro Apple M1 Pro 16G

Code version

master

There are two problem happened in my test.

1. rtp_stream can not bind on mac.

In trace_err log: "Could not bind port for RTP streaming after 100 tries".
If I use tag v3.5.0-rc1. This problem will not happen. By searching for commit history, I found "SOCK_ADDR_SIZE" macro.
By debug, if I use SOCK_ADDR_SIZE, bind will not return -1.
In this merge request, #251. I get the reason. "OSX refuse to work
with an inexact (too large) addrlen." But in this merge, not fix rtp_stream. I will do it in my merge request.

2. \r\n in injection file

If \r\n is used at the end of each line in the injected file, it will result in the last field being read out as xxx\r.
Such as my last field is a float: 7800.0. In debug, I found the value is 7800.0/r. And this value can not convert into float.
In this commit 131ad9d I found example to fix first line.

@exploman
Copy link
Author

exploman commented May 11, 2024

Pull request: #729
@wdoekes

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

No branches or pull requests

1 participant