New issue
Advanced search Search tips

Issue 621175 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Cherrypick clang FORTIFY-related fixes

Project Member Reported by g...@chromium.org, Jun 17 2016

Issue description

Clang ToT has multiple FORTIFY-related bugfixes that are necessary for getting FORTIFY to work better on clang. These should be pulled into the clang we use on ChromeOS.

Revision numbers that need to be grabbed:
269154
270781
270784
272080
272971
 

Comment 1 by g...@chromium.org, Jun 21 2016

Status: Fixed (was: Started)
Fixed by CL:353375 (...which seems to have been merged, regardless of what the bots say).
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 23 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/d0b4dc7c580894b965700a377b65d6e41667bf72

commit d0b4dc7c580894b965700a377b65d6e41667bf72
Author: George Burgess IV <gbiv@google.com>
Date: Thu Jun 16 22:58:04 2016

Add patches to clang for the new glibc FORTIFY.

In order for FORTIFY on clang to be nearly as capable as FORTIFY on gcc,
clang needs a special FORTIFY implementation. This patch cherry-picks
bugfixes in clang for said FORTIFY impl.

BUG= chromium:621175 
TEST=cbuildbot daisy-release falco-release x86-alex-release oak-release kunimitsu-release

Change-Id: I525b70cca8e6973d3642e18ef61998fe4be160be
Reviewed-on: https://chromium-review.googlesource.com/353375
Commit-Ready: George Burgess <gbiv@google.com>
Tested-by: George Burgess <gbiv@google.com>
Reviewed-by: Yunlian Jiang <yunlian@chromium.org>

[add] https://crrev.com/d0b4dc7c580894b965700a377b65d6e41667bf72/sys-devel/llvm/files/clang-3.8-fortify-fix.patch
[rename] https://crrev.com/d0b4dc7c580894b965700a377b65d6e41667bf72/sys-devel/llvm/llvm-3.9_pre265926-r5.ebuild

Sign in to add a comment