Re: [FFmpeg-devel] [PATCH] avutil/twofish: Fixed decryption

2021-12-17 Thread Andreas Rheinhardt
Sebastian Kirmayer: > The previous implementation swapped the two halves of the plaintext. The > existing tests only decrypted data with a plaintext of all zeroes, which is > not affected by swapping the halves. Tests which detect the old buggy behavior > have been added. > > Signed-off-by:

Re: [FFmpeg-devel] [PATCH 01/11] avcodec/Makefile: Remove superfluous avformat->DNXHD dependencies

2021-12-17 Thread Andreas Rheinhardt
Andreas Rheinhardt: > There is no mxfenc dependency any more since commit > b9a26b9d55f77ebbff3596e46be54bb5fed469d3. > Also remove a dnxhddata.h inclusion in mxfenc that was forgotten > in the very same commit. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/Makefile | 2 -- >

Re: [FFmpeg-devel] [PATCH 1/2] tests/dnn: Make DNN tests regular libavfilter tests

2021-12-17 Thread Andreas Rheinhardt
Andreas Rheinhardt: > They test libavfilter internal API, so they should be libavfilter > test programs (which implies: linked statically to libavfilter > to access internal APIs and linked normally (statically or dynamically > depending upon the build configuration) against all the other libs). >

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/ass: Faster ff_ass_add_rect()

2021-12-17 Thread Andreas Rheinhardt
Michael Niedermayer: > Signed-off-by: Michael Niedermayer > --- > libavcodec/ass.c | 32 ++-- > libavcodec/ass.h | 7 +++ > 2 files changed, 33 insertions(+), 6 deletions(-) > > diff --git a/libavcodec/ass.c b/libavcodec/ass.c > index 725e4d42ba1..d0a4d678bb4

[FFmpeg-devel] [PATCH 1/4] avcodec/ass: Faster ff_ass_add_rect()

2021-12-17 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/ass.c | 32 ++-- libavcodec/ass.h | 7 +++ 2 files changed, 33 insertions(+), 6 deletions(-) diff --git a/libavcodec/ass.c b/libavcodec/ass.c index 725e4d42ba1..d0a4d678bb4 100644 --- a/libavcodec/ass.c +++

[FFmpeg-devel] [PATCH 2/4] avcodec/ccaption_dec: Use ff_ass_add_rect2()

2021-12-17 Thread Michael Niedermayer
Fixes: Timeout Fixes: 42258/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_CCAPTION_fuzzer-5540144118104064 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/ccaption_dec.c | 3 ++- 1 file

[FFmpeg-devel] [PATCH 4/4] avcodec/tiff: Use ff_set_dimensions() for setting up mjpeg context dimensions

2021-12-17 Thread Michael Niedermayer
Fixes: OOM Fixes: 42263/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TIFF_fuzzer-565619113984 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/tiff.c | 15 --- 1 file

[FFmpeg-devel] [PATCH 3/4] avcodec/tiff: Pass max_pixels to mjpeg context

2021-12-17 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/tiff.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c index 870e0666aa3..9af602eef70 100644 --- a/libavcodec/tiff.c +++ b/libavcodec/tiff.c @@ -2155,6 +2155,7 @@ static av_cold int

Re: [FFmpeg-devel] [PATCH v4 3/3] libavdevice/avfoundation.m: Allow to select devices by unique ID.

2021-12-17 Thread Marvin Scholz
On 17 Dec 2021, at 16:12, Romain Beauxis wrote: This is the third patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. Changes: v2: None v3: * Switched unique ID to use system-prodvided unique ID * Implemented unique IDs for screen capture v4:

Re: [FFmpeg-devel] [PATCH v11 1/2] avformat/imf: Demuxer

2021-12-17 Thread Andreas Rheinhardt
p...@sandflow.com: > From: Pierre-Anthony Lemieux > > Signed-off-by: Pierre-Anthony Lemieux > --- > > Notes: > The IMF demuxer accepts as input an IMF CPL. The assets referenced by the > CPL can be > contained in multiple deliveries, each defined by an ASSETMAP file: > >

Re: [FFmpeg-devel] [PATCH v10 1/2] avformat/imf: Demuxer

2021-12-17 Thread Pierre-Anthony Lemieux
On Fri, Dec 17, 2021 at 12:54 PM Lynne wrote: > > Dec 17, 2021, 3:25 PM by an...@khirnov.net: > > > Quoting Pierre-Anthony Lemieux (2021-12-15 21:41:25) > > > >> On Wed, Dec 15, 2021 at 12:20 PM Anton Khirnov wrote: > >> > > >> > Quoting Pierre-Anthony Lemieux (2021-12-15 01:17:26) > >> > > > >

Re: [FFmpeg-devel] [PATCH v10 1/2] avformat/imf: Demuxer

2021-12-17 Thread Lynne
Dec 17, 2021, 3:25 PM by an...@khirnov.net: > Quoting Pierre-Anthony Lemieux (2021-12-15 21:41:25) > >> On Wed, Dec 15, 2021 at 12:20 PM Anton Khirnov wrote: >> > >> > Quoting Pierre-Anthony Lemieux (2021-12-15 01:17:26) >> > > > >> > > > Now the question is whether a malicious attacker can

Re: [FFmpeg-devel] [PATCH] Add 32 bit-per-sample capability to FLAC encoder

2021-12-17 Thread Martijn van Beurden
Op vr 17 dec. 2021 om 12:43 schreef Michael Niedermayer : > > +sub->coefs[i] = sub->coefs[i]*0.98; > > This is ugly, the amount of actual overflow should be known at this point > so no arbitrary downscale should be needed here

[FFmpeg-devel] [PATCH v4 5/5] avfilter: add vf_yadif_videotoolbox

2021-12-17 Thread Aman Karmani
From: Aman Karmani deinterlaces CVPixelBuffers, i.e. AV_PIX_FMT_VIDEOTOOLBOX frames for example, an interlaced mpeg2 video can be decoded by avcodec, uploaded into a CVPixelBuffer, deinterlaced by Metal, and then encoded to h264 by VideoToolbox as follows: ffmpeg \

[FFmpeg-devel] [PATCH v4 4/5] avfilter: add metal utilities

2021-12-17 Thread Aman Karmani
From: Aman Karmani Reviewed-by: Ridley Combs Signed-off-by: Aman Karmani --- libavfilter/metal/utils.h | 35 +++ libavfilter/metal/utils.m | 73 +++ 2 files changed, 108 insertions(+) create mode 100644 libavfilter/metal/utils.h create

[FFmpeg-devel] [PATCH v4 3/5] avutil: add obj-c helpers into header-only include

2021-12-17 Thread Aman Karmani
From: Aman Karmani Reviewed-by: Ridley Combs Signed-off-by: Aman Karmani --- libavutil/objc.h | 32 1 file changed, 32 insertions(+) create mode 100644 libavutil/objc.h diff --git a/libavutil/objc.h b/libavutil/objc.h new file mode 100644 index

[FFmpeg-devel] [PATCH v4 2/5] build: detect Metal.framework and build .metal files

2021-12-17 Thread Aman Karmani
From: Aman Karmani Reviewed-by: Ridley Combs Signed-off-by: Aman Karmani --- .gitignore | 3 +++ configure | 12 +++- ffbuild/common.mak | 9 + 3 files changed, 23 insertions(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index

[FFmpeg-devel] [PATCH v4 1/5] avfilter/vf_yadif_cuda: simplify filter definition

2021-12-17 Thread Aman Karmani
From: Aman Karmani Signed-off-by: Aman Karmani Signed-off-by: Philip Langdale --- libavfilter/vf_yadif_cuda.c | 19 +-- 1 file changed, 1 insertion(+), 18 deletions(-) diff --git a/libavfilter/vf_yadif_cuda.c b/libavfilter/vf_yadif_cuda.c index da1ab5a8ff..685b8a2035 100644

Re: [FFmpeg-devel] [PATCH v3 2/5] build: detect Metal.framework and build .metal files

2021-12-17 Thread Aman Karmani
On Fri, Dec 17, 2021 at 12:54 AM Martin Storsjö wrote: > On Thu, 16 Dec 2021, Aman Karmani wrote: > > > From: Aman Karmani > > > > Signed-off-by: Aman Karmani > > --- > > .gitignore | 3 +++ > > configure | 8 +++- > > ffbuild/common.mak | 9 + > > 3 files changed, 19

Re: [FFmpeg-devel] [PATCH 000/279 v2] New channel layout API

2021-12-17 Thread James Almer
On 12/17/2021 4:20 PM, Marton Balint wrote: On Thu, 16 Dec 2021, James Almer wrote: On 12/16/2021 9:04 PM, Marton Balint wrote:  On Thu, 16 Dec 2021, James Almer wrote:  Resending the first two patches only, since this is meant to  show the implementation of one of the several

Re: [FFmpeg-devel] [PATCH 000/279 v2] New channel layout API

2021-12-17 Thread Marton Balint
On Thu, 16 Dec 2021, James Almer wrote: On 12/16/2021 9:04 PM, Marton Balint wrote: On Thu, 16 Dec 2021, James Almer wrote: Resending the first two patches only, since this is meant to show the implementation of one of the several suggestions made in the previous set that need to

Re: [FFmpeg-devel] [PATCH v3] avformat/movenc: fix duration in mdhd box

2021-12-17 Thread Martin Storsjö
On Fri, 17 Dec 2021, Zhao Zhili wrote: mvhd and tkhd present the post-editlist duration, while mdhd should have the pre-editlist duration. Regression since c2424b1f3. --- libavformat/movenc.c | 2 +- tests/ref/fate/movenc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) Thanks, this

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxf: support MCA audio information

2021-12-17 Thread Marton Balint
On Fri, 17 Dec 2021, Marc-Antoine ARNAUD wrote: Hi all, Can I have an update on this patch submission ? Is something required to be done before it can be merged ? New channel layout API is on its way, which makes in-demuxer channel reordering uneeded. Therefore the reordering option

Re: [FFmpeg-devel] [PATCH 000/279 v2] New channel layout API

2021-12-17 Thread Marton Balint
On Fri, 17 Dec 2021, Michael Niedermayer wrote: On Fri, Dec 17, 2021 at 01:04:19AM +0100, Marton Balint wrote: On Thu, 16 Dec 2021, James Almer wrote: Resending the first two patches only, since this is meant to show the implementation of one of the several suggestions made in the

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxf: support MCA audio information

2021-12-17 Thread Marc-Antoine ARNAUD
Hi all, Can I have an update on this patch submission ? Is something required to be done before it can be merged ? Thanks you, Marc-Antoine Le ven. 3 déc. 2021 à 10:57, Marc-Antoine Arnaud a écrit : > > --- > doc/demuxers.texi | 10 ++ > libavformat/mxf.h | 3 + >

Re: [FFmpeg-devel] [PATCH v5 1/3] libavdevice/avfoundation.m: use AudioConvert, extend supported formats

2021-12-17 Thread Andreas Rheinhardt
Romain Beauxis: > (Apologies for the previously mangled message, I’m still honing my tools > here!) > Things like this and the changelog do not really belong into a commit message to be committed; the proper place for such comments is between the "---" and the "libavdevice/avfoundation.m | 250

[FFmpeg-devel] [PATCH v5 3/3] libavdevice/avfoundation.m: Allow to select devices by unique ID

2021-12-17 Thread Romain Beauxis
(Apologies for the previously mangled message, I’m still honing my tools here!) This is the third patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. Changes: v2: None v3: * Switched unique ID to use system-prodvided unique ID * Implemented unique

[FFmpeg-devel] [PATCH v5 2/3] libavdevice/avfoundation.m: Replace mutex-based concurrency handling in avfoundation.m by a thread-safe fifo queue with maximum length

2021-12-17 Thread Romain Beauxis
(Apologies for the previously mangled message, I’m still honing my tools here!) This is the second patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. Changes: v2: None v3: Switched queue implementation to CMSimpleQueue v4: None v5: Fix

[FFmpeg-devel] [PATCH v5 1/3] libavdevice/avfoundation.m: use AudioConvert, extend supported formats

2021-12-17 Thread Romain Beauxis
(Apologies for the previously mangled message, I’m still honing my tools here!) This is the first patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. Changes: * v2: None * v3: None * v4: None * v5: Fix indentation/wrapping * Implement support for

Re: [FFmpeg-devel] [PATCH 3/4] avcodec/alacdsp: fix integer overflow in decorrelate_stereo()

2021-12-17 Thread Michael Niedermayer
On Fri, Dec 03, 2021 at 06:19:55PM +0100, Michael Niedermayer wrote: > Fixes: signed integer overflow: -16777216 * 131 cannot be represented in type > 'int' > Fixes: > 23835/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALAC_fuzzer-5669943160078336 > Fixes: >

[FFmpeg-devel] [PATCH v4 3/3] libavdevice/avfoundation.m: Allow to select devices by unique ID.

2021-12-17 Thread Romain Beauxis
This is the third patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. Changes: v2: None v3: * Switched unique ID to use system-prodvided unique ID * Implemented unique IDs for screen capture v4: Cleanup This patch adds a unique ID to

[FFmpeg-devel] [PATCH v4 2/3] libavdevice/avfoundation.m: Replace mutex-based concurrency handling in avfoundation.m by a thread-safe fifo queue with maximum length.

2021-12-17 Thread Romain Beauxis
This is the second patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. Changes: v2: None v3: Switched queue implementation to CMSimpleQueue v4: None This patch fixes the concurrency model. Avfoundation runs its own producing thread to send produced

[FFmpeg-devel] [PATCH v4 1/3] libavdevice/avfoundation.m: use AudioConvert, extend supported formats

2021-12-17 Thread Romain Beauxis
This is the first patch of a series of 3 that cleanup and enhance the avfoundation implementation for libavdevice. Changes: * v2: None * v3: None * v4: None * Implement support for AudioConverter * Switch to AudioConverter's API to convert unsupported PCM formats (non-interleaved,

Re: [FFmpeg-devel] [PATCH v10 1/2] avformat/imf: Demuxer

2021-12-17 Thread Anton Khirnov
Quoting Pierre-Anthony Lemieux (2021-12-15 21:41:25) > On Wed, Dec 15, 2021 at 12:20 PM Anton Khirnov wrote: > > > > Quoting Pierre-Anthony Lemieux (2021-12-15 01:17:26) > > > > > > > > Now the question is whether a malicious attacker can craft those two > > > > files to get access to anything

[FFmpeg-devel] [PATCH v3] avformat/movenc: fix duration in mdhd box

2021-12-17 Thread Zhao Zhili
mvhd and tkhd present the post-editlist duration, while mdhd should have the pre-editlist duration. Regression since c2424b1f3. --- libavformat/movenc.c | 2 +- tests/ref/fate/movenc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/movenc.c

[FFmpeg-devel] [PATCH v2] avformat/movenc: fix duration in mdhd box

2021-12-17 Thread Zhao Zhili
It's the duration of this media, should not take account of editlist. --- libavformat/movenc.c | 2 +- tests/ref/fate/movenc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 0f912dd012..f76ef430cf 100644 ---

Re: [FFmpeg-devel] [PATCH] avformat/movenc: fix duration in mdhd box

2021-12-17 Thread Martin Storsjö
On Fri, 17 Dec 2021, Zhao Zhili wrote: It's the duration of this media, should not take account of editlist. --- libavformat/movenc.c | 9 +++-- tests/ref/fate/movenc | 2 +- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index

[FFmpeg-devel] [PATCH] avformat/movenc: fix duration in mdhd box

2021-12-17 Thread Zhao Zhili
It's the duration of this media, should not take account of editlist. --- libavformat/movenc.c | 9 +++-- tests/ref/fate/movenc | 2 +- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 0f912dd012..643beac6f2 100644 ---

Re: [FFmpeg-devel] [PATCH 001/279 v2] Add a new channel layout API

2021-12-17 Thread James Almer
On 12/16/2021 8:27 PM, Marton Balint wrote: On Thu, 16 Dec 2021, James Almer wrote: - Added a 16 byte fixed array to AVChannelCustom to give custom  labels to channels in Custom order layouts. These labes will  be used by the helpers when querying channels by name or  describing the

Re: [FFmpeg-devel] [PATCH 21/24] ffmpeg_mux: split of_write_packet()

2021-12-17 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2021-12-17 00:42:25) >> Anton Khirnov: >>> diff --git a/fftools/ffmpeg_mux.c b/fftools/ffmpeg_mux.c >>> index d4b674c9e2..e97ec8ab93 100644 >>> --- a/fftools/ffmpeg_mux.c >>> +++ b/fftools/ffmpeg_mux.c >>> @@ -102,39 +102,12 @@ static int

Re: [FFmpeg-devel] [PATCH] Add 32 bit-per-sample capability to FLAC encoder

2021-12-17 Thread Michael Niedermayer
On Thu, Dec 16, 2021 at 08:43:21PM +0100, Martijn van Beurden wrote: [...] > @@ -972,7 +1007,25 @@ static int encode_residual_ch(FlacEncodeContext *s, int > ch) > for (i = 0; i < sub->order; i++) > sub->coefs[i] = coefs[sub->order-1][i]; > > -if (s->bps_code * 4 +

Re: [FFmpeg-devel] [PATCH 05/24] ffmpeg: move some muxing-related code into a separate file

2021-12-17 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2021-12-17 02:55:43) >> Anton Khirnov: >>> This is a first step towards making muxers more independent from the >>> rest of the code. >>> --- >>> fftools/Makefile | 11 +- >>> fftools/ffmpeg.c | 273 ++-- >>>

Re: [FFmpeg-devel] [PATCH] Add 32 bit-per-sample capability to FLAC encoder

2021-12-17 Thread Michael Niedermayer
On Thu, Dec 16, 2021 at 10:09:13PM +0100, Paul B Mahol wrote: > use wavpack, this is bad patch. the flac specification supports 32bit "FLAC supports from 4 to 32 bits per sample" so theres nothing fundamental wrong with adding 32bit support thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH v1] avformat/aviobuf: ffio_copy_url_options

2021-12-17 Thread Zane van Iperen
Will apply tomorrow unless there are objections. On 15/12/21 10:35, p...@sandflow.com wrote: From: Pierre-Anthony Lemieux Signed-off-by: Pierre-Anthony Lemieux --- Notes: Refactors save_avio_options() from dashdec.c and hls.c into a common ffio_copy_url_options() in

Re: [FFmpeg-devel] [PATCH 000/279 v2] New channel layout API

2021-12-17 Thread Michael Niedermayer
On Fri, Dec 17, 2021 at 01:04:19AM +0100, Marton Balint wrote: > > > On Thu, 16 Dec 2021, James Almer wrote: > > > Resending the first two patches only, since this is meant to > > show the implementation of one of the several suggestions made > > in the previous set that need to be discussed

Re: [FFmpeg-devel] [PATCH 21/24] ffmpeg_mux: split of_write_packet()

2021-12-17 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2021-12-17 00:42:25) > Anton Khirnov: > > diff --git a/fftools/ffmpeg_mux.c b/fftools/ffmpeg_mux.c > > index d4b674c9e2..e97ec8ab93 100644 > > --- a/fftools/ffmpeg_mux.c > > +++ b/fftools/ffmpeg_mux.c > > @@ -102,39 +102,12 @@ static int queue_packet(OutputFile *of,

Re: [FFmpeg-devel] [PATCH 16/24] ffmpeg: access output file chapters through a wrapper

2021-12-17 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2021-12-17 00:08:07) > Anton Khirnov: > > Avoid accessing the muxer context directly, as this will become > > forbidden in future commits. > > --- > > fftools/ffmpeg.c | 15 +-- > > fftools/ffmpeg.h | 2 ++ > > fftools/ffmpeg_mux.c | 7 +++ > >

[FFmpeg-devel] [PATCH] avformat/movenc: fix assert failure in get_cluster_duration()

2021-12-17 Thread Zhao Zhili
When editlist is disabled, the workaournd method of shift dts to zero and increase the first sample duration doesn't work if the timestamp is larger than mp4 spec restriction (e.g., sample_delta in stts entry). Further more, it triggers get_cluster_duration() assert failure. This patch will drop

Re: [FFmpeg-devel] [PATCH 11/24] ffmpeg: set want_sdp when initializing the muxer

2021-12-17 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2021-12-16 22:40:31) > Anton Khirnov: > > Allows making the variable local to ffmpeg_mux. > > --- > > fftools/ffmpeg.c | 9 + > > fftools/ffmpeg.h | 1 - > > fftools/ffmpeg_mux.c | 5 + > > 3 files changed, 6 insertions(+), 9 deletions(-) > > > >

Re: [FFmpeg-devel] [PATCH 05/24] ffmpeg: move some muxing-related code into a separate file

2021-12-17 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2021-12-17 02:55:43) > Anton Khirnov: > > This is a first step towards making muxers more independent from the > > rest of the code. > > --- > > fftools/Makefile | 11 +- > > fftools/ffmpeg.c | 273 ++-- > > fftools/ffmpeg.h

Re: [FFmpeg-devel] [PATCH] avcodec/hevcdec: fix the order of in-loop filters

2021-12-17 Thread 黄哲雄
To verify if this problem has been fixed, the following bitstream can be used: https://streams.videolan.org/ffmpeg/incoming/BlowingBubbles_ctu16.265 This bitstream is encoded by the following command line: x265 --input BlowingBubbles_416x240_50.yuv --input-res 416x240 --fps 50 -s 16 -o bs.265 The

Re: [FFmpeg-devel] [PATCH v3 2/5] build: detect Metal.framework and build .metal files

2021-12-17 Thread Martin Storsjö
On Thu, 16 Dec 2021, Aman Karmani wrote: From: Aman Karmani Signed-off-by: Aman Karmani --- .gitignore | 3 +++ configure | 8 +++- ffbuild/common.mak | 9 + 3 files changed, 19 insertions(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index

[FFmpeg-devel] [PATCH] avcodec/hevcdec: fix the order of in-loop filters

2021-12-17 Thread Zhexiong Huang
As we know, in-loop filters in HEVC are applied by the following ordered steps: firstly deblocking filter, then sample adaptive offset filter if enabled. However, in the current version of FFmpeg, pixels without being deblocking-filtered could be used by SAO filter when CTU size is 16 and chroma