On Sun, Jun 30, 2019 at 12:00:43AM +0100, Cameron Cawley wrote:
> ---
> libavformat/rpl.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavformat/rpl.c b/libavformat/rpl.c
> index b4859320f4..579ab4f57e 100644
> --- a/libavformat/rpl.c
> +++ b/libavformat/rpl.c
> @@ -192,7 +192,7 @@ static int rpl_read_header(AVFormatContext *s)
> // samples, though. This code will ignore additional tracks.
> error |= read_line(pb, line, sizeof(line));
> audio_format = read_int(line, &endptr, &error); // audio format ID
> - strcpy(audio_codec, endptr);
> + av_strlcpy(audio_codec, endptr, RPL_LINE_LENGTH);
> if (audio_format) {
> ast = avformat_new_stream(s, NULL);
> if (!ast)
> @@ -203,7 +203,7 @@ static int rpl_read_header(AVFormatContext *s)
> ast->codecpar->channels = read_line_and_int(pb, &error); //
> number of audio channels
> error |= read_line(pb, line, sizeof(line));
> ast->codecpar->bits_per_coded_sample = read_int(line, &endptr,
> &error); // audio bits per sample
> - strcpy(audio_type, endptr);
> + av_strlcpy(audio_type, endptr, RPL_LINE_LENGTH);
> // At least one sample uses 0 for ADPCM, which is really 4 bits
> // per sample.
> if (ast->codecpar->bits_per_coded_sample == 0)please include this in the patch that adds the strcpy() thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
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".
