Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(396)

Issue 2711753005: Use -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION for libFuzzer and AFL. (Closed)

Created:
3 years, 10 months ago by mmoroz
Modified:
3 years, 10 months ago
Reviewers:
Oliver Chang, kcc2
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Use -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION for libFuzzer and AFL. R=kcc@chromium.org, ochang@chromium.org BUG=695292, 694495 Review-Url: https://codereview.chromium.org/2711753005 Cr-Commit-Position: refs/heads/master@{#452783} Committed: https://chromium.googlesource.com/chromium/src/+/0ec18db7916d7549ee8fe61bb6063ca8922ed86e

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -0 lines) Patch
M build/config/sanitizers/BUILD.gn View 2 chunks +7 lines, -0 lines 0 comments Download
M build/config/sanitizers/OWNERS View 1 chunk +1 line, -0 lines 0 comments Download
M build/config/sanitizers/sanitizers.gni View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 12 (5 generated)
mmoroz
3 years, 10 months ago (2017-02-23 10:30:53 UTC) #1
Oliver Chang
lgtm, thanks!
3 years, 10 months ago (2017-02-23 17:40:02 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2711753005/1
3 years, 10 months ago (2017-02-24 08:07:17 UTC) #4
commit-bot: I haz the power
Try jobs failed on following builders: mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_ng/builds/395022)
3 years, 10 months ago (2017-02-24 08:28:42 UTC) #6
mmoroz
On 2017/02/24 08:28:42, commit-bot: I haz the power wrote: > Try jobs failed on following ...
3 years, 10 months ago (2017-02-24 09:23:23 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2711753005/1
3 years, 10 months ago (2017-02-24 09:23:48 UTC) #9
commit-bot: I haz the power
3 years, 10 months ago (2017-02-24 09:55:58 UTC) #12
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/0ec18db7916d7549ee8fe61bb606...

Powered by Google App Engine
This is Rietveld 408576698