Opened 5 years ago

Closed 5 years ago

#611 closed defect (fixed)

missing G726 support for rtp and rtsp

Reported by: thunder.m Owned by:
Priority: normal Component: avformat
Version: git-master Keywords: g726 rtp
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Hi, G726 formats are not supported in RTSP protocol, i created patch with G726 support for rtp and rtsp decoder. Please feel free to modify it, i don't need any copyright, my only point is to get it to ffmpeg main tree.

Attachments (4)

rtp_g726.diff (4.5 KB) - added by thunder.m 5 years ago.
Patch for adding support of G726 decoder to RTP and RTSP streams.
rtsp.log (3.0 KB) - added by thunder.m 5 years ago.
Log from stream (h264 + G726)
0001-libavformat-add-support-for-G726-audio-decoder-in-RT.patch (5.1 KB) - added by thunder.m 5 years ago.
Signed patch
0001-libavformat-add-G726-audio-decoder-in-RTP.patch (5.9 KB) - added by thunder.m 5 years ago.
Updated signed patch

Download all attachments as: .zip

Change History (11)

Changed 5 years ago by thunder.m

Patch for adding support of G726 decoder to RTP and RTSP streams.

Changed 5 years ago by thunder.m

Log from stream (h264 + G726)

comment:1 Changed 5 years ago by thunder.m

Camera with RTSP stream, H264 video and G726 audio is for example AirLive? OD-2025HD, but there are much more such devices, they work without problems in vlc. And now they works also in ffmpeg with this patch.

comment:2 Changed 5 years ago by michael

Please add a license header to the new file and a commit message to the patch (git format-patch -1 will create a patch out of a last local commit with commit message)

thanks

Changed 5 years ago by thunder.m

Signed patch

comment:3 Changed 5 years ago by thunder.m

I also send patch to ffmpeg-devel list, so we have to wait :)

comment:4 Changed 5 years ago by cehoyos

  • Keywords rtp added
  • Version changed from unspecified to git-master

Please copy the license header (the first 20 lines of any file in the directory libavformat) into your new file rtpdec_g726.c, change the second line into something like "RTP G.726 protocol (RFC XXX)" and put your name in the third line.

Changed 5 years ago by thunder.m

Updated signed patch

comment:5 Changed 5 years ago by thunder.m

Ok, i add licence header to new file, i hope that is everything :)

comment:6 Changed 5 years ago by thunder.m

Patch is since now in ffmpeg git, so it should be also in version 0.8.7, thanks, this ticket can be closed.

Last edited 5 years ago by thunder.m (previous) (diff)

comment:7 Changed 5 years ago by cehoyos

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

Per your request merged into release/0.8.

Note: See TracTickets for help on using tickets.