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

SRT: Errors caused by streaming #2318

Closed
dongrixinyu opened this issue Apr 27, 2021 · 5 comments · Fixed by #2671
Closed

SRT: Errors caused by streaming #2318

dongrixinyu opened this issue Apr 27, 2021 · 5 comments · Fixed by #2671
Assignees
Labels
SRT It's about SRT protocol. TransByAI Translated by AI/GPT.
Milestone

Comments

@dongrixinyu
Copy link

dongrixinyu commented Apr 27, 2021

Description'

Please ensure that the markdown structure is maintained.

Please describe the issue you encountered here.
'
Make sure to maintain the markdown structure.

  1. SRS version: release4.0
  2. The log of SRS is as follows:
[2021-04-26 17:33:49.490][Trace][113743][x9b94nc4] rtmp client construct url:rtmp://127.0.0.1:1935/live?vhost=uplive.sls.com/mdp265
srs: ./src/srt/ts_demux.cpp:415: int ts_demux::pes_parse(unsigned char*, size_t, unsigned char**, size_t&, uint64_t&, uint64_t&): Assertion `0x80 == p[pos]' failed.

  1. The configuration of SRS is as follows:
# SRT config.

listen              1935;
max_connections     1000;
daemon              off;
srs_log_tank        console;

http_api {
    enabled         on;
    listen          1985;
}

http_server {
    enabled         on;
    listen          18080;
    dir             ./objs/nginx/html;
}

rtc_server {        
    enabled         on;
    listen          18000;
    candidate       $CANDIDATE;
}

srt_server {
    enabled on;
    listen 10081;
    maxbw 1000000000;
    connect_timeout 4000;
    peerlatency 300;
    recvlatency 300;
}

vhost __defaultVhost__ {
}
vhost srs.srt.com.cn {
}
vhost uplive.sls.com {
}
stats {
    network         0;
    disk            sda sdb xvda xvdb;
}

Replay

How to replay bug?

Steps to reproduce the bug

1. Push srt stream to the server using a tool software

  1. srt://192.168.35.2:10081?streamid=#!::h=live/livestream,m=publish
    1. The streaming is in ts format, with video codec h264 and audio codec aac

Expected behavior

What is the problem with that error?

TRANS_BY_GPT3

@dongrixinyu
Copy link
Author

dongrixinyu commented Apr 27, 2021

Changed to another version, which is the latest development branch, the develop branch, and there is no longer this issue.

TRANS_BY_GPT3

@winlinvip winlinvip added the SRT It's about SRT protocol. label Jun 16, 2021
@runner365
Copy link
Contributor

runner365 commented Jun 17, 2021

Switched to another version, which is the latest development branch, the "develop" branch, and the issue is no longer present.

Looking at the logs, there seems to be an issue with the MPEG-TS stream. The initial fields are incorrect, causing an assertion error. Could you please check if the streaming is in MPEG-TS format?

TRANS_BY_GPT3

@winlinvip winlinvip changed the title srt 推流造成的错误 SRT: 推流造成的错误 Aug 19, 2021
@winlinvip winlinvip added this to the SRS 4.0 release milestone Aug 26, 2021
@LionMiao
Copy link

LionMiao commented Sep 27, 2021

Switched to another version, which is the latest development branch, the "develop" branch, and the issue is no longer present.

Has your issue been resolved? I followed your instructions and switched to the latest "develop" branch, but the error remains the same.
WeChat Image_20210927163056

TRANS_BY_GPT3

@runner365
Copy link
Contributor

runner365 commented Oct 13, 2021

@dongrixinyu @LionMiao
Hello both, this issue has been partially fixed and a pull request is being submitted.

I have set up a server on the public network. Are both of you in the SRS group? Could you please contact me? I would like to stream to my server and help resolve this issue. Thank you in advance.

If you are not in the group, please contact me on WeChat: sweibd. I will assist in resolving this issue.

TRANS_BY_GPT3

@runner365 runner365 linked a pull request Oct 13, 2021 that will close this issue
@runner365
Copy link
Contributor

runner365 commented Oct 19, 2021

@dongrixinyu @LionMiao The bug has been fixed and merged into the develop branch.

TRANS_BY_GPT3

@winlinvip winlinvip changed the title SRT: 推流造成的错误 SRT: Errors caused by streaming Jul 28, 2023
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SRT It's about SRT protocol. TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants