Bug 1200138

Summary: binutils: ld sporadically generates binaries without relro protection even when told so
Product: Red Hat Enterprise Linux 7 Reporter: Florian Weimer <fweimer>
Component: binutilsAssignee: Jeff Law <law>
Status: CLOSED ERRATA QA Contact: Miloš Prchlík <mprchlik>
Severity: medium Docs Contact: Robert Krátký <rkratky>
Priority: high    
Version: 7.1CC: arubin, asn, bressers, ebenes, gdeschner, hannsj_uhl, jakub, jherrman, jkurik, mcermak, mprchlik, ohudlick, rkratky, sgrubb
Target Milestone: rcKeywords: ZStream
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: binutils-2.23.52.0.1-31.el7 Doc Type: Bug Fix
Doc Text:
RELRO protection now properly applied when requested Previously, binary files started by the system loader would, in some cases, lack the Relocation Read-Only (RELRO) protection even though this had been explicitly requested when the application was built. This was due to a miscommunication between the static linker and the system loader. The underlying source code of the linker has been adjusted to ensure that it makes it possible for the loader to apply the RELRO protection, thus restoring the security feature for applications. Applications and all dependent object files, archives, and libraries built with an alpha or beta version of _binutils_ should be rebuilt to correct this defect. This update fixes the issue on the AMD64, Intel 64, 64-bit PowerPC, and 64-bit ARM architectures.
Story Points: ---
Clone Of:
: 1201345 1227839 (view as bug list) Environment:
Last Closed: 2015-11-19 03:41:02 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1189104, 1189204, 1198591, 1201345, 1206541, 1206545, 1209858, 1227839    

Description Florian Weimer 2015-03-09 20:06:09 UTC
Created attachment 999663 [details]
reproducer.tar.gz

Description of problem:

Binaries generated by ld sometimes lack relro protection, although it has explicitly been requested on the command line.

This has potential security implications, please treat this as embargoed for now.  We will decide how to disclose this once we better understand this issue.

Version-Release number of selected component (if applicable):

binutils-2.23.52.0.1-16.el7.x86_64

How reproducible:

Always (with the reproducer).

Steps to Reproduce:

tar xf reproducer.tar.gz 
cd reproducer
cmd="/usr/bin/ld --eh-frame-hdr --build-id -m elf_x86_64 -shared -o output.so -L. crti.o crtbeginS.o  winbind_krb5_locator_17.o -z relro -z now -no-undefined --export-dynamic -lpthread --as-needed -rpath XXX -Bdynamic -lwbclient  -lc  crtendS.o crtn.o"
$(echo "$cmd" | sed s/-rpath.XXX//); eu-readelf -a output.so | grep RELRO
$cmd; eu-readelf -a output.so | grep RELRO


Actual results:

[fweimer@oldenburg reproducer]$ $(echo "$cmd" | sed s/-rpath.XXX//); eu-readelf -a output.so | grep RELRO
  GNU_RELRO      0x001d68 0x0000000000201d68 0x0000000000201d68 0x000298 0x000298 R   0x1
   01      [RELRO: .ctors .dtors .jcr .data.rel.ro .dynamic .got] .bss
   02      [RELRO: .dynamic]
   06      [RELRO: .ctors .dtors .jcr .data.rel.ro .dynamic .got]
[fweimer@oldenburg reproducer]$ $cmd; eu-readelf -a output.so | grep RELRO
[fweimer@oldenburg reproducer]$ 

(relro only in the first case.)

Expected results:

[fweimer@oldenburg reproducer]$ $(echo "$cmd" | sed s/-rpath.XXX//); eu-readelf -a output.so | grep RELRO
  GNU_RELRO      0x001d68 0x0000000000201d68 0x0000000000201d68 0x000298 0x000298 R   0x1
   01      [RELRO: .ctors .dtors .jcr .data.rel.ro .dynamic .got] .bss
   02      [RELRO: .dynamic]
   06      [RELRO: .ctors .dtors .jcr .data.rel.ro .dynamic .got]
[fweimer@oldenburg reproducer]$ $cmd; eu-readelf -a output.so | grep RELRO
  GNU_RELRO      0x001d68 0x0000000000201d68 0x0000000000201d68 0x000298 0x000298 R   0x1
   01      [RELRO: .ctors .dtors .jcr .data.rel.ro .dynamic .got] .bss
   02      [RELRO: .dynamic]
   06      [RELRO: .ctors .dtors .jcr .data.rel.ro .dynamic .got]
[fweimer@oldenburg reproducer]$ 

(relro in both cases.)

Additional info:

The reproducer is constructed in such a way that it should not depend on installed system libraries; in this case, they come from RHEL 6.6.  With different libc versions, this reproducer does not show the bug, although we have seen similar issues with the RHEL 7 libc.

Comment 2 Florian Weimer 2015-03-09 20:13:22 UTC
Also reproducible with binutils-2.24-30.fc21.x86_64.

Comment 5 Florian Weimer 2015-03-11 17:36:51 UTC
The reproducer does not work with binutils-2.25-5.fc22.x86_64 from Fedora 22 and binutils-2.25-6.fc23.x86_64 from current Fedora rawhide.

No need to embargo this because it is public information, based on the upstream discussion/commits Jakub provided.

Comment 12 Miloš Prchlík 2015-10-13 09:00:04 UTC
Verified for build binutils-2.23.52.0.1-54.el7.

Comment 14 errata-xmlrpc 2015-11-19 03:41:02 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHSA-2015-2079.html