[FFmpeg-devel] [PATCH] avformat/mxfdec: Do not process zero modified_date timestamp.

Marton Balint cus at passwd.hu
Sat Dec 22 20:50:59 EET 2018



On Sat, 22 Dec 2018, Michael Niedermayer wrote:

> This causes windows to fail as the timestamp is outside its supported range
> Fixes regression & fate

LGTM, exactly my patch.

Thanks,
Marton


>
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
> libavformat/mxfdec.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index f5e3a736e5..6e96107498 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -2590,7 +2590,7 @@ static int64_t mxf_timestamp_to_int64(uint64_t timestamp)
> 
> #define SET_TS_METADATA(pb, name, var, str) do { \
>     var = avio_rb64(pb); \
> -    if ((ret = avpriv_dict_set_timestamp(&s->metadata, name, mxf_timestamp_to_int64(var))) < 0) \
> +    if (var && (ret = avpriv_dict_set_timestamp(&s->metadata, name, mxf_timestamp_to_int64(var))) < 0) \
>         return ret; \
> } while (0)
> 
> -- 
> 2.20.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list