+James On Mon, Jun 17, 2019 at 6:31 PM Chris Cunningham <chcunning...@chromium.org> wrote:
> This behavior was added in 2010 to suport some old (and invalid) ogm > files. > https://github.com/FFmpeg/FFmpeg/commit/81b743eb1026547270b88ac6a5cb451a3907ee94 > > But this makes it possible to change the codec in the later headers, > causing codec to be out of sync with internal avctx (eventually > triggering Abrt). Updating the internal ctx for this degenerate case > was deemed not worth it. See discussion here: > https://patchwork.ffmpeg.org/patch/11983/ > --- > libavformat/oggdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c > index e815f42134..19d77f3107 100644 > --- a/libavformat/oggdec.c > +++ b/libavformat/oggdec.c > @@ -545,7 +545,7 @@ static int ogg_packet(AVFormatContext *s, int *sid, > int *dstart, int *dsize, > ogg->curidx = idx; > os->incomplete = 0; > > - if (os->header) { > + if (!ogg->headers) { > if ((ret = os->codec->header(s, idx)) < 0) { > av_log(s, AV_LOG_ERROR, "Header processing failed: %s\n", > av_err2str(ret)); > return ret; > -- > 2.22.0.410.gd8fdbe21b5-goog > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
