This bug was created as a clone Bug 620058 which contains more context.
Comment on attachment 8890094 [details] Bug 1359912 - Add -z relro linker flags. https://reviewboard.mozilla.org/r/161168/#review166552 Yay for minor security wins! But the Try push is on fire, so no r+. Also, do you need to announce this when it lands because it could introduce new classes of crashes due to a segfault accessing a (now) read-only segment? Will the crash signature make this obvious?
Attachment #8890094 - Flags: review?(gps) → review-
Comment on attachment 8890094 [details] Bug 1359912 - Add -z relro linker flags. https://reviewboard.mozilla.org/r/161168/#review166600 This patch didn't change. But I trust you and enough eyes are on other code related to this that I feel comfortable giving this my stamp of approval.
Attachment #8890094 - Flags: review?(gps) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/adb09e2d7a30 Add -z relro linker flags. r=gps
This busted noopt debug builds on linux like https://treeherder.mozilla.org/logviewer.html#?job_id=118224316&repo=autoland Backed out.
Backout by email@example.com: https://hg.mozilla.org/integration/autoland/rev/ab2472a1042b Backed out changeset adb09e2d7a30 for breaking linux noopt debug builds a=backout
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/465b7a02cb33 Add -z relro linker flags. r=gps
https://hg.mozilla.org/projects/jamun/rev/465b7a02cb336ac45c43664f2198bf6cd225132b Bug 1359912 - Add -z relro linker flags. r=gps
You need to log in before you can comment on or make changes to this bug.