[FFmpeg-devel] [PATCH] set "header" value correctly for all ogg streams

David Conrad lessen42
Mon Feb 8 10:38:07 CET 2010


On Feb 6, 2010, at 4:27 AM, Reimar D?ffinger wrote:

> Hello,
> currently the "header" value used for things like repeating the headers again
> when seeking to the start of the file is only set for one stream.
> Below patch fixes it, thus fixing http://roundup.ffmpeg.org/roundup/ffmpeg/issue1723
> Index: libavformat/oggdec.c
> ===================================================================
> --- libavformat/oggdec.c	(revision 21597)
> +++ libavformat/oggdec.c	(working copy)
> @@ -367,10 +367,10 @@
> 
>     ogg->curidx = idx;
> 
> -    if (os->header < 0){
> +    if (!ogg->headers){
>         int hdr = os->codec->header (s, idx);
> +        os->header = os->seq;
>         if (!hdr){
> -            os->header = os->seq;
>             os->segp = segp;
>             os->psize = psize;
>             ogg->headers = 1;

Looks ok



More information about the ffmpeg-devel mailing list