From: Daniel Micay <danielmicay@gmail.com>
To: Linus Torvalds <torvalds@linux-foundation.org>,
kernel test robot <xiaolong.ye@intel.com>,
Ananth N Mavinakayanahalli <ananth@linux.vnet.ibm.com>,
Anil S Keshavamurthy <anil.s.keshavamurthy@intel.com>,
Masami Hiramatsu <mhiramat@kernel.org>
Cc: Kees Cook <keescook@chromium.org>, Arnd Bergmann <arnd@arndb.de>,
Mark Rutland <mark.rutland@arm.com>,
Daniel Axtens <dja@axtens.net>,
Rasmus Villemoes <linux@rasmusvillemoes.dk>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Chris Metcalf <cmetcalf@ezchip.com>,
Thomas Gleixner <tglx@linutronix.de>,
"H. Peter Anvin" <hpa@zytor.com>, Ingo Molnar <mingo@elte.hu>,
Andrew Morton <akpm@linux-foundation.org>,
LKML <linux-kernel@vger.kernel.org>, LKP <lkp@01.org>
Subject: Re: [lkp-robot] [include/linux/string.h] 6974f0c455: kernel_BUG_at_lib/string.c
Date: Thu, 20 Jul 2017 00:41:36 -0400 [thread overview]
Message-ID: <1500525696.2055.5.camel@gmail.com> (raw)
In-Reply-To: <CA+55aFwuebKQ0-sAiRJWpMcaVJ8MjXvCg56DO5q8jMiPDXvc8A@mail.gmail.com>
> So the fortify_string code has decided that only a single-byte (or
> empty) memcpy is ok.
>
> And that, in turn, seems to be because we're copying from
> optprobe_template_entry, which is declared as
>
> extern __visible kprobe_opcode_t optprobe_template_entry;
>
> so the fortify code decides it's a single character.
>
> Does just changing all those things to be declared as arrays fix
> things?
Yeah, that fixes it because GCC will consider the size of 'char foo[]'
unknown (i.e. (size_t)-1 from __builtin_object_size).
GCC doesn't know this essentially constant value at compile-time so it
wasn't a compile-time error:
#define TMPL_END_IDX \
((long)&optprobe_template_end - (long)&optprobe_template_entry)
-fsanitize=object-size works the same way for pointer dereferences so
replacing might fix some issues for CONFIG_UBSAN_SANITIZE_ALL. I guess
that's way too noisy at the moment thus the !COMPILE_TEST.
next prev parent reply other threads:[~2017-07-20 4:41 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-20 1:42 [lkp-robot] [include/linux/string.h] 6974f0c455: kernel_BUG_at_lib/string.c kernel test robot
2017-07-20 1:42 ` kernel test robot
2017-07-20 4:02 ` Daniel Micay
2017-07-20 4:04 ` Linus Torvalds
2017-07-20 4:04 ` Linus Torvalds
2017-07-20 4:41 ` Daniel Micay [this message]
2017-07-20 13:46 ` Masami Hiramatsu
2017-07-20 13:46 ` Masami Hiramatsu
2017-07-20 14:50 ` Ye Xiaolong
2017-07-21 1:30 ` Fengguang Wu
2017-07-21 1:52 ` Ye Xiaolong
2017-07-21 1:56 ` Fengguang Wu
2017-07-21 2:03 ` Ye Xiaolong
2017-07-20 15:51 ` Masami Hiramatsu
2017-07-20 15:51 ` Masami Hiramatsu
2017-07-20 18:41 ` Linus Torvalds
2017-07-20 18:41 ` Linus Torvalds
2017-07-20 22:20 ` Masami Hiramatsu
2017-07-20 22:20 ` Masami Hiramatsu
2017-07-21 3:29 ` Masami Hiramatsu
2017-07-21 3:29 ` Masami Hiramatsu
2017-07-21 3:29 ` Masami Hiramatsu
2017-07-21 1:59 ` Ye Xiaolong
2017-07-21 1:59 ` Ye Xiaolong
2017-07-21 5:48 ` Kees Cook
2017-07-21 5:48 ` Kees Cook
2017-07-21 9:15 ` Andy Shevchenko
2017-07-21 9:15 ` Andy Shevchenko
2017-07-21 10:03 ` Petr Mladek
2017-07-21 10:03 ` Petr Mladek
2017-07-25 23:35 ` Kees Cook
2017-07-25 23:35 ` Kees Cook
2017-07-26 0:11 ` Linus Torvalds
2017-07-26 0:11 ` Linus Torvalds
2017-07-26 0:38 ` Kees Cook
2017-07-26 0:38 ` Kees Cook
2017-07-26 2:37 ` Daniel Micay
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=1500525696.2055.5.camel@gmail.com \
--to=danielmicay@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=ananth@linux.vnet.ibm.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=anil.s.keshavamurthy@intel.com \
--cc=arnd@arndb.de \
--cc=cmetcalf@ezchip.com \
--cc=dja@axtens.net \
--cc=hpa@zytor.com \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@rasmusvillemoes.dk \
--cc=lkp@01.org \
--cc=mark.rutland@arm.com \
--cc=mhiramat@kernel.org \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=xiaolong.ye@intel.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.