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

Issue 2509843005: [fuzzing] consistently honor --abort-on-stack-overflow. (Closed)

Created:
4 years, 1 month ago by Yang
Modified:
4 years, 1 month ago
CC:
v8-reviews_googlegroups.com
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[fuzzing] consistently honor --abort-on-stack-overflow. R=machenbach@chromium.org BUG=chromium:664068 Committed: https://crrev.com/1fe704e113a9b5a95d65c68b0b3719b4b85d4a56 Cr-Commit-Position: refs/heads/master@{#41075}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -10 lines) Patch
M src/isolate.h View 1 chunk +7 lines, -8 lines 0 comments Download
M src/prototype.h View 1 chunk +1 line, -2 lines 0 comments Download
M src/regexp/regexp-parser.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
Yang
4 years, 1 month ago (2016-11-17 12:44:31 UTC) #1
Michael Achenbach
lgtm rubberstamp +jarin as he originally authored this
4 years, 1 month ago (2016-11-17 12:49:00 UTC) #3
Jarin
lgtm. Thanks for fixing that!
4 years, 1 month ago (2016-11-17 12:56:35 UTC) #4
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/2509843005/1
4 years, 1 month ago (2016-11-17 13:01:01 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-11-17 13:28:09 UTC) #7
commit-bot: I haz the power
4 years, 1 month ago (2016-11-17 22:38:15 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/1fe704e113a9b5a95d65c68b0b3719b4b85d4a56
Cr-Commit-Position: refs/heads/master@{#41075}

Powered by Google App Engine
This is Rietveld 408576698