Bug 1313841 - nasm 2.12 build failure on PowerPC
nasm 2.12 build failure on PowerPC
Product: Fedora
Classification: Fedora
Component: nasm (Show other bugs)
powerpc Unspecified
unspecified Severity high
: ---
: ---
Assigned To: Mikolaj Izdebski
Fedora Extras Quality Assurance
Depends On:
Blocks: F-ExcludeArch-ppc64le/PPC64LETracker
  Show dependency treegraph
Reported: 2016-03-02 07:35 EST by Michel Normand
Modified: 2016-03-03 15:26 EST (History)
6 users (show)

See Also:
Fixed In Version: nasm-2.12-2.fc24
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2016-03-03 02:39:36 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
nasm_change_unused.patch (1.33 KB, patch)
2016-03-02 07:37 EST, Michel Normand
no flags Details | Diff

  None (edit)
Description Michel Normand 2016-03-02 07:35:50 EST
Description of problem: nasm 2.12 build failure on PowerPC

as reported by koji
gcc -c -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -m64 -W -Wall -std=c99 -pedantic -Werror=implicit -Werror=missing-braces -Werror=return-type -Werror=trigraphs -Werror=pointer-arith -Werror=missing-prototypes -Werror=missing-declarations -Werror=comment -Werror=vla -DHAVE_CONFIG_H -I. -I. -o realpath.o realpath.c
In file included from /usr/include/bits/sigcontext.h:27:0,
                 from /usr/include/signal.h:306,
                 from /usr/include/sys/param.h:28,
                 from realpath.c:47:
/usr/include/asm/sigcontext.h:17:23: error: expected identifier or '(' before '[' token
  unsigned long _unused[4];
Comment 1 Michel Normand 2016-03-02 07:37 EST
Created attachment 1132295 [details]

The attached patch is suggested correction
Comment 2 Michel Normand 2016-03-02 08:27:37 EST
related scratch build passed without error.
Comment 3 Mikolaj Izdebski 2016-03-03 02:23:09 EST
Forwarded upstream: http://bugzilla.nasm.us/show_bug.cgi?id=3392339
Comment 4 Mikolaj Izdebski 2016-03-03 02:39:36 EST
Fixed in nasm-2.12-2.fc24
Comment 5 H. Peter Anvin 2016-03-03 15:26:26 EST
There is more to it than that.  The symbol "_unused" is in application namespace, and this code leaks it.  It should be changed to __unused or something else that isn't in application namespace.

Note You need to log in before you can comment on or make changes to this bug.