On Thu, Jun 21, 2018 at 10:09:14PM +0000, Carl Eugen Hoyos wrote: > ffmpeg | branch: master | Carl Eugen Hoyos <ceffm...@gmail.com> | Fri Jun 22 > 00:08:13 2018 +0200| [40b7e6071815fc416a4efc5dc1616f5460a3aacb] | committer: > Carl Eugen Hoyos > > lavf/amr: Make the heuristic for auto-detection even stricter. > > Fixes ticket #7270. > > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=40b7e6071815fc416a4efc5dc1616f5460a3aacb > --- > > libavformat/amr.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavformat/amr.c b/libavformat/amr.c > index f954803d46..6cc06bceac 100644 > --- a/libavformat/amr.c > +++ b/libavformat/amr.c > @@ -201,7 +201,7 @@ static int amrnb_probe(AVProbeData *p) > i++; > } > } > - if (valid > 100 && valid > invalid) > + if (valid > 100 && valid >> 4 > invalid) > return AVPROBE_SCORE_EXTENSION / 2 + 1; > return 0; > } > @@ -258,8 +258,8 @@ static int amrwb_probe(AVProbeData *p) > i++; > } > } > - if (valid > 100 && valid > invalid) > - return AVPROBE_SCORE_EXTENSION / 2 - 1; > + if (valid > 100 && valid >> 4 > invalid) > + return AVPROBE_SCORE_EXTENSION / 2 + 1;
This breaks detecting the h263 file: https://samples.ffmpeg.org/V-codecs/h263/h263-raw/messenger.h263 [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is what and why we do it that matters, not just one of them.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
