From patchwork Sat Dec 3 17:13:51 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Marton Balint X-Patchwork-Id: 1670 Delivered-To: ffmpegpatchwork@gmail.com Received: by 10.103.65.86 with SMTP id o83csp871500vsa; Sat, 3 Dec 2016 09:14:14 -0800 (PST) X-Received: by 10.28.209.67 with SMTP id i64mr2740962wmg.48.1480785254576; Sat, 03 Dec 2016 09:14:14 -0800 (PST) Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id x69si7145102wme.157.2016.12.03.09.14.14; Sat, 03 Dec 2016 09:14:14 -0800 (PST) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 2F77F689E0C; Sat, 3 Dec 2016 19:14:03 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from iq.passwd.hu (iq.passwd.hu [217.27.212.140]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 36F98689D87 for ; Sat, 3 Dec 2016 19:13:57 +0200 (EET) Received: from localhost (localhost [127.0.0.1]) by iq.passwd.hu (Postfix) with ESMTP id C7F12102F9B; Sat, 3 Dec 2016 18:14:05 +0100 (CET) X-Virus-Scanned: amavisd-new at passwd.hu Received: from iq.passwd.hu ([127.0.0.1]) by localhost (iq.passwd.hu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6rvc8GfF5R-P; Sat, 3 Dec 2016 18:14:03 +0100 (CET) Received: from bluegene.passwd.hu (localhost [127.0.0.1]) by iq.passwd.hu (Postfix) with ESMTP id 99237100BA6; Sat, 3 Dec 2016 18:14:03 +0100 (CET) From: Marton Balint To: ffmpeg-devel@ffmpeg.org Date: Sat, 3 Dec 2016 18:13:51 +0100 Message-Id: <20161203171351.16229-1-cus@passwd.hu> X-Mailer: git-send-email 2.10.0 In-Reply-To: References: Subject: [FFmpeg-devel] [PATCH] lavfi/formats: allow unknown channel layouts by default X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: Marton Balint MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Since the default in the libav fork is to only allow known layouts, making unknown layouts allowed by default here can be a security risk for filters directly merged from libav. However, usually it is simple to detect such cases, use of av_get_channel_layout_nb_channels is a good indicator, so I suggest we change this regardless. See http://ffmpeg.org/pipermail/ffmpeg-devel/2016-November/203204.html. This patch indirectly adds unknown channel layout support for filters where query_formats is not specified: abench afifo ainterleave anullsink apad aperms arealtime aselect asendcmd asetnsamples asetpts asettb ashowinfo asyncts azmq And it removes .query_format callback from filters where it was only there to support unknown layouts, as this is now the default: aloop ametadata anull asidedata asplit atrim Signed-off-by: Marton Balint --- libavfilter/af_anull.c | 1 - libavfilter/f_loop.c | 1 - libavfilter/f_metadata.c | 1 - libavfilter/f_sidedata.c | 1 - libavfilter/formats.c | 5 ----- libavfilter/formats.h | 10 ---------- libavfilter/split.c | 1 - libavfilter/trim.c | 1 - 8 files changed, 21 deletions(-) diff --git a/libavfilter/af_anull.c b/libavfilter/af_anull.c index fff456e..1d0af45 100644 --- a/libavfilter/af_anull.c +++ b/libavfilter/af_anull.c @@ -46,7 +46,6 @@ static const AVFilterPad avfilter_af_anull_outputs[] = { AVFilter ff_af_anull = { .name = "anull", .description = NULL_IF_CONFIG_SMALL("Pass the source unchanged to the output."), - .query_formats = ff_query_formats_all, .inputs = avfilter_af_anull_inputs, .outputs = avfilter_af_anull_outputs, }; diff --git a/libavfilter/f_loop.c b/libavfilter/f_loop.c index 69bfb10..5a32807 100644 --- a/libavfilter/f_loop.c +++ b/libavfilter/f_loop.c @@ -233,7 +233,6 @@ AVFilter ff_af_aloop = { .priv_size = sizeof(LoopContext), .priv_class = &aloop_class, .uninit = auninit, - .query_formats = ff_query_formats_all, .inputs = ainputs, .outputs = aoutputs, }; diff --git a/libavfilter/f_metadata.c b/libavfilter/f_metadata.c index 24deccf..1f613ec 100644 --- a/libavfilter/f_metadata.c +++ b/libavfilter/f_metadata.c @@ -373,7 +373,6 @@ AVFilter ff_af_ametadata = { .priv_class = &ametadata_class, .init = init, .uninit = uninit, - .query_formats = ff_query_formats_all, .inputs = ainputs, .outputs = aoutputs, .flags = AVFILTER_FLAG_SUPPORT_TIMELINE_GENERIC, diff --git a/libavfilter/f_sidedata.c b/libavfilter/f_sidedata.c index 4863cc9..45d246b 100644 --- a/libavfilter/f_sidedata.c +++ b/libavfilter/f_sidedata.c @@ -139,7 +139,6 @@ AVFilter ff_af_asidedata = { .priv_size = sizeof(SideDataContext), .priv_class = &asidedata_class, .init = init, - .query_formats = ff_query_formats_all, .inputs = ainputs, .outputs = aoutputs, .flags = AVFILTER_FLAG_SUPPORT_TIMELINE_GENERIC, diff --git a/libavfilter/formats.c b/libavfilter/formats.c index 20f45e3..0661618 100644 --- a/libavfilter/formats.c +++ b/libavfilter/formats.c @@ -596,11 +596,6 @@ static int default_query_formats_common(AVFilterContext *ctx, int ff_default_query_formats(AVFilterContext *ctx) { - return default_query_formats_common(ctx, ff_all_channel_layouts); -} - -int ff_query_formats_all(AVFilterContext *ctx) -{ return default_query_formats_common(ctx, ff_all_channel_counts); } diff --git a/libavfilter/formats.h b/libavfilter/formats.h index 77981f5..7e05ece 100644 --- a/libavfilter/formats.h +++ b/libavfilter/formats.h @@ -187,16 +187,6 @@ av_warn_unused_result int ff_default_query_formats(AVFilterContext *ctx); /** - * Set the formats list to all existing formats. - * This function behaves like ff_default_query_formats(), except it also - * accepts channel layouts with unknown disposition. It should only be used - * with audio filters. - */ -av_warn_unused_result -int ff_query_formats_all(AVFilterContext *ctx); - - -/** * Create a list of supported formats. This is intended for use in * AVFilter->query_formats(). * diff --git a/libavfilter/split.c b/libavfilter/split.c index 6cf4ab1..6630087 100644 --- a/libavfilter/split.c +++ b/libavfilter/split.c @@ -144,7 +144,6 @@ AVFilter ff_af_asplit = { .priv_class = &asplit_class, .init = split_init, .uninit = split_uninit, - .query_formats = ff_query_formats_all, .inputs = avfilter_af_asplit_inputs, .outputs = NULL, .flags = AVFILTER_FLAG_DYNAMIC_OUTPUTS, diff --git a/libavfilter/trim.c b/libavfilter/trim.c index 9daaeaf..1dbbabb 100644 --- a/libavfilter/trim.c +++ b/libavfilter/trim.c @@ -365,7 +365,6 @@ AVFilter ff_af_atrim = { .name = "atrim", .description = NULL_IF_CONFIG_SMALL("Pick one continuous section from the input, drop the rest."), .init = init, - .query_formats = ff_query_formats_all, .priv_size = sizeof(TrimContext), .priv_class = &atrim_class, .inputs = atrim_inputs,