Control: severity -1 normal On Tue, Feb 07, 2023 at 04:38:20PM -0500, Nick Rosbrook wrote: > Package: mmc-utils > Severity: serious > Tags: patch ftbfs > Justification: fails to build from source (but built successfully in the past) > User: ubuntu-de...@lists.ubuntu.com > Usertags: origin-ubuntu lunar ubuntu-patch > > Dear Maintainer, > > In Ubuntu, mcc-utils FTBFS[1] on ppc64el with the following:
Note that this is an Ubuntu-only FTBFS. For some reason Canonical has chosen to make ppc64el in Ubuntu different from all other architectures in Ubuntu and all architectures in Debian (including ppc64el) by defaulting to build with -O3 instead of -O2. > In file included from /usr/include/endian.h:35, > from /usr/include/powerpc64le-linux-gnu/sys/types.h:176, > from /usr/include/stdlib.h:395, > from mmc_cmds.c:21: > In function ‘__bswap_32’, > inlined from ‘do_rpmb_write_block’ at mmc_cmds.c:2462:27: > /usr/include/powerpc64le-linux-gnu/bits/byteswap.h:52:10: error: ‘cnt’ may > be used uninitialized [-Werror=maybe-uninitialized] > 52 | return __builtin_bswap32 (__bsx); > | ^~~~~~~~~~~~~~~~~~~~~~~~~ >... The fix is not wrong, but I don't think there is an actual bug: cnt is uninitialized only in the ret != 0 case, that does already exit(1) two lines earlier. cu Adrian
