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

DVR: fix large file mp4 dvr failed. #2800

Merged
merged 1 commit into from
Dec 23, 2021

Conversation

xiaozhihong
Copy link
Collaborator

No description provided.

@xiaozhihong xiaozhihong linked an issue Dec 17, 2021 that may be closed by this pull request
@xiaozhihong
Copy link
Collaborator Author

xiaozhihong commented Dec 17, 2021

DVR MP4 section, stco supports 32-bit offset, co64 supports 64-bit offset. Both encoding and decoding of these two boxes are implemented in SRS. However, only stco is used when writing. When the file is too large and the offset exceeds UINT32_MAX, the MP4 offset will be incorrect, resulting in playback failure for files beyond 4GB.

TRANS_BY_GPT3

@winlinvip winlinvip self-assigned this Dec 17, 2021
@xiaozhihong xiaozhihong force-pushed the 4.0release branch 2 times, most recently from 7ca416f to b439802 Compare December 22, 2021 11:24
@winlinvip winlinvip merged commit aa85ce0 into ossrs:4.0release Dec 23, 2021
peipeiguo pushed a commit to peipeiguo/srs that referenced this pull request Sep 7, 2022
peipeiguo pushed a commit to peipeiguo/srs that referenced this pull request Sep 9, 2022
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DVR: mp4 recording, the part exceeding 4.1GB cannot be played.
3 participants