Skip to content

Commit

Permalink
deps: V8: cherry-pick cc9a8a37445e
Browse files Browse the repository at this point in the history
Original commit message:

    fix overflow check in error formatting

    Bug: v8:12494
    Change-Id: Iba2684173296aa236f1a1c73a5606c21472eff06
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3426634
    Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
    Commit-Queue: Gus Caplan <snek@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#78909}

Refs: v8/v8@cc9a8a3

PR-URL: #42065
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
  • Loading branch information
devsnek authored Mar 30, 2022
1 parent a35f553 commit 1193290
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion common.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@

# Reset this number to 0 on major V8 upgrades.
# Increment by one for each non-official patch applied to deps/v8.
'v8_embedder_string': '-node.56',
'v8_embedder_string': '-node.57',

##### V8 defaults for Node.js #####

Expand Down
3 changes: 2 additions & 1 deletion deps/v8/src/execution/messages.cc
Original file line number Diff line number Diff line change
Expand Up @@ -823,7 +823,8 @@ MaybeHandle<Object> ErrorUtils::FormatStackTrace(Isolate* isolate,
Handle<FixedArray> elems = Handle<FixedArray>::cast(raw_stack);

const bool in_recursion = isolate->formatting_stack_trace();
if (!in_recursion) {
const bool has_overflowed = i::StackLimitCheck{isolate}.HasOverflowed();
if (!in_recursion && !has_overflowed) {
Handle<Context> error_context = error->GetCreationContext();
DCHECK(error_context->IsNativeContext());

Expand Down

0 comments on commit 1193290

Please sign in to comment.