[FFmpeg-devel] [PATCH 6/8] avformat/movenc: Do not pass AVCodecParameters in avpriv_request_sample

James Almer jamrial at gmail.com
Wed Jun 27 21:54:06 EEST 2018


On 6/27/2018 3:11 PM, Michael Niedermayer wrote:
> Fixes: out of array read
> Fixes: ffmpeg_crash_8.avi
> 
> Found-by: Thuan Pham, Marcel Böhme, Andrew Santosa and Alexandru Razvan Caciulescu with AFLSmart
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/movenc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/movenc.c b/libavformat/movenc.c
> index 00567db586..2603b9c95f 100644
> --- a/libavformat/movenc.c
> +++ b/libavformat/movenc.c
> @@ -429,7 +429,7 @@ static int handle_eac3(MOVMuxContext *mov, AVPacket *pkt, MOVTrack *track)
>  
>              if (hdr->substreamid == info->num_ind_sub + 1) {
>                  //info->num_ind_sub++;
> -                avpriv_request_sample(track->par, "Multiple independent substreams");
> +                avpriv_request_sample(mov, "Multiple independent substreams");

mov->fc

>                  ret = AVERROR_PATCHWELCOME;
>                  goto end;
>              } else if (hdr->substreamid < info->num_ind_sub ||
> @@ -439,7 +439,7 @@ static int handle_eac3(MOVMuxContext *mov, AVPacket *pkt, MOVTrack *track)
>              }
>          } else {
>              if (hdr->substreamid != 0) {
> -                avpriv_request_sample(track->par, "Multiple dependent substreams");
> +                avpriv_request_sample(mov, "Multiple dependent substreams");

Same.

>                  ret = AVERROR_PATCHWELCOME;
>                  goto end;
>              }
> 



More information about the ffmpeg-devel mailing list