On 25.01.2021 20:40, emco...@ffastrans.com wrote:
[...]
What you can do instead is to push both identifications, the old one and
the one from the current program into the identification array, this way
the processing chain can be reconstructed. Unforutnately i have never
seen anyone doing thi
Am 2021-01-26 00:34, schrieb Marton Balint:
Uhm guys, it is very bad practice: if you just insert a different
manufacturer name then you just cheat. SMPTE 377 has a clear statement
about what goes to the identification, you cannot just write the infos
from a different program there.
Libavfor
In my previous email the patch got mangled by the web mail interface, so I am
sending it again. Sorry for the duplicate.
This patch allows ffmpeg to capture a specific window instead of the whole
screen. An example:
ffmpeg -f x11grab -window_id 0xa0a -i :0 output.mp4
---
doc/indevs.texi
James Almer (12021-01-25):
> If by agenda you mean trying to solve the hacky state of the library, then
> sure. Otherwise, please get rid of any potential slanderous thoughts you may
> be have about it, because it sounds like you may be having trouble
> distinguishing expressed personal opinions wi
On Tue, 26 Jan 2021, at 11:28, Nicolas George wrote:
> I will accept other people's opinion when it is founded on knowledge.
Unfortunately, that's not how it works.
We need to accept the will of majority of developers, even if you (or someone
else) is not convinced.
--
Jean-Baptiste Kempf - Pr
Signed-off-by: Paul B Mahol
---
doc/filters.texi | 22 +++
libavfilter/Makefile | 1 +
libavfilter/allfilters.c | 1 +
libavfilter/vf_colortemperature.c | 280 ++
4 files changed, 304 insertions(+)
create mode 100644 libavfi
Paul B Mahol wrote:
>+@item temperature
>+Set the temperature in Kelvins. Allowed range is from 1000 to 4.
>+Default value is 6500 K.
I would use the singular "Kelvin", otherwise LGTM.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://f
1b373b41d940e3058cdfb3d17703e23ed665353c made it a bit harder to find
out that a call to avpriv_dv_produce_packet is dead when the DV demuxer
is disabled; too hard for GCC on -O0. So simplify the check a bit.
Signed-off-by: Andreas Rheinhardt
---
libavformat/avidec.c | 2 +-
1 file changed, 1 in
Peter Ross:
> ---
> libavformat/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/Makefile b/libavformat/Makefile
> index 3a8fbcbe5f..c4ce6ceb9b 100644
> --- a/libavformat/Makefile
> +++ b/libavformat/Makefile
> @@ -120,7 +120,7 @@ OBJS-$(CONFIG_AST_DEM
Andreas Rheinhardt:
> This reverts commit 6ac0e7818399a57e4684202bac79f35b3561ad1e.
>
> The mpeg4video parser can reach code that presumes that a certain VLC
> has been initialized; yet Libav did not ensure this and Libav bug #1012
> [1] is about an ensuing crash.
>
> Instead of fixing the root c
On Tue, Jan 26, 2021 at 2:35 AM James Almer wrote:
> On 1/25/2021 2:12 PM, James Almer wrote:
> > On 1/25/2021 11:15 AM, Nuo Mi wrote:
> >> +static int FUNC(ref_pic_list_struct)(CodedBitstreamContext *ctx,
> >> RWContext *rw,
> >> + H266RefPicListStruct *curren
On Tue, Jan 26, 2021 at 5:54 AM James Almer wrote:
> On 1/25/2021 2:12 PM, James Almer wrote:
> > Did you add this just to set active_vps? I thought the
> > idea was to remove all that since it's not a concept defined in h266.
>
> The following should get rid of active_*, but the actual fields ca
Andreas Rheinhardt:
> A macro that expands to a function definition might look like a
> declaration, but it isn't and therefore an extra ';' at the end is
> unnecessary and actually invalid (both GCC and Clang warn about this
> when using -pedantic).
>
> Signed-off-by: Andreas Rheinhardt
> ---
>
On 1/26/2021 10:35 AM, Nuo Mi wrote:
On Tue, Jan 26, 2021 at 5:54 AM James Almer wrote:
On 1/25/2021 2:12 PM, James Almer wrote:
Did you add this just to set active_vps? I thought the
idea was to remove all that since it's not a concept defined in h266.
The following should get rid of activ
On Tue, Jan 26, 2021 at 9:40 PM James Almer wrote:
> On 1/26/2021 10:35 AM, Nuo Mi wrote:
> > On Tue, Jan 26, 2021 at 5:54 AM James Almer wrote:
> >
> >> On 1/25/2021 2:12 PM, James Almer wrote:
> >>> Did you add this just to set active_vps? I thought the
> >>> idea was to remove all that since
On 1/25/2021 11:15 AM, Nuo Mi wrote:
@@ -1637,6 +2047,16 @@ static const SEIMessageTypeDescriptor
cbs_sei_h265_types[] = {
SEI_MESSAGE_TYPE_END
};
+static const SEIMessageTypeDescriptor cbs_sei_h266_types[] = {
You forgot to use this in ff_cbs_sei_find_type() below. Otherwise it
wi
On Tue, Jan 26, 2021 at 10:06 PM James Almer wrote:
> On 1/25/2021 11:15 AM, Nuo Mi wrote:
> > @@ -1637,6 +2047,16 @@ static const SEIMessageTypeDescriptor
> cbs_sei_h265_types[] = {
> > SEI_MESSAGE_TYPE_END
> > };
> >
> > +static const SEIMessageTypeDescriptor cbs_sei_h266_types[] = {
>
On 1/25/2021 11:15 AM, Nuo Mi wrote:
---
libavcodec/vvc_sei.h | 47
1 file changed, 47 insertions(+)
create mode 100644 libavcodec/vvc_sei.h
diff --git a/libavcodec/vvc_sei.h b/libavcodec/vvc_sei.h
new file mode 100644
index 00..90724669d
On Tue, Jan 26, 2021 at 10:36 PM James Almer wrote:
> On 1/25/2021 11:15 AM, Nuo Mi wrote:
> > ---
> > libavcodec/vvc_sei.h | 47
> > 1 file changed, 47 insertions(+)
> > create mode 100644 libavcodec/vvc_sei.h
> >
> > diff --git a/libavcodec/vvc_
On 1/26/2021 12:09 PM, Nuo Mi wrote:
On Tue, Jan 26, 2021 at 10:36 PM James Almer wrote:
On 1/25/2021 11:15 AM, Nuo Mi wrote:
---
libavcodec/vvc_sei.h | 47
1 file changed, 47 insertions(+)
create mode 100644 libavcodec/vvc_sei.h
diff --gi
Quoting Michael Niedermayer (2021-01-25 21:29:21)
> On Sun, Jan 24, 2021 at 02:14:43PM +0100, Anton Khirnov wrote:
> > Quoting Michael Niedermayer (2021-01-23 23:10:54)
> > > Fixes: out of array access
> > > Fixes:
> > > 29202/clusterfuzz-testcase-minimized-ffmpeg_dem_KUX_fuzzer-5112845840809984
>
tis 2021-01-26 klockan 00:34 +0100 skrev Marton Balint:
> On Mon, 25 Jan 2021, emco...@ffastrans.com wrote:
> > Uhm guys, it is very bad practice: if you just insert a different
> > manufacturer name then you just cheat. SMPTE 377 has a clear statement
> > about what goes to the identification, y
---
libavutil/frame.h | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/libavutil/frame.h b/libavutil/frame.h
index 392315f40f..f3e664bbc9 100644
--- a/libavutil/frame.h
+++ b/libavutil/frame.h
@@ -437,7 +437,12 @@ typedef struct AVFrame {
int quality;
/**
-
Quoting Lynne (2021-01-24 18:34:36)
> >>
> >> Some devs (JEEB) wanted reception timestamps and original, overflowed
> >> timestamps for MPEG-TS.
> >> I'd be willing to add a reception timestamp as long as we add an
> >> additional time_base field
> >> and make it independent of the packet's pts f
On 1/26/2021 1:02 PM, Anton Khirnov wrote:
Quoting Lynne (2021-01-24 18:34:36)
Some devs (JEEB) wanted reception timestamps and original, overflowed
timestamps for MPEG-TS.
I'd be willing to add a reception timestamp as long as we add an additional
time_base field
and make it independent of t
Fixes: -2.21166e+304 is outside the range of representable values of type 'long'
Fixes:
29169/clusterfuzz-testcase-minimized-ffmpeg_DEMUXER_fuzzer-5725452796821504
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermay
On Sun, Jan 24, 2021 at 02:12:35PM +0100, Anton Khirnov wrote:
> Quoting Michael Niedermayer (2021-01-23 23:10:51)
> > Signed-off-by: Michael Niedermayer
> > ---
> > libavutil/common.h | 8
>
> This is a public header, so a minor bump+APIchanges entry are needed.
will apply with this
t
On Sun, Jan 24, 2021 at 02:13:07PM +0100, Anton Khirnov wrote:
> Quoting Michael Niedermayer (2021-01-23 23:10:52)
> > Fixes: negation of -2147483648 cannot be represented in type 'int'; cast to
> > an unsigned type to negate this value to itself
> > Fixes:
> > 29053/clusterfuzz-testcase-minimize
Quoting James Almer (2021-01-26 17:07:28)
> On 1/26/2021 1:02 PM, Anton Khirnov wrote:
> > Quoting Lynne (2021-01-24 18:34:36)
>
> Some devs (JEEB) wanted reception timestamps and original, overflowed
> timestamps for MPEG-TS.
> I'd be willing to add a reception timestamp as lo
Andreas Rheinhardt:
> Signed-off-by: Andreas Rheinhardt
> ---
> libavformat/url.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/url.h b/libavformat/url.h
> index e33edf0cb9..1ef5ca35d5 100644
> --- a/libavformat/url.h
> +++ b/libavformat/url.h
> @@ -87,8 +8
On Tue, Jan 26, 2021 at 04:37:38PM +0100, Anton Khirnov wrote:
> Quoting Michael Niedermayer (2021-01-25 21:29:21)
> > On Sun, Jan 24, 2021 at 02:14:43PM +0100, Anton Khirnov wrote:
> > > Quoting Michael Niedermayer (2021-01-23 23:10:54)
> > > > Fixes: out of array access
> > > > Fixes:
> > > > 29
On Sat, Jan 23, 2021 at 11:10:55PM +0100, Michael Niedermayer wrote:
> Fixes: out of array access
> Fixes:
> 29440/clusterfuzz-testcase-minimized-ffmpeg_dem_KUX_fuzzer-5985279812960256.fuzz
>
> Found-by: continuous fuzzing process
> https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
On Sun, Jan 24, 2021 at 02:17:05PM +0100, Anton Khirnov wrote:
> Quoting Michael Niedermayer (2021-01-23 23:10:56)
> > Fixes: Timeout
> > Fixes:
> > 29656/clusterfuzz-testcase-minimized-ffmpeg_dem_FLV_fuzzer-5840098987999232
> >
> > Found-by: continuous fuzzing process
> > https://github.com/goo
Parsers are not forced to use a ParseContext and the other stuff from
parser.h which is just designed to help parsers recombining frames. But
this parser does not do this at all, i.e. the ParseContext is unused.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/dolby_e_parser.c | 4
1 file c
These two functions are always called after another; after all, what
ff_dolby_e_parse_init does is obviously part of parsing the frame header.
Also move the DolbyEHeaderInfo into DBEContext so that parsing the frame
header only needs one struct (both users used a DBEContext immediately
followed by
convert_input, a nontrivial auxiliary function used by both the general
parsing code as well as the decoder itself, has been duplicated in
c7016e35a624a75bb5b82bee932ddfe28d013b3f; this commit removes said
duplication.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/dolby_e.c | 40 ++-
Set the sample rate when parsing the header instead and only copy the
value in the decoder and the parser.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/dolby_e.c| 2 +-
libavcodec/dolby_e.h| 6 ++
libavcodec/dolby_e_parse.c | 6 +-
libavcodec/dolby_e_parser.c | 2 +-
On Sun, Jan 17, 2021 at 12:07:23AM +0100, Michael Niedermayer wrote:
> Fixes: signed integer overflow: 922337203685000 + 900 cannot be
> represented in type 'long long'
> Fixes:
> 26910/clusterfuzz-testcase-minimized-ffmpeg_dem_MPSUB_fuzzer-665448017480908
>
> Found-by: continuous fuzzin
On Sun, Jan 17, 2021 at 12:07:24AM +0100, Michael Niedermayer wrote:
> Fixes: signed integer overflow: -9223372036853488158 - 9000 cannot be
> represented in type 'long long'
> Fixes:
> 26910/clusterfuzz-testcase-minimized-ffmpeg_dem_MPSUB_fuzzer-6696625298866176
>
> Found-by: continuous fuz
On Sun, Jan 17, 2021 at 12:07:25AM +0100, Michael Niedermayer wrote:
> Fixes: signed integer overflow: 65535 * 65535 cannot be represented in type
> 'int'
> Fixes:
> 26910/clusterfuzz-testcase-minimized-ffmpeg_dem_MVI_fuzzer-6649291124899840
>
> Found-by: continuous fuzzing process
> https://gi
On Sun, Jan 17, 2021 at 12:07:27AM +0100, Michael Niedermayer wrote:
> Note, the value is checked a few lines later already
>
> Fixes: signed integer overflow: -440402016 - 1879048064 cannot be represented
> in type 'int'
> Fixes:
> 26910/clusterfuzz-testcase-minimized-ffmpeg_dem_NUT_fuzzer-6603
On Sun, Jan 17, 2021 at 12:07:28AM +0100, Michael Niedermayer wrote:
> Fixes: signed integer overflow: 234080282628234040 * 100 cannot be
> represented in type 'long long'
> Fixes:
> 26910/clusterfuzz-testcase-minimized-ffmpeg_dem_REALTEXT_fuzzer-6649867065753600
>
> Found-by: continuous fuzzing
ff_qp_table_extract() frees the table on failure.
Found-by: Andreas Rheinhardt
---
libavfilter/vf_pp.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/libavfilter/vf_pp.c b/libavfilter/vf_pp.c
index 29ab777e01..19a0f9fc6e 100644
--- a/libavfilter/vf_pp.c
+++ b/libavfilter/vf_pp.c
@@ -145,7 +1
MJPEG does not have a single quantiser scale, so this does not fit into
the intended API use.
This removes the last use of the long-deprecated QP table API.
---
libavcodec/mjpegdec.c | 11 ++-
tests/ref/fate/exif-image-embedded | 6 --
tests/ref/fate/exif-image-jpg
These macros are private and should not be used by external callers.
---
fftools/ffmpeg_opt.c | 13 -
1 file changed, 13 deletions(-)
diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
index bf2eb26246..472016b93c 100644
--- a/fftools/ffmpeg_opt.c
+++ b/fftools/ffmpeg_opt.c
@@ -
On 1/26/2021 2:01 PM, Anton Khirnov wrote:
These macros are private and should not be used by external callers.
---
fftools/ffmpeg_opt.c | 13 -
1 file changed, 13 deletions(-)
diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
index bf2eb26246..472016b93c 100644
--- a/fftoo
On 1/26/2021 1:17 PM, Anton Khirnov wrote:
Quoting James Almer (2021-01-26 17:07:28)
On 1/26/2021 1:02 PM, Anton Khirnov wrote:
Quoting Lynne (2021-01-24 18:34:36)
Some devs (JEEB) wanted reception timestamps and original, overflowed
timestamps for MPEG-TS.
I'd be willing to add a reception
Andreas Rheinhardt:
> The VLC for the macroblock address increment uses nine bits;
> yet there is no code with this length: All codes are either shorter or
> longer. So one can make the table smaller without changing the amount of
> codes that need more than one round of parsing.
>
> Signed-off-by
On 25/01/2021 23:20, Nicolas George wrote:
Mark Thompson (12021-01-25):
Merging the libraries (in source form, orthogonal to merging the
binaries) only makes sense if we are going to continue using the
libavformat internals, and that is exactly the thing we are trying to
get rid of.
"We"?
It
This work is sponsored by Open Broadcast Systems.
Signed-off-by: Paul B Mahol
---
configure | 5 +
doc/protocols.texi | 32 +
libavformat/Makefile| 1 +
libavformat/librist.c | 251
libavformat/protocols.c | 1 +
5 file
This also affected other users of VLCs from msmpeg4dec, namely vc1_block
and wmv2dec.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/msmpeg4.h| 2 +-
libavcodec/msmpeg4dec.c | 20 ++--
libavcodec/vc1_block.c | 12
libavcodec/wmv2dec.c| 20 +++-
Both motion vector tables have the same number of elements, hence one
can inline said number and remove the field containing the number of
elements from the structure.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/msmpeg4data.c | 6 ++
libavcodec/msmpeg4data.h | 2 +-
libavcodec/msmpeg4de
From: Levi Dooley
There was an assumption in the existing code that indentation would not occur
more than once on the same row.
This was a bad assumption. There are examples of 608 streams which call
handle_pac multiple times on the same row with different indentation.
As the code was before th
On Tue, Jan 26, 2021 at 02:21:27PM +0100, Andreas Rheinhardt wrote:
> 1b373b41d940e3058cdfb3d17703e23ed665353c made it a bit harder to find
> out that a call to avpriv_dv_produce_packet is dead when the DV demuxer
> is disabled; too hard for GCC on -O0. So simplify the check a bit.
>
> Signed-off-
---
On 26/01/2021 13:40, James Almer wrote:
On 1/26/2021 10:35 AM, Nuo Mi wrote:
On Tue, Jan 26, 2021 at 5:54 AM James Almer wrote:
On 1/25/2021 2:12 PM, James Almer wrote:
Did you add this just to set active_vps? I thought the
idea was to remove all that since it's not a concept defined in
On Mon, Jan 25, 2021 at 08:36:59PM +, Mark Thompson wrote:
> On 24/01/2021 21:41, Michael Niedermayer wrote:
> > On Sat, Nov 21, 2020 at 06:49:55PM +, Mark Thompson wrote:
> > > On 21/11/2020 17:37, Michael Niedermayer wrote:
> > > > On Sun, Nov 15, 2020 at 09:37:45PM +, Mark Thompson w
Parameter Set_s_ in the title. (That made me look carefully, so I guess it's
good.)
On 22/01/2021 19:34, James Almer wrote:
Defined in H.274, it shares the same value as Active Parameter Sets (which was
removed for codecs that reference H.274).
It's really a rename. The concept of activene
> -Original Message-
> From: Guo, Yejun
> Sent: 2021年1月26日 9:25
> To: ffmpeg-devel@ffmpeg.org
> Cc: Guo, Yejun
> Subject: [PATCH 3/3] dnn_backend_openvino.c: remove extra semicolon
>
> Signed-off-by: Guo, Yejun
> ---
> libavfilter/dnn/dnn_backend_openvino.c | 6 +++---
> 1 file chang
On 1/23/2021 6:13 PM, Marton Balint wrote:
On Tue, 19 Jan 2021, Anton Khirnov wrote:
Quoting Marton Balint (2021-01-18 21:53:30)
On Mon, 18 Jan 2021, Anton Khirnov wrote:
> Quoting Marton Balint (2021-01-10 02:20:45)
>> Signed-off-by: Marton Balint
>> ---
>> libavcodec/flashsv2enc.c | 7
Async depth will allow qsv filter cache few frames, and avoid force
switch and end filter task frame by frame. This change will improve
performance for some multi-task case, for example 1:N transcode(
decode + vpp + encode) with all QSV plugins.
Signed-off-by: Fei Wang
---
libavfilter/qsvvpp.c
> 2021年1月18日 下午2:22，liuqi05 写道：
>
> because there have no Initialization in SegmentTemplate,
> so it will have no init_section for init segment file.
> but in the is_common_init_section_exist function it will be used for
> check to url, url_offset and size, so check init_section
> before use in
From: Jiangjie Gao
---
libavutil/time.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/libavutil/time.c b/libavutil/time.c
index 740afc4785..dd04870983 100644
--- a/libavutil/time.c
+++ b/libavutil/time.c
@@ -90,7 +90,16 @@ int av_usleep(unsigned usec)
#elif HAV
Signed-off-by: Guo, Yejun
---
libavutil/frame.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavutil/frame.h b/libavutil/frame.h
index 392315f40f..1aeafef6de 100644
--- a/libavutil/frame.h
+++ b/libavutil/frame.h
@@ -194,7 +194,7 @@ enum AVFrameSideDataType {
AV_FRAM
Jan 27, 2021, 06:21 by yejun@intel.com:
> Signed-off-by: Guo, Yejun
> ---
> libavutil/frame.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavutil/frame.h b/libavutil/frame.h
> index 392315f40f..1aeafef6de 100644
> --- a/libavutil/frame.h
> +++ b/libavutil/frame
> -Original Message-
> From: ffmpeg-devel On Behalf Of Lynne
> Sent: 2021年1月27日 14:00
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH] libavutil/frame.h: correct typo for
> AVFilmGrainParams in comment
>
> Jan 27, 2021, 06:21 by yejun@intel.com
On 26.01.2021 16:39, Tomas Härdin wrote:
tis 2021-01-26 klockan 00:34 +0100 skrev Marton Balint:
Maybe they should post patches instead of having workarounds? And I
explained, libavformat MXF muxer will still be identifiable, but the
proper field will be used for identifying the SDK used, not
Co
66 matches
Mail list logo
