From: Kees Cook <keescook@chromium.org>
To: linux-kernel@vger.kernel.org
Cc: Russell King <linux@arm.linux.org.uk>,
linux-arm-kernel@lists.infradead.org,
Andy Lutomirski <luto@amacapital.net>,
Eric Paris <eparis@redhat.com>,
libseccomp-discuss@lists.sourceforge.net,
Will Drewry <wad@chromium.org>,
James Hogan <james.hogan@imgtec.com>
Subject: [PATCH v2] seccomp: not compatible with ARM OABI
Date: Thu, 7 Nov 2013 09:47:46 -0800 [thread overview]
Message-ID: <20131107174746.GA22344@www.outflux.net> (raw)
Make sure that seccomp filter won't be built when ARM OABI is in use,
since there is work needed to distinguish calling conventions. Until
that is done (which is likely never since OABI is deprecated), make
sure seccomp filter is unavailable in the OABI world.
Signed-off-by: Kees Cook <keescook@chromium.org>
---
v2:
- toggle availability via HAVE_ARCH_SECCOMP_FILTER; James Hogan.
---
arch/arm/Kconfig | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 0a1dc697333c..a0a8590f3609 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -23,7 +23,7 @@ config ARM
select HARDIRQS_SW_RESEND
select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL
select HAVE_ARCH_KGDB
- select HAVE_ARCH_SECCOMP_FILTER
+ select HAVE_ARCH_SECCOMP_FILTER if (AEABI && !OABI_COMPAT)
select HAVE_ARCH_TRACEHOOK
select HAVE_BPF_JIT
select HAVE_CONTEXT_TRACKING
@@ -1735,6 +1735,11 @@ config OABI_COMPAT
in memory differs between the legacy ABI and the new ARM EABI
(only for non "thumb" binaries). This option adds a tiny
overhead to all syscalls and produces a slightly larger kernel.
+
+ The seccomp filter system will not be available when this is
+ selected, since there is no way yet to sensibly distinguish
+ between calling conventions during filtering.
+
If you know you'll be using only pure EABI user space then you
can say N here. If this option is not selected and you attempt
to execute a legacy ABI binary then the result will be
--
1.7.9.5
--
Kees Cook
Chrome OS Security
next reply other threads:[~2013-11-07 17:48 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-07 17:47 Kees Cook [this message]
2013-11-07 18:16 ` [PATCH v2] seccomp: not compatible with ARM OABI Andy Lutomirski
2013-11-07 18:39 ` Kees Cook
2013-11-07 18:56 ` Eric Paris
2013-11-07 19:05 ` Andy Lutomirski
2013-11-08 16:29 ` [libseccomp-discuss] " Paul Moore
2013-11-08 16:39 ` Andy Lutomirski
2013-11-08 18:23 ` Paul Moore
2013-11-07 20:33 ` Dave Martin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131107174746.GA22344@www.outflux.net \
--to=keescook@chromium.org \
--cc=eparis@redhat.com \
--cc=james.hogan@imgtec.com \
--cc=libseccomp-discuss@lists.sourceforge.net \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=luto@amacapital.net \
--cc=wad@chromium.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox