[FFmpeg-devel] [PATCH] ffplay: dynamically allocate filename buffer

Ganesh Ajjanagadde gajjanag at mit.edu
Sat Sep 26 23:10:46 CEST 2015


On Sat, Sep 26, 2015 at 4:50 PM, Timo Rothenpieler
<timo at rothenpieler.org> wrote:
>> @@ -3120,7 +3121,10 @@ static VideoState *stream_open(const char *filename, AVInputFormat *iformat)
>>      is = av_mallocz(sizeof(VideoState));
>>      if (!is)
>>          return NULL;
>> -    av_strlcpy(is->filename, filename, sizeof(is->filename));
>> +    is->filename = av_malloc(strlen(filename)+1);
>> +    if (!is->filename)
>> +        goto fail;
>> +    strcpy(is->filename, filename);
>
> av_strdup

thanks, updated patch.

>
>>      is->iformat = iformat;
>>      is->ytop    = 0;
>>      is->xleft   = 0;
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list