Re: [PATCH] sh: Fix build with CONFIG_UBSAN=y

From: John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>
To: Kees Cook <keescook@chromium.org>
Cc: Yoshinori Sato <ysato@users.sourceforge.jp>,
	kernel test robot <lkp@intel.com>, Rich Felker <dalias@libc.org>,
	Masahiro Yamada <masahiroy@kernel.org>,
	Nicolas Schier <n.schier@avm.de>,
	 linux-sh@vger.kernel.org, linux-kernel@vger.kernel.org,
	 linux-hardening@vger.kernel.org
Subject: Re: [PATCH] sh: Fix build with CONFIG_UBSAN=y
Date: Sun, 11 Feb 2024 11:39:49 +0100	[thread overview]
Message-ID: <cd1a36a234c8fc61c5febe646ae0f05ed20ae32a.camel@physik.fu-berlin.de> (raw)
In-Reply-To: <9fda57fc1b1ba6ad9bd6f7df3fb12674d0f4f940.camel@physik.fu-berlin.de>

Hi Kees,

On Fri, 2024-02-02 at 12:06 +0100, John Paul Adrian Glaubitz wrote:
> > When I use GCC 13.2 (I'm specifically on Ubuntu 23.10) and the randconfig
> > linked from the report:
> > https://download.01.org/0day-ci/archive/20240131/202401310416.s8HLiLnC-lkp@intel.com/config
> > (which is notably enabling CONFIG_UBSAN=y and CONFIG_UBSAN_SHIFT=y) then I
> > see at the final link stage:
> > 
> > /usr/bin/sh4-linux-gnu-ld: arch/sh/boot/compressed/misc.o: in function `zlib_inflate_table':
> > misc.c:(.text+0x650): undefined reference to `__ubsan_handle_shift_out_of_bounds'
> > ...
> > 
> > After the patch, it's solved.
> 
> OK, let me test with gcc 13.x. My build host is currently running openSUSE Leap 15.5.

I just wanted to try reproduce the problem again with the reproducer in [1] as well
as with gcc-13.2.0, but your branch devel/overflow/ubsan-only no longer exists.

Can you tell me where to find the patches now?

Adrian

> [1] https://download.01.org/0day-ci/archive/20240131/202401310416.s8HLiLnC-lkp@intel.com/reproduce

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913