[FFmpeg-devel] [PATCH 5/5] avformat/mp3dec: require 90% of a file to be mp3 if only 1 mp3 frame is found in sequence

[Aman Gupta]

On Sat, May 12, 2018 at 9:33 AM, Michael Niedermayer <michael at niedermayer.cc
> wrote:

> This eliminates several low score detections of non mp3 files
>

Thanks for working on this. Spurious mp3 detection has been a real headache.

Aman


>
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/mp3dec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/mp3dec.c b/libavformat/mp3dec.c
> index d419f8c9fe..ef884934e1 100644
> --- a/libavformat/mp3dec.c
> +++ b/libavformat/mp3dec.c
> @@ -112,7 +112,7 @@ static int mp3_read_probe(AVProbeData *p)
>      else if(ff_id3v2_match(buf0, ID3v2_DEFAULT_MAGIC) &&
> 2*ff_id3v2_tag_len(buf0) >= p->buf_size)
>                             return p->buf_size < PROBE_BUF_MAX ?
> AVPROBE_SCORE_EXTENSION / 4 : AVPROBE_SCORE_EXTENSION - 2;
>      else if(first_frames > 1 && whole_used) return 5;
> -    else if(max_frames>=1 && p->buf_size < 2*max_framesizes) return 1;
> +    else if(max_frames>=1 && p->buf_size < 10*max_framesizes) return 1;
>      else                   return 0;
>  //mpegps_mp3_unrecognized_format.mpg has max_frames=3
>  }
> --
> 2.17.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>