FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[v14 backport] deps: V8: cherry-pick cc9a8a37445e by devsnek · Pull Request #41826 · nodejs/node · GitHub

/ node Public

[v14 backport] deps: V8: cherry-pick cc9a8a37445e - #41826

Closed
devsnek wants to merge 1 commit into
v14.x-stagingfrom
cherrypick-41823-14x
ConversationCommits (1)ChecksFiles changed
Closed

[v14 backport] deps: V8: cherry-pick cc9a8a37445e#41826
devsnek wants to merge 1 commit into
v14.x-stagingfrom
cherrypick-41823-14x

Conversation

devsnek commented Feb 2, 2022

Copy link
Copy Markdown
Member

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

deps: V8: cherry-pick cc9a8a37445e
941b023 
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

nodejs-github-bot commented Feb 2, 2022

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/v8-update

nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. v14.x v8 engine Issues and PRs related to the V8 dependency. labels Feb 2, 2022

nodejs-github-bot commented Feb 2, 2022 •
edited by richardlau
Loading

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/42323/
V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/4496/

devsnek requested a review from mcollina February 3, 2022 14:38
mcollina approved these changes Feb 3, 2022
View reviewed changes

mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

lgtm

richardlau force-pushed the v14.x-staging branch 2 times, most recently from 30ba86c to 7f2825b Compare March 17, 2022 23:36
juanarbol pushed a commit that referenced this pull request Apr 28, 2022
deps: V8: cherry-pick cc9a8a37445e
3d6255f 
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: #41826
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Danielle Adams <adamzdanielle@gmail.com>

juanarbol commented Apr 28, 2022 •
edited
Loading

Copy link
Copy Markdown
Member

Thank you!

Landed in 39e44f8 🎉

juanarbol closed this Apr 28, 2022
juanarbol added a commit that referenced this pull request Apr 28, 2022
2022-04-29, Version 14.19.2 'Fermium' (LTS)
df18654 
Notable changes

V8:
V8 had a stack overflow issue affecting the `vm` module, cherry-picking `941b023
from V8 solves this issue.

Learn more at: #41826

Contributed by @devsnek

Using `getHeapSnapshot()` was causing a Node.js crash due a V8 issue, this is fixed by backporting v8/v8@367b0c1
from V8.

Learn more at: #42637

Contributed by Gabriel Schulhof (@legendecas)

PR-url: TBD
juanarbol mentioned this pull request Apr 28, 2022
Merged
juanarbol pushed a commit that referenced this pull request May 1, 2022
deps: V8: cherry-pick cc9a8a37445e
39e44f8 
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: #41826
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Danielle Adams <adamzdanielle@gmail.com>
juanarbol added a commit that referenced this pull request May 1, 2022
2022-05-03, Version 14.19.2 'Fermium' (LTS)
bfc3dbc 
Notable Changes

doc:

* New release key for Bryan English

Learn more at: #42102
Contributed by Bryan English (@bengl)

npm:

* Upgrade `npm` to `v16.14.17`.

Learn more at: #42900
Contributed by Ruy Adorno (@ruyadorno)

V8:

* V8 had a stack overflow issue affecting the `vm` module, cherry-picking `cc9a8a37445e` (941b023)
from V8 solves this issue.

Learn more at: #41826
Contributed by @devsnek

* Using `getHeapSnapshot()` was causing a Node.js crash due a V8 issue, this is fixed by backporting `367b0c1e7a32` (v8/v8@367b0c1)
from V8.

Learn more at: #42637
Contributed by Chengzhong Wu (@legendecas)
juanarbol added a commit that referenced this pull request May 2, 2022
2022-05-03, Version 14.19.2 'Fermium' (LTS)
0527910 
Notable Changes

doc:

* New release key for Bryan English

Learn more at: #42102
Contributed by Bryan English (@bengl)

npm:

* Upgrade `npm` to `v6.14.17`.

Learn more at: #42900
Contributed by Ruy Adorno (@ruyadorno)

V8:

* V8 had a stack overflow issue affecting the `vm` module, cherry-picking `cc9a8a37445e` (941b023)
from V8 solves this issue.

Learn more at: #41826
Contributed by @devsnek

* Using `getHeapSnapshot()` was causing a Node.js crash due a V8 issue, this is fixed by backporting `367b0c1e7a32` (v8/v8@367b0c1)
from V8.

Learn more at: #42637
Contributed by Chengzhong Wu (@legendecas)
juanarbol added a commit that referenced this pull request May 3, 2022
2022-05-03, Version 14.19.2 'Fermium' (LTS)
3fc68fc 
Notable Changes

doc:

* New release key for Bryan English

Learn more at: #42102
Contributed by Bryan English (bengl)

npm:

* Upgrade `npm` to `v6.14.17`.

Learn more at: #42900
Contributed by Ruy Adorno (ruyadorno)

V8:

* V8 had a stack overflow issue affecting the `vm` module,
cherry-picking `cc9a8a37445e`
(v8/v8@cc9a8a3)
from V8 solves this issue.

Learn more at: #41826
Contributed by Gus Caplan (devsnek)

* Using `getHeapSnapshot()` was causing a Node.js crash due a V8 issue,
this is fixed by backporting `367b0c1e7a32`
(v8/v8@367b0c1)
from V8.

Learn more at: #42637
Contributed by Chengzhong Wu (legendecas)

PR-URL: #42899
juanarbol added a commit that referenced this pull request May 4, 2022
2022-05-04, Version 14.19.2 'Fermium' (LTS)
e365bec 
Notable Changes

doc:

* New release key for Bryan English

Learn more at: #42102
Contributed by Bryan English (bengl)

npm:

* Upgrade `npm` to `v6.14.17`.

Learn more at: #42900
Contributed by Ruy Adorno (ruyadorno)

V8:

* V8 had a stack overflow issue affecting the `vm` module,
cherry-picking `cc9a8a37445e`
(v8/v8@cc9a8a3)
from V8 solves this issue.

Learn more at: #41826
Contributed by Gus Caplan (devsnek)

* Using `getHeapSnapshot()` was causing a Node.js crash due a V8 issue,
this is fixed by backporting `367b0c1e7a32`
(v8/v8@367b0c1)
from V8.

Learn more at: #42637
Contributed by Chengzhong Wu (legendecas)

PR-URL: #42899
juanarbol added a commit that referenced this pull request May 4, 2022
2022-05-04, Version 14.19.2 'Fermium' (LTS)
755721a 
Notable Changes

doc:

* New release key for Bryan English

Learn more at: #42102
Contributed by Bryan English (bengl)

npm:

* Upgrade `npm` to `v6.14.17`.

Learn more at: #42900
Contributed by Ruy Adorno (ruyadorno)

V8:

* V8 had a stack overflow issue affecting the `vm` module,
cherry-picking `cc9a8a37445e`
(v8/v8@cc9a8a3)
from V8 solves this issue.

Learn more at: #41826
Contributed by Gus Caplan (devsnek)

* Using `getHeapSnapshot()` was causing a Node.js crash due a V8 issue,
this is fixed by backporting `367b0c1e7a32`
(v8/v8@367b0c1)
from V8.

Learn more at: #42637
Contributed by Chengzhong Wu (legendecas)

PR-URL: #42899
aduh95 deleted the cherrypick-41823-14x branch April 12, 2024 14:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

mcollina mcollina approved these changes
benjamingr benjamingr approved these changes
gengjiawen gengjiawen approved these changes
+1 more reviewer
danielleadams danielleadams approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

Back | FazBrowse Home | New Git URL