On Thu, Jun 27, 2019 at 08:56:02PM +0100, Cameron Cawley wrote:
> ---
>  libavformat/rpl.c | 13 ++++++++++++-
>  1 file changed, 12 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/rpl.c b/libavformat/rpl.c
> index dbbcd13b41..b4859320f4 100644
> --- a/libavformat/rpl.c
> +++ b/libavformat/rpl.c
> @@ -121,6 +121,7 @@ static int rpl_read_header(AVFormatContext *s)
>      int error = 0;
>      const char *endptr;
>      char audio_type[RPL_LINE_LENGTH];
> +    char audio_codec[RPL_LINE_LENGTH];
>  
>      uint32_t i;
>  
> @@ -189,7 +190,9 @@ static int rpl_read_header(AVFormatContext *s)
>  
>      // ARMovie supports multiple audio tracks; I don't have any
>      // samples, though. This code will ignore additional tracks.
> -    audio_format = read_line_and_int(pb, &error);  // audio format ID
> +    error |= read_line(pb, line, sizeof(line));
> +    audio_format = read_int(line, &endptr, &error);  // audio format ID

> +    strcpy(audio_codec, endptr);

This should be using a function checking the available space

[...]

thx

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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".

Reply via email to