diff mbox series

[FFmpeg-devel] avformat/concatdec: fix NEEDS_UNSAFE flag value

Message ID 20211112062911.738629-1-yyoung2001@gmail.com
State Accepted
Commit 23a1f11d02a29b788d441e1b0846a28c0f479df9
Headers show
Series [FFmpeg-devel] avformat/concatdec: fix NEEDS_UNSAFE flag value | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished

Commit Message

Googleplex Nov. 12, 2021, 6:29 a.m. UTC
NEEDS_UNSAFE has the same value as NEEDS_FILE,
causing "duration not allowed if safe" error
while duration directive doesn't require unsafe mode.

Signed-off-by: Googleplex <yyoung2001@gmail.com>
---
 libavformat/concatdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Nicolas George Nov. 14, 2021, 7:38 p.m. UTC | #1
Googleplex (12021-11-12):
> NEEDS_UNSAFE has the same value as NEEDS_FILE,
> causing "duration not allowed if safe" error
> while duration directive doesn't require unsafe mode.
> 
> Signed-off-by: Googleplex <yyoung2001@gmail.com>
> ---
>  libavformat/concatdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Good catch, thanks. Applied.

Regards,
diff mbox series

Patch

diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c
index 2557f38b26..8d80e536d1 100644
--- a/libavformat/concatdec.c
+++ b/libavformat/concatdec.c
@@ -409,7 +409,7 @@  static int concat_read_close(AVFormatContext *avf)
 }
 
 #define MAX_ARGS 3
-#define NEEDS_UNSAFE   (1 << 1)
+#define NEEDS_UNSAFE   (1 << 0)
 #define NEEDS_FILE     (1 << 1)
 #define NEEDS_STREAM   (1 << 2)