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

2015-09-26 Thread Ganesh Ajjanagadde
On Sat, Sep 26, 2015 at 4:50 PM, Timo Rothenpieler wrote: >> @@ -3120,7 +3121,10 @@ static VideoState *stream_open(const char *filename, >> AVInputFormat *iformat) >> is = av_mallocz(sizeof(VideoState)); >> if (!is) >> return NULL; >> -

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

2015-09-26 Thread Timo Rothenpieler
> @@ -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 =

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

2015-09-26 Thread Ganesh Ajjanagadde
filename was set to an arbitrary 1024 characters. ffplay would thus be unable to play files whose name exceeds that arbitrary threshold. This patch dynamically allocates and frees the filename buffer to remove such limitations. Signed-off-by: Ganesh Ajjanagadde ---