Opened 3 years ago

Closed 3 years ago

Last modified 3 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 3 years ago.
Wireshark dump
logs.txt (260 bytes) - added by ioeir2 3 years ago.
ffplay output
rtsp.c.trac5051.patch (389 bytes) - added by ioeir2 3 years ago.
Patch for fixing the bug

Download all attachments as: .zip

Change History (6)

Changed 3 years ago by ioeir2

Wireshark dump

Changed 3 years ago by ioeir2

ffplay output

Changed 3 years ago by ioeir2

Patch for fixing the bug

comment:1 Changed 3 years ago by cehoyos

  • Component changed from ffmpeg to avformat
  • Keywords regression added
  • Priority changed from normal to important

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 Changed 3 years ago by ioeir2

  • Resolution set to invalid
  • Status changed from new to closed

comment:3 Changed 3 years ago by ioeir2

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.