Ticket #611 (closed defect: fixed)

Opened 19 months ago

Last modified 19 months ago

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

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

Change History

Changed 19 months ago by thunder.m

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

Changed 19 months ago by thunder.m

Log from stream (h264 + G726)

comment:1 Changed 19 months 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 19 months 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 19 months ago by thunder.m

Signed patch

comment:3 Changed 19 months ago by thunder.m

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

comment:4 Changed 19 months 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 19 months ago by thunder.m

Updated signed patch

comment:5 Changed 19 months ago by thunder.m

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

comment:6 Changed 19 months 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 19 months ago by thunder.m (previous) (diff)

comment:7 Changed 19 months ago by cehoyos

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

Per your request merged into release/0.8.

Note: See TracTickets for help on using tickets.