[FFmpeg-devel] [PATCH 2/5] avformat/mxfdec: reindent code

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mxfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index fdd0dd2a88..02a2a6d97a 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -363,7 +363,7 @@ stati

[FFmpeg-devel] [PATCH 1/5] avcodec/v4l2_m2m_enc: reindent code

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/v4l2_m2m_enc.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/libavcodec/v4l2_m2m_enc.c b/libavcodec/v4l2_m2m_enc.c index 8454e2326c..98b93c61af 100644 --- a/libavcodec/v4l2_m2m_enc.c +++ b/lib

Re: [FFmpeg-devel] [PATCH 1/5] avformat/dashenc: fix invalid pointer access if avio_get_dyn_buf failed

On Wed, Apr 29, 2020 at 06:55:33PM +0200, Nicolas George wrote: > lance.lmw...@gmail.com (12020-04-29): > > Thanks, I catch your point now. Most of existing code haven't return ERROR, > > so I > > choose the same way to process it. If you think it's not OK, we'll change > > more > > code I think.

Re: [FFmpeg-devel] [PATCH 1/5] avformat/dashenc: fix invalid pointer access if avio_get_dyn_buf failed

On Wed, Apr 29, 2020 at 05:39:36PM +0200, Nicolas George wrote: > Limin Wang (12020-04-29): > > yes, avio_write can process zero len with NULL pointer, but here it'll use > > buf+written_len, so > > it's invalid access I think. So what's the broken? Maybe I haven't catch > > your point. > > What

[FFmpeg-devel] [PATCH 4/5] avformat/movenc: check the return value of avio_get_dyn_buf()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/movenc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 32e8109268..1a97e7c883 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5251,10 +5251,11 @@

[FFmpeg-devel] [PATCH 5/5] avformat/movenc: reindent after the previous commit

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/movenc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 1a97e7c883..ad04143f14 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5252,9 +5252,9

[FFmpeg-devel] [PATCH 1/5] avformat/dashenc: fix invalid pointer access if avio_get_dyn_buf failed

From: Limin Wang If an error occurs, avio_get_dyn_buf() will return 0 and buf is NULL, so it's necessary to check the return value for the following code will access the buf pointer with index. In addition, the buf len should be greater than written_len to avoid the buffer overflow access. Si

[FFmpeg-devel] [PATCH 2/5] avformat/matroskaenc: fix invalid pointer access if avio_get_dyn_buf failed

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/matroskaenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 784973a951..f0474da44f 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -374,9 +37

[FFmpeg-devel] [PATCH 3/5] avformat/matroskaenc: reindent after the previous commit

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/matroskaenc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index f0474da44f..41446c3825 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroska

[FFmpeg-devel] [PATCH v2] avformat/srtenc: split write time into function for better readability

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/srtenc.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/libavformat/srtenc.c b/libavformat/srtenc.c index d811a4da0e..c53b227313 100644 --- a/libavformat/srtenc.c +++ b/libavformat/srtenc.c

Re: [FFmpeg-devel] [PATCH] avformat/srtenc: split write time into function for better readability

On Wed, Apr 29, 2020 at 11:12:21AM +0200, Andreas Rheinhardt wrote: > lance.lmw...@gmail.com: > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > libavformat/srtenc.c | 24 ++-- > > 1 file changed, 18 insertions(+), 6 deletions(-) > > > > diff --git a/libavfo

[FFmpeg-devel] [PATCH] avformat/srtenc: split write time into function for better readability

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/srtenc.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/libavformat/srtenc.c b/libavformat/srtenc.c index d811a4da0e..ca127354ed 100644 --- a/libavformat/srtenc.c +++ b/libavformat/srtenc.c

[FFmpeg-devel] [PATCH] avformat/utils: change the duration to int64_t for update_initial_durations

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 3b53f97bee..44109d866d 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -1174,7 +1174,7 @@ static v

[FFmpeg-devel] [PATCH 1/2] avformat/vividas: simplify, use av_rescale_q() instead

From: Limin Wang note it'll cause a small difference in accuracy for the pts, please see the testing result below: $ wget http://samples.ffmpeg.org/archive/all/unknown+unknown+unknown+unknown+5029_paramount_en_1250.viv $ ./ffmpeg -t 0.04 -i ./unknown+unknown+unknown+unknown+5029_paramount_en

[FFmpeg-devel] [PATCH 2/2] avformat/wavenc: simplify, use av_rescale_q() instead

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/wavenc.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavformat/wavenc.c b/libavformat/wavenc.c index f6f5710802..1027f107ee 100644 --- a/libavformat/wavenc.c +++ b/libavformat/wavenc.c @@ -434,10 +434,9

Re: [FFmpeg-devel] [PATCH v2] avformat/url: check url root node when rel include double dot

On Mon, Apr 27, 2020 at 06:51:47PM +0800, Steven Liu wrote: > fix ticket: 8625 > and add testcase into url for double dot corner case I think you need update ./tests/ref/fate/url also. make fate-url SAMPLES=../fate-suite > > Suggested-by: Martin Storsjö > Signed-off-by: Steven Liu > --- > l

[FFmpeg-devel] [PATCH v1] MAINTAINERS: add myself to the general developers list

From: Limin Wang Signed-off-by: Limin Wang --- I have actively contributed to FFmpeg in the past year and should be familiar with the entire code base and the rules of the developers. Up to now, more than one hundred patches have been accepted and pushed with the help of other developers. Howev

[FFmpeg-devel] [PATCH v1 5/6] avformat/smoothstreamingenc: use av_reallocp_array()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/smoothstreamingenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/smoothstreamingenc.c b/libavformat/smoothstreamingenc.c index 0e4f531..d10d5d1 100644 --- a/libavformat/smoothstreamingenc.c +++ b/liba

[FFmpeg-devel] [PATCH v1 4/6] avformat/dashenc: use av_reallocp_array()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/dashenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 96c0ea3..e3e187c 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -1731,7 +1731,7 @@ stat

[FFmpeg-devel] [PATCH v1 2/6] avformat/dashenc: cosmetics

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/dashenc.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index b082536..5fbe4dd 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashe

[FFmpeg-devel] [PATCH v1 6/6] avformat/rtmpproto: use av_reallocp_array()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/rtmpproto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 42aacbd..e23426b 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -163,7 +163,7

[FFmpeg-devel] [PATCH v1 1/6] fftools/ffmpeg: use local variable with same contents directly

From: Limin Wang Signed-off-by: Limin Wang --- fftools/ffmpeg.c | 31 ++- 1 file changed, 18 insertions(+), 13 deletions(-) diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c index d896b14..d2b0e71 100644 --- a/fftools/ffmpeg.c +++ b/fftools/ffmpeg.c @@ -501,32 +501,3

[FFmpeg-devel] [PATCH v1 3/6] avformat/dashenc: use local variable and avoid calculate duration multiple times

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/dashenc.c | 14 +- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 5fbe4dd..96c0ea3 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -1916,6

[FFmpeg-devel] [PATCH v1] avformat/movenc: cosmetics

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/movenc.c | 138 +-- 1 file changed, 69 insertions(+), 69 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 253cff86eb..6b0235f115 100644 --- a/libavformat/movenc.c +++ b

[FFmpeg-devel] [PATCH v4 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

From: Limin Wang The patch will make audio and subtitle packets be marked as AV_PKT_FLAG_KEY. For audio, it'll caused the audio sample to be sync sample. To verify ref/fate/movenc results: 1. Get the movenc test data [lmwang@vpn ffmpeg]$ libavformat/tests/movenc -w && mkdir -p audio_old && mv *

[FFmpeg-devel] [PATCH v4 1/2] avformat/utils: move is_intra_only() to header and rename to ff_is_intra_only()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/internal.h | 2 ++ libavformat/utils.c| 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/internal.h b/libavformat/internal.h index 329b2e9..716e42c 100644 --- a/libavformat/internal.h +++ b/libavfor

[FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/internal.h | 2 + libavformat/mux.c| 7 +++- tests/ref/fate/binsub-movtextenc | 2 +- tests/ref/fate/movenc| 50 +++ tests/ref/fate/sub2video | 86

[FFmpeg-devel] [PATCH v3 1/2] avformat/utils: move is_intra_only() to header and rename to ff_is_intra_only()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/internal.h | 2 ++ libavformat/utils.c| 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/internal.h b/libavformat/internal.h index 329b2e9..716e42c 100644 --- a/libavformat/internal.h +++ b/libavfor

[FFmpeg-devel] [PATCH v2 1/4] fftools/ffmpeg: set AV_PKT_FLAG_KEY for the subtitle packet

From: Limin Wang For better code review, the patch is helping to update the fate tests result so the change of ffmpeg.c willbe deleted in the last patch. Signed-off-by: Limin Wang --- fftools/ffmpeg.c | 1 + tests/ref/fate/binsub-movtextenc | 2 +- tests/ref/fate/sub2video

[FFmpeg-devel] [PATCH v2 1/4] fftools/ffmpeg: set AV_PKT_FLAG_KEY for the subtitle packet

From: Limin Wang For better code review, the patch is helping to update the fate tests result so the change of ffmpeg.c will be deleted in the last patch. Signed-off-by: Limin Wang --- fftools/ffmpeg.c | 1 + tests/ref/fate/binsub-movtextenc | 2 +- tests/ref/fate/sub2video

[FFmpeg-devel] [PATCH v2 3/4] avformat/utils: move is_intra_only() to header and rename to ff_is_intra_only()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/internal.h | 2 ++ libavformat/utils.c| 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/internal.h b/libavformat/internal.h index 329b2e972d..716e42cc3e 100644 --- a/libavformat/internal.h +++ b/li

[FFmpeg-devel] [PATCH v2 4/4] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

From: Limin Wang delete the previous change of ffmpeg.c and movenc.c as mux.c will be in charge of setting AV_PKT_FLAG_KEY. Signed-off-by: Limin Wang --- fftools/ffmpeg.c | 1 - libavformat/internal.h | 2 ++ libavformat/mux.c | 7 ++- libavformat/tests/movenc.c | 1

[FFmpeg-devel] [PATCH v2 2/4] avformat/tests/movenc: set AV_PKT_FLAG_KEY for audio packet

From: Limin Wang For better code review, the patch is helping to update the fate tests result so the change of movenc.c will be deleted in the last patch. Signed-off-by: Limin Wang --- libavformat/tests/movenc.c | 1 + tests/ref/fate/movenc | 50 +++--- 2

[FFmpeg-devel] [PATCH v1] avcodec/utils: use av_rescale()

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 26c038dfd9..005d596dfd 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2229,8 +2229,8 @@ int64_t f

[FFmpeg-devel] [PATCH v1] avcodec/h264_metadata_bsf: add option to delete SEI user data

From: Limin Wang Signed-off-by: Limin Wang --- doc/bitstream_filters.texi | 3 +++ libavcodec/h264_metadata_bsf.c | 19 +++ 2 files changed, 22 insertions(+) diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi index 8fe5b3ad75..652eb4620f 100644 --- a/doc/

[FFmpeg-devel] [PATCH v3 3/3] avcodec/libx264: return error if unknown picture type encountered

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/libx264.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index edd343e..3fa2311 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -476,7 +476,8 @@ static in

[FFmpeg-devel] [PATCH v3 2/3] avcodec/libx264: return immediately if encode_nals return 0

From: Limin Wang x264_encoder_encode can return 0 with nnal 0. As a result, encode_nals will return 0. In this condition, it's better to return 0 immediately to avoid the following unneeded pict_type and flags setting. Signed-off-by: Limin Wang --- libavcodec/libx264.c | 3 +++ 1 file changed,

[FFmpeg-devel] [PATCH v3 1/3] avcodec/libx265: Fix Uninitialized scalar variable

From: Limin Wang return error if unknown picture type encountered Fixes CID 1457234 Signed-off-by: Limin Wang Signed-off-by: Limin Wang --- haven't merge yet, so rebase to the git master libavcodec/libx265.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/libx265.c b/libav

[FFmpeg-devel] [PATCH v1 2/2] avformat/http: Fix for invalid use of av_strtok

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/http.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index c941557..20545ed 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -786,6 +786,7 @@ static int pa

[FFmpeg-devel] [PATCH v1 1/2] avformat/ftp: Fix for invalid use of av_strtok

From: Limin Wang By the av_strtok() description: * On the first call to av_strtok(), s should point to the string to * parse, and the value of saveptr is ignored. In subsequent calls, s * should be NULL, and saveptr should be unchanged since the previous * call. Signed-off-by: Limin Wang --

[FFmpeg-devel] [PATCH v2] avcodec/bsf: simplify the code

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/bsf.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/libavcodec/bsf.c b/libavcodec/bsf.c index b9fc771..3cacbd4 100644 --- a/libavcodec/bsf.c +++ b/libavcodec/bsf.c @@ -534,11 +534,7 @@ int av_bsf_list_parse_str

[FFmpeg-devel] [PATCH v1] avcodec/bsf: simplify the code

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/bsf.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/libavcodec/bsf.c b/libavcodec/bsf.c index 7b96183e64..c4c939c205 100644 --- a/libavcodec/bsf.c +++ b/libavcodec/bsf.c @@ -533,7 +533,7 @@ end: int av_b

[FFmpeg-devel] [PATCH v1 1/2] avcodec/nvenc_h264: add spatial_aq and temporal_aq option to consistent with nvenc_hevc

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/nvenc_h264.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/nvenc_h264.c b/libavcodec/nvenc_h264.c index 479155fe15..a2e77e67d0 100644 --- a/libavcodec/nvenc_h264.c +++ b/libavcodec/nvenc_h264.c @@ -99,7 +99,9 @@ static

[FFmpeg-devel] [PATCH v1 2/2] avcodec/nvenc_hevc: add spatial-aq and temporal-aq option to consistent with nvenc_h264

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/nvenc_hevc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/nvenc_hevc.c b/libavcodec/nvenc_hevc.c index 7c9b3848f1..d06cb0d6b0 100644 --- a/libavcodec/nvenc_hevc.c +++ b/libavcodec/nvenc_hevc.c @@ -96,7 +96,9 @@ static

[FFmpeg-devel] [PATCH v1] fftools/ffprobe: Use format specifier PRId64 instead of %lld

From: Limin Wang Signed-off-by: Limin Wang --- fftools/ffprobe.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index e3f221f560..a32bc529d9 100644 --- a/fftools/ffprobe.c +++ b/fftools/ffprobe.c @@ -408,7 +408,7 @@ stati

[FFmpeg-devel] [PATCH v1 2/4] avfilter/af_loudnorm: Add compact mode to simplify stats parsing

From: Limin Wang Signed-off-by: Limin Wang --- doc/filters.texi | 2 +- libavfilter/af_loudnorm.c | 12 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/doc/filters.texi b/doc/filters.texi index 738a40df4e..19171d48dc 100644 --- a/doc/filters.texi +++ b/doc/

[FFmpeg-devel] [PATCH v1 3/4] avfilter/af_loudnorm: Add support for two pass stats for measure

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/af_loudnorm.c | 41 ++- 1 file changed, 40 insertions(+), 1 deletion(-) diff --git a/libavfilter/af_loudnorm.c b/libavfilter/af_loudnorm.c index 3012aa2471..380d0570d4 100644 --- a/libavfilter/af_lou

[FFmpeg-devel] [PATCH v1 4/4] avfilter/ebur128: prefer to use variable instead of type for sizeof

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/ebur128.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavfilter/ebur128.c b/libavfilter/ebur128.c index c8986fb5e1..ca2fca0066 100644 --- a/libavfilter/ebur128.c +++ b/libavfilter/ebur128.c @@

[FFmpeg-devel] [PATCH v1 1/4] avfilter/af_loudnorm: Add file option for the measured stats

From: Limin Wang Signed-off-by: Limin Wang --- doc/filters.texi | 3 +++ libavfilter/af_loudnorm.c | 54 +-- 2 files changed, 55 insertions(+), 2 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index 3931d8d79e..738a40df4e 100644 ---

[FFmpeg-devel] [PATCH v1] avformat/hlsenc: return media_url directly if failed to get seperator

From: Limin Wang Fix ticket: 8606 Signed-off-by: Limin Wang --- libavformat/hlsenc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index a29b2625e4..e34833fd89 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -1251,6 +1251,8 @@

[FFmpeg-devel] [PATCH v1] doc/utils: add more examples for valid time duration

From: Limin Wang Signed-off-by: Limin Wang --- doc/utils.texi | 9 + 1 file changed, 9 insertions(+) diff --git a/doc/utils.texi b/doc/utils.texi index 05a2f81626..ca599f145b 100644 --- a/doc/utils.texi +++ b/doc/utils.texi @@ -126,6 +126,15 @@ The following examples are all valid time

[FFmpeg-devel] [PATCH v3] avformat/dashenc: remove the arbitrary restrictions for filename

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/dashenc.c | 26 +- 1 file changed, 9 insertions(+), 17 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 94d463972a..903fdd0aa6 100644 --- a/libavformat/dashenc.c +++ b/libavformat/da

[FFmpeg-devel] [PATCH v2] avformat/dashenc: remove the arbitrary restrictions for filename

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/dashenc.c | 28 1 file changed, 12 insertions(+), 16 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 94d4639..f80f6f2 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashe

[FFmpeg-devel] [PATCH v1] avformat/dashenc: use av_asprintf()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/dashenc.c | 23 ++- 1 file changed, 6 insertions(+), 17 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 94d4639..0c281a4 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @

[FFmpeg-devel] [PATCH v1] avformat/movenc: cosmetics

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/movenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 6db09b7..7140ad3 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -2711,7 +2711,7 @@ static in

[FFmpeg-devel] [PATCH v3 2/2] avcodec/mpeg12enc: Support mpeg2 encoder profile with const options

From: Limin Wang Signed-off-by: Limin Wang --- doc/encoders.texi | 8 libavcodec/mpeg12enc.c | 22 +- libavcodec/mpegvideo.h | 1 + 3 files changed, 26 insertions(+), 5 deletions(-) diff --git a/doc/encoders.texi b/doc/encoders.texi index e23b6b3..5022b94 10

[FFmpeg-devel] [PATCH v2 2/2] avcodec/mpeg12enc: Support mpeg2 encoder profile with const options

From: Limin Wang make setting profile more user friendly Signed-off-by: Limin Wang --- doc/encoders.texi | 8 libavcodec/mpeg12enc.c | 22 +- libavcodec/mpegvideo.h | 1 + 3 files changed, 26 insertions(+), 5 deletions(-) diff --git a/doc/encoders.texi b/do

[FFmpeg-devel] [PATCH v1] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mux.c| 16 tests/ref/fate/binsub-movtextenc | 2 +- tests/ref/fate/movenc| 50 +++ tests/ref/fate/sub2video | 86 4 files changed

[FFmpeg-devel] [PATCH v1] avformat/concat: priv_data should be freed internally

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/concat.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavformat/concat.c b/libavformat/concat.c index ea3bc1dfde..cfe14760eb 100644 --- a/libavformat/concat.c +++ b/libavformat/concat.c @@ -75,7 +75,6 @@ static av_cold int conca

[FFmpeg-devel] [PATCH v1 2/2] avcodec/mpeg12enc: Support mpeg2 encoder profile with const options

From: Limin Wang make setting profile more user friendly Signed-off-by: Limin Wang --- doc/encoders.texi | 8 libavcodec/mpeg12enc.c | 19 ++- libavcodec/mpegvideo.h | 1 + 3 files changed, 23 insertions(+), 5 deletions(-) diff --git a/doc/encoders.texi b/doc/e

[FFmpeg-devel] [PATCH v1 1/2] avcodec/mpeg12enc: Use FF_PROFILE_MPEG2_xxx macros

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/mpeg12enc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/mpeg12enc.c b/libavcodec/mpeg12enc.c index b460e78a59..643ba8165a 100644 --- a/libavcodec/mpeg12enc.c +++ b/libavcodec/mpeg12enc.c @@ -162

[FFmpeg-devel] [PATCH v1 2/3] avformat/libsrt: clarify option description for timeout and latency unit of measurement

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/libsrt.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/libsrt.c b/libavformat/libsrt.c index 2d6fc4b..c01d639 100644 --- a/libavformat/libsrt.c +++ b/libavformat/libsrt.c @@ -96,8 +96,8 @@

[FFmpeg-devel] [PATCH v1 3/3] avformat/libsrt: change open_timeout to int64_t to avoid integer overflow

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/libsrt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/libsrt.c b/libavformat/libsrt.c index c01d639..4de575b 100644 --- a/libavformat/libsrt.c +++ b/libavformat/libsrt.c @@ -361,7 +361,7 @@ static int

[FFmpeg-devel] [PATCH v1 1/3] doc/protocols: clarify timeout and latency unit of measurement for SRT

From: Limin Wang Signed-off-by: Limin Wang --- doc/protocols.texi | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/doc/protocols.texi b/doc/protocols.texi index e510019..7aa7585 100644 --- a/doc/protocols.texi +++ b/doc/protocols.texi @@ -1255,7 +1255,7 @@ option

[FFmpeg-devel] [PATCH v1 2/2] avfilter/vf_subtitles: add force_style option for ass subtitle filter

From: Limin Wang Signed-off-by: Limin Wang --- doc/filters.texi | 4 libavfilter/vf_subtitles.c | 23 +++ 2 files changed, 27 insertions(+) diff --git a/doc/filters.texi b/doc/filters.texi index 328e984e92..d26d90d1bb 100644 --- a/doc/filters.texi +++ b/doc/

[FFmpeg-devel] [PATCH v1 1/2] avfilter/vf_subtitles: remove unneeded ifdef

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_subtitles.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index a3b4029af4..2f312b0ba8 100644 --- a/libavfilter/vf_subtitles.c +++ b/libavfilter/vf_s

[FFmpeg-devel] [PATCH v2 3/4] avformat: add subtitle support in master playlist m3u8

From: Limin Wang Test with the following command for the webvtt subtitle: $ ./ffmpeg -y -i input_with_subtitle.mkv \ -b:v:0 5250k -c:v h264 -pix_fmt yuv420p -profile:v main -level 4.1 \ -b:a:0 256k \ -c:s webvtt -c:a mp2 -ar 48000 -ac 2 -map 0:v -map 0:a:0 -map 0:s:0 \ -f hls -var_stream_map

[FFmpeg-devel] [PATCH v3 2/3] avfilter/vf_mix: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_mix.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_mix.c b/libavfilter/vf_mix.c index 9e1ae79..9bb1b7a 100644 --- a/libavfilter/vf_mix.c +++ b/libavfilter/vf_mix.c @@ -108,7 +108,10 @@ static

[FFmpeg-devel] [PATCH v3 1/3] avfilter/af_acrossover: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/af_acrossover.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavfilter/af_acrossover.c b/libavfilter/af_acrossover.c index f70c50b..002f378 100644 --- a/libavfilter/af_acrossover.c +++ b/libavfilter/af_acros

[FFmpeg-devel] [PATCH v3 3/3] avfilter/af_adelay: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/af_adelay.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavfilter/af_adelay.c b/libavfilter/af_adelay.c index c964777..6ac81c2 100644 --- a/libavfilter/af_adelay.c +++ b/libavfilter/af_adelay.c @@ -155,7 +1

[FFmpeg-devel] [PATCH v2 2/3] avfilter/vf_mix: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_mix.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_mix.c b/libavfilter/vf_mix.c index 9e1ae79..e47f7e2 100644 --- a/libavfilter/vf_mix.c +++ b/libavfilter/vf_mix.c @@ -108,7 +108,8 @@ static av

[FFmpeg-devel] [PATCH v2 1/3] avfilter/af_acrossover: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/af_acrossover.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavfilter/af_acrossover.c b/libavfilter/af_acrossover.c index f70c50b..91f77cc 100644 --- a/libavfilter/af_acrossover.c +++ b/libavfilter/af_acros

[FFmpeg-devel] [PATCH v2 3/3] avfilter/af_adelay: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/af_adelay.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavfilter/af_adelay.c b/libavfilter/af_adelay.c index c964777..2bb05e7 100644 --- a/libavfilter/af_adelay.c +++ b/libavfilter/af_adelay.c @@ -155,7 +1

[FFmpeg-devel] [PATCH v1 2/4] avformat/hlsplaylist: simplify code for checking whether the string is empty

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/hlsplaylist.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/hlsplaylist.c b/libavformat/hlsplaylist.c index 9cbd02353f..56244496c0 100644 --- a/libavformat/hlsplaylist.c +++ b/libavformat/hlsplayl

[FFmpeg-devel] [PATCH v1 1/4] avformat/hlsenc: remove the first slash of the relative path line in the master m3u8 file

From: Limin Wang Please testing with the following command: ./ffmpeg -y -i input.mkv \ -b:v:0 5250k -c:v h264 -pix_fmt yuv420p -profile:v main -level 4.1 \ -b:a:0 256k \ -c:a mp2 -ar 48000 -ac 2 -map 0:v -map 0:a:0\ -f hls -var_stream_map "v:0,a:0" \ -master_pl_name master.m3u8 -t 300 -hls_t

[FFmpeg-devel] [PATCH v1 3/4] avformat: add subtitle support in master playlist m3u8

From: Limin Wang Test with the following command for the webvtt subtitle: $ ./ffmpeg -y -i input_with_subtitle.mkv \ -b:v:0 5250k -c:v h264 -pix_fmt yuv420p -profile:v main -level 4.1 \ -b:a:0 256k \ -c:s webvtt -c:a mp2 -ar 48000 -ac 2 -map 0:v -map 0:a:0 -map 0:s:0 \ -f hls -var_stream_map

[FFmpeg-devel] [PATCH v1 4/4] avformat/hlsenc: use av_asprintf()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/hlsenc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index d7b9c0e20a..694dab42dd 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -2950,13 +2950,11 @

[FFmpeg-devel] [PATCH v1 1/3] avfilter/af_acrossover: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/af_acrossover.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/af_acrossover.c b/libavfilter/af_acrossover.c index 20d1d2bda9..c9404b95cd 100644 --- a/libavfilter/af_acrossover.c +++ b/libavfilter/af_a

[FFmpeg-devel] [PATCH v1 3/3] avfilter/af_adelay: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/af_adelay.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/af_adelay.c b/libavfilter/af_adelay.c index c9647771f2..7f498ba190 100644 --- a/libavfilter/af_adelay.c +++ b/libavfilter/af_adelay.c @@ -155,

[FFmpeg-devel] [PATCH v1 2/3] avfilter/vf_mix: Check sscanf() return value

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_mix.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_mix.c b/libavfilter/vf_mix.c index 9e1ae79e00..e47f7e2301 100644 --- a/libavfilter/vf_mix.c +++ b/libavfilter/vf_mix.c @@ -108,7 +108,8 @@ sta

[FFmpeg-devel] [PATCH v1 2/2] avfilter/vf_showinfo: limit the max number of timecode

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_showinfo.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_showinfo.c b/libavfilter/vf_showinfo.c index 71534d1fa8..b0b0051357 100644 --- a/libavfilter/vf_showinfo.c +++ b/libavfilter/vf_showin

[FFmpeg-devel] [PATCH v1 1/2] avfilter/vf_showinfo: check if the s12m data size is valid

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_showinfo.c | 4 1 file changed, 4 insertions(+) diff --git a/libavfilter/vf_showinfo.c b/libavfilter/vf_showinfo.c index 79b79db2d3..71534d1fa8 100644 --- a/libavfilter/vf_showinfo.c +++ b/libavfilter/vf_showinfo.c @@ -318,6 +3

[FFmpeg-devel] [PATCH v1 1/4] avcodec/xsubdec: replace data_size with got_sub_ptr for better readability

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/xsubdec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/xsubdec.c b/libavcodec/xsubdec.c index 05c4a64ee5..c1c6e11dc3 100644 --- a/libavcodec/xsubdec.c +++ b/libavcodec/xsubdec.c @@ -46,7 +46,7 @@ sta

[FFmpeg-devel] [PATCH v1 4/4] avcodec/dvbsubdec: replace data_size with got_sub_ptr for better readability

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/dvbsubdec.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/dvbsubdec.c b/libavcodec/dvbsubdec.c index 6e7e13b6eb..f63a1f3bf6 100644 --- a/libavcodec/dvbsubdec.c +++ b/libavcodec/dvbsubdec.c @@ -

[FFmpeg-devel] [PATCH v1 3/4] avcodec/libzvbi-teletextdec: replace data_size with got_sub_ptr for better readability

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/libzvbi-teletextdec.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/libzvbi-teletextdec.c b/libavcodec/libzvbi-teletextdec.c index fe6504ff8a..8031b02286 100644 --- a/libavcodec/libzvbi-teletextdec.

[FFmpeg-devel] [PATCH v1 2/4] avcodec/pgssubdec: replace data_size with got_sub_ptr for better readability

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/pgssubdec.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/pgssubdec.c b/libavcodec/pgssubdec.c index 7fadcb8b4b..9c59a2297f 100644 --- a/libavcodec/pgssubdec.c +++ b/libavcodec/pgssubdec.c @@ -

[FFmpeg-devel] [PATCH v1] avformat/mxfdec: use av_asprintf()

From: Limin Wang Signed-off-by: Limin Wang --- libavformat/mxfdec.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 9113e2a09c..3374f36a88 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -2017,7 +2017,7

[FFmpeg-devel] [PATCH v1] fftools/ffmpeg: set AV_PKT_FLAG_KEY for the subtitle packet

From: Limin Wang This fixes webvtt segment output. Please testing with the following command and check the output: ./ffmpeg -i ../fate-suite/sub/MicroDVD_capability_tester.srt -f segment -segment_time 10 \ -segment_list_size 0 -segment_list sub.m3u8 -segment_format webvtt -scodec webvtt s

Re: [FFmpeg-devel] [PATCH v9 2/4] avcodec/hevc_sei: add support for user data unregistered SEI message

On Wed, Mar 18, 2020 at 11:12:54AM -0300, James Almer wrote: > On 3/17/2020 11:37 PM, Limin Wang wrote: > > On Wed, Mar 18, 2020 at 12:13:33AM +, Derek Buitenhuis wrote: > >> On 17/03/2020 23:11, Limin Wang wrote: > >>> The user data unregistered allows arbitrary data to be carried in the > >>

Re: [FFmpeg-devel] [PATCH v9 2/4] avcodec/hevc_sei: add support for user data unregistered SEI message

On Wed, Mar 18, 2020 at 10:42:12AM +, Kieran Kunhya wrote: > On Wed, 18 Mar 2020 at 02:38, Limin Wang wrote: > > > On Wed, Mar 18, 2020 at 12:13:33AM +, Derek Buitenhuis wrote: > > > On 17/03/2020 23:11, Limin Wang wrote: > > > > The user data unregistered allows arbitrary data to be carr

[FFmpeg-devel] [PATCH v9 4/4] avcodec/h264: create user data unregistered SEI side data for H.264

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/h264_sei.c | 30 +-- libavcodec/h264_sei.h | 2 + libavcodec/h264_slice.c | 14 tests/ref/fate/mov-zombie | 195 ++ 4 files changed, 171 insertions(+), 70 deletions(-) d

[FFmpeg-devel] [PATCH v9 1/4] avutil: add AV_FRAME_DATA_SEI_UNREGISTERED side data type

From: Limin Wang Signed-off-by: Limin Wang --- haven't got further feedback after 2 month, so I rebased the patches on ffmpeg master, please the core developer help to push to master. doc/APIchanges | 6 ++ libavutil/frame.c | 1 + libavutil/frame.h | 8 libavutil/versio

[FFmpeg-devel] [PATCH v9 3/4] avfilter/vf_showinfo: display H.26[45] user data unregistered sei message

From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_showinfo.c | 37 + 1 file changed, 37 insertions(+) diff --git a/libavfilter/vf_showinfo.c b/libavfilter/vf_showinfo.c index 79b79db..36b9bf1 100644 --- a/libavfilter/vf_showinfo.c +++ b/libavfilt

[FFmpeg-devel] [PATCH v1 2/2] avfilter/vf_overlay: remove the duplicated framesync option

From: Limin Wang After applied the patch, we can change default eof_action from repeat to pass still. ./ffmpeg -y -filter_complex "movie=./input.mkv,setpts=PTS-STARTPTS[main];movie=./overlay.mkv:loop=2,setpts=PTS-STARTPTS[overlay];[main][overlay]overlay=10:10: enable='between(t,0,25):eof_acti

[FFmpeg-devel] [PATCH v1] avfilter/src_movie: Fix the loop function of dynamic logo

From: Limin Wang The following command will attempt to create the input and overlay test sequence for you. ./ffmpeg -f lavfi -i color=white:duration=100:r=25:size=1280x720 input.mkv ./ffmpeg -f lavfi -i "testsrc=duration=5:size=320x240:rate=25" overlay.mkv Please try with below command and co

[FFmpeg-devel] [PATCH v9 2/4] avcodec/hevc_sei: add support for user data unregistered SEI message

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/hevc_sei.c | 31 +++ libavcodec/hevc_sei.h | 6 ++ libavcodec/hevcdec.c| 14 ++ tests/ref/fate/hevc-monochrome-crop | 3 +++ 4 files changed, 54 in

[FFmpeg-devel] [PATCH v1] avcodec/libx264: return immediately if encode_nals return 0

From: Limin Wang x264_encoder_encode can return 0 with nnal 0. As a result, encode_nals will return 0. In this condition, it's better to return 0 immediately to avoid the following unneeded pict_type and flags setting. Signed-off-by: Limin Wang --- libavcodec/libx264.c | 3 +++ 1 file changed,

[FFmpeg-devel] [PATCH v2 2/2] avcodec/libx264: return error if unknown picture type encountered

From: Limin Wang Signed-off-by: Limin Wang --- libavcodec/libx264.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index ca8f6c0..4fef0be 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -471,7 +471,8 @@ static in

Re: [FFmpeg-devel] [PATCH v8 1/4] avutil: add AV_FRAME_DATA_SEI_UNREGISTERED side data type

ping the patchset for further comments. On Wed, Jan 08, 2020 at 11:46:00PM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > doc/APIchanges | 3 +++ > libavutil/frame.c | 1 + > libavutil/frame.h | 8 > libavutil/version.h | 2 +- >

Re: [FFmpeg-devel] [PATCH v3] avutil/frame: Use av_realloc_array()

ping On Thu, Dec 26, 2019 at 08:33:38AM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavutil/frame.c | 7 ++- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/libavutil/frame.c b/libavutil/frame.c > index e403809..2e76

<    8   9   10   11   12   13   14   15   16   17   >