[FFmpeg-devel] [PATCHv2] ffplay: dynamically allocate filename buffer
Ganesh Ajjanagadde
gajjanagadde at gmail.com
Sat Sep 26 23:09:52 CEST 2015
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 <gajjanagadde at gmail.com>
---
ffplay.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/ffplay.c b/ffplay.c
index fb4af24..a31db85 100644
--- a/ffplay.c
+++ b/ffplay.c
@@ -290,7 +290,7 @@ typedef struct VideoState {
SDL_Rect last_display_rect;
int eof;
- char filename[1024];
+ char *filename;
int width, height, xleft, ytop;
int step;
@@ -1137,6 +1137,7 @@ static void stream_close(VideoState *is)
sws_freeContext(is->img_convert_ctx);
#endif
sws_freeContext(is->sub_convert_ctx);
+ av_free(is->filename);
av_free(is);
}
@@ -3120,7 +3121,9 @@ 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_strdup(filename);
+ if (!is->filename)
+ goto fail;
is->iformat = iformat;
is->ytop = 0;
is->xleft = 0;
--
2.5.3
More information about the ffmpeg-devel
mailing list