[FFmpeg-devel] avformat/lrcdec: fix losing opening bracket
Michael Niedermayer
michael at niedermayer.cc
Wed Jul 11 20:37:11 EEST 2018
On Tue, Jul 10, 2018 at 08:10:00PM +0530, Umair Khan wrote:
> Hi,
>
> Patch attached.
>
> -Umair
> lrcdec.c | 3 +++
> 1 file changed, 3 insertions(+)
> 6782b07bfa45139361c15527ee5fb78e5893b9ba 0001-avformat-lrcdec-fix-losing-opening-bracket.patch
> From 95a5d2e5575c5f895e4251775a48f7deb4c86124 Mon Sep 17 00:00:00 2001
> From: Umair Khan <omerjerk at gmail.com>
> Date: Tue, 10 Jul 2018 19:51:41 +0530
> Subject: [PATCH 1/1] avformat/lrcdec: fix losing opening bracket
>
> Fixes #7255
>
> Signed-off-by: Umair Khan <omerjerk at gmail.com>
> ---
> libavformat/lrcdec.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/libavformat/lrcdec.c b/libavformat/lrcdec.c
> index f4e9a4efa9..d1e28aff87 100644
> --- a/libavformat/lrcdec.c
> +++ b/libavformat/lrcdec.c
> @@ -68,6 +68,9 @@ static int64_t count_ts(const char *p)
> (p[offset] >= '0' && p[offset] <= '9'))) {
> offset++;
> } else {
> + if (in_brackets) {
> + --offset;
> + }
> break;
this is probably ok but it doesnt fix this completely
consider 2 [[
or a [123
The code should probably remember the last position after a correctly
parsed timestamp or maybe use sscanf() to parse these timestamps
Also it seems the code doesnt handle timestamps in the middle of lines
(not related to your patch though)
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180711/14155796/attachment.sig>
More information about the ffmpeg-devel
mailing list