[FFmpeg-devel] [PATCH 3/8] avutil/timecode: do not trash bits on invalid av_timecode_get_smpte arguments
lance.lmwang at gmail.com
lance.lmwang at gmail.com
Sun Sep 6 16:37:18 EEST 2020
On Sat, Sep 05, 2020 at 08:22:12PM +0200, Marton Balint wrote:
> The function has no way to return error, so let's clip or calculate modulo.
>
> Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
> libavutil/timecode.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/libavutil/timecode.c b/libavutil/timecode.c
> index c0956adadb..806638ddfc 100644
> --- a/libavutil/timecode.c
> +++ b/libavutil/timecode.c
> @@ -84,6 +84,11 @@ uint32_t av_timecode_get_smpte(AVRational rate, int drop, int hh, int mm, int ss
> ff /= 2;
> }
>
> + hh = hh % 24;
> + mm = av_clip(mm, 0, 59);
> + ss = av_clip(ss, 0, 59);
> + ff = ff % 40;
> +
> tc |= drop << 30;
> tc |= (ff / 10) << 28;
> tc |= (ff % 10) << 24;
> --
> 2.26.2
lgtm
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
--
Thanks,
Limin Wang
More information about the ffmpeg-devel
mailing list