diff options
author | peter <peter@FreeBSD.org> | 2003-10-17 15:43:13 +0000 |
---|---|---|
committer | peter <peter@FreeBSD.org> | 2003-10-17 15:43:13 +0000 |
commit | 81586929f375fc23eb8749aebf4eba5c6582b35a (patch) | |
tree | be3567a614ccda703207b4e8d2a45033f3bfd846 /lib/csu | |
parent | 749fb5ef58b19542a686bf4de16338c621c4cfdb (diff) | |
download | FreeBSD-src-81586929f375fc23eb8749aebf4eba5c6582b35a.zip FreeBSD-src-81586929f375fc23eb8749aebf4eba5c6582b35a.tar.gz |
Explicitly specify an alignment for abitag. Without it, gcc specifies a
section alignnment of 16 bytes for amd64 and this breaks file(1).
Before:
./cp: ELF 64-bit LSB executable, AMD x86-64, version 1 (FreeBSD), for \
FreeBSD 127.7.9, statically linked, stripped
after: ^^^^^^^
./ls: ELF 64-bit LSB executable, AMD x86-64, version 1 (FreeBSD), for \
FreeBSD 5.0.1, dynamically linked (uses shared libs), stripped
The reason for this is that the NOTE sections are not contiguous
internally. If the note section has an alignment of 16, then anything
that looks for the data is supposed to round up the payload start to
the next multiple of the alignment. But FreeBSD/amd64 broke because the
structure is declared as a single structure, not a (header,payload) group,
where the payload had an explicit alignment roundup.
The alternative is to change things like file(1) to ignore the ELF payload
alignment rules for the PT_NOTE section only for FreeBSD.
Diffstat (limited to 'lib/csu')
-rw-r--r-- | lib/csu/common/crtbrand.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/csu/common/crtbrand.c b/lib/csu/common/crtbrand.c index 9405f91..dde60b3 100644 --- a/lib/csu/common/crtbrand.c +++ b/lib/csu/common/crtbrand.c @@ -43,7 +43,7 @@ static const struct { int32_t type; char name[sizeof ABI_VENDOR]; int32_t desc; -} abitag __attribute__ ((section (ABI_SECTION))) __unused = { +} abitag __attribute__ ((section (ABI_SECTION), aligned(4))) __unused = { sizeof ABI_VENDOR, sizeof(int32_t), ABI_NOTETYPE, |