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

Issue 2608123002: Merged: Prevent read-after-buffer in SourcePositionTableIterator::Advance. (Closed)

Created:
3 years, 11 months ago by vogelheim
Modified:
3 years, 11 months ago
Reviewers:
Michael Hablich, marja
CC:
v8-reviews_googlegroups.com, v8-merges_googlegroups.com, ulan
Target Ref:
refs/pending/branch-heads/5.6
Project:
v8
Visibility:
Public.

Description

Merged: Prevent read-after-buffer in SourcePositionTableIterator::Advance. Revision: da72a3176fba32b39049d070fbd2d789bbbfe8b6 BUG=chromium:667142 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true R=marja@chromium.org, hablich@chromium.org Review-Url: https://codereview.chromium.org/2608123002 . Cr-Commit-Position: refs/branch-heads/5.6@{#62} Cr-Branched-From: bdd3886218dfe76e8560eb8a18401942452ae859-refs/heads/5.6.326@{#1} Cr-Branched-From: 879f6599eee6e1dfcbe9a24bf688b261c03e9558-refs/heads/master@{#41014} Committed: https://chromium.googlesource.com/v8/v8/+/31b99bba343bd71c24e1e70440800488ff57b679

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/source-position-table.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (2 generated)
vogelheim
3 years, 11 months ago (2017-01-02 12:12:03 UTC) #1
vogelheim
NOTE: This script will no longer automatically update include/v8-version.h and create a tag. This is ...
3 years, 11 months ago (2017-01-02 12:12:07 UTC) #2
marja
rubberstamp lgtm
3 years, 11 months ago (2017-01-02 12:13:48 UTC) #4
vogelheim
3 years, 11 months ago (2017-01-02 12:16:05 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
31b99bba343bd71c24e1e70440800488ff57b679.

Powered by Google App Engine
This is Rietveld 408576698