On Sun, Dec 09, 2018 at 02:38:55PM +0100, Paul B Mahol wrote:
> Fixes #5155
>
> Signed-off-by: Paul B Mahol <one...@gmail.com>
> ---
> libavformat/mxfenc.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> index 3549b4137d..b3c8dc43bd 100644
> --- a/libavformat/mxfenc.c
> +++ b/libavformat/mxfenc.c
> @@ -2726,6 +2726,12 @@ static int mxf_write_packet(AVFormatContext *s,
> AVPacket *pkt)
> }
> }
>
> + if (st->codecpar->sample_aspect_ratio.num &&
> st->codecpar->sample_aspect_ratio.den) {
> + av_reduce_q(&sc->aspect_ratio,
> + av_mul_q(st->codecpar->sample_aspect_ratio,
> + av_make_q(st->codecpar->width,
> st->codecpar->height)), INT_MAX);
> + }AVRational uses int, so av_reduce* should not be needed if INT_MAX is ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Opposition brings concord. Out of discord comes the fairest harmony. -- Heraclitus
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
