Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#5051 closed defect (invalid)

ffmpeg obtains only first 1024 bytes for some RTSP streams

Reported by: ioeir2 Owned by:
Priority: important Component: avformat
Version: unspecified Keywords: rtsp regression
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Summary of the bug:
How to reproduce:

% ffplay "rtsp://user:pass@myip:8001/0/audio" -rtsp_transport tcp
ffmpeg 2.8.3, ffmpeg git dec 1

The error introduced in commit 764ec70149728be82304c163ccc4e280f1629201 on Jul 30, 2015. Reverting this commit fix the issue. No such issue on ffmpeg 2.7.3 version.

The bug is related to the closed defect #4952 which partially reverted commit 764ec70149728be82304c163ccc4e280f1629201, but instead of "if (ch == '$' && q == buf) {" should be "if (ch == '$') {" to fix the issue.

Attachments (3)

audio_rtsp.pcapng (31.8 KB ) - added by ioeir2 8 years ago.
Wireshark dump
logs.txt (260 bytes ) - added by ioeir2 8 years ago.
ffplay output
rtsp.c.trac5051.patch (389 bytes ) - added by ioeir2 8 years ago.
Patch for fixing the bug

Download all attachments as: .zip

Change History (6)

by ioeir2, 8 years ago

Attachment: audio_rtsp.pcapng added

Wireshark dump

by ioeir2, 8 years ago

Attachment: logs.txt added

ffplay output

by ioeir2, 8 years ago

Attachment: rtsp.c.trac5051.patch added

Patch for fixing the bug

comment:1 by Carl Eugen Hoyos, 8 years ago

Component: ffmpegavformat
Keywords: regression added
Priority: normalimportant

Please either:

  • send your patch - made with git format-patch to the development mailing list. Patches are ignored on this bug tracker.

or

  • test current FFmpeg git head and provide the command line that allows to reproduce the issue together with the complete, uncut console output to make this a valid ticket.

comment:2 by ioeir2, 8 years ago

Resolution: invalid
Status: newclosed

comment:3 by ioeir2, 8 years ago

The proposed fix makes some other RTSP streams to be invalid. It should not be merged into master.

Note: See TracTickets for help on using tickets.