[PATCH 1/2] ext4: Fix integer overflow in ext4fs_read_symlink()

Tom Rini trini at konsulko.com
Fri Jul 12 18:54:43 CEST 2024


On Fri, Jul 12, 2024 at 01:26:35PM +0200, Richard Weinberger wrote:
> Am Freitag, 12. Juli 2024, 13:19:32 CEST schrieb Heinrich Schuchardt:
> > > Hm, I see man built-ins in the U-Boot source.
> > > Why is this one special?
> > 
> > See the definition of COMPILER_HAS_GENERIC_BUILTIN_OVERFLOW in
> > include/linux/compiler-clang.h.
> 
> So I can't use __builtin_add_overflow() because
> u-boot is still supporting outdated compilers that don't offer __builtin_add_overflow()?
> 
> Maybe this is the ideal moment to raise the bar?
> __builtin_add_overflow() is supported by gcc >= 5.1 and clang >= 8.

We already fail on gcc older than 6, for ARM. And we don't have a
minimum clang specifically, but it's probably newer than 8. So this
should be fine.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240712/f0ae31d9/attachment.sig>


More information about the U-Boot mailing list