[FFmpeg-devel] [PATCH] matroskadec: skip invalid tags with no TagName
Aurelien Jacobs
aurel
Wed Nov 3 01:21:07 CET 2010
On Sat, Oct 30, 2010 at 09:54:39AM +0200, Anton Khirnov wrote:
> fixes segfault in issue 2328
> ---
> libavformat/matroskadec.c | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
> index 136839a..b712415 100644
> --- a/libavformat/matroskadec.c
> +++ b/libavformat/matroskadec.c
> @@ -1033,6 +1033,11 @@ static void matroska_convert_tag(AVFormatContext *s, EbmlList *list,
>
> for (i=0; i < list->nb_elem; i++) {
> const char *lang = strcmp(tags[i].lang, "und") ? tags[i].lang : NULL;
> +
> + if (!tags[i].name) {
> + av_log(s, AV_LOG_WARNING, "Skipping tag with no TagName.\n");
> + continue;
> + }
TagName is mandatory according to the spec. Maybe the message should
indicate that the file is invalid ?
Except this, patch looks OK.
Aurel
More information about the ffmpeg-devel
mailing list