Skip to content

[v16.x backport] debugger: throw a more useful error when the frame is missing - #43595

Closed
cola119 wants to merge 5 commits into
nodejs:v16.x-stagingfrom
cola119:backport-42776-to-v16.x
Closed

[v16.x backport] debugger: throw a more useful error when the frame is missing#43595
cola119 wants to merge 5 commits into
nodejs:v16.x-stagingfrom
cola119:backport-42776-to-v16.x

Conversation

@cola119

Copy link
Copy Markdown
Member

backport of #42776

cc @danielleadams

PR-URL: #42776
Fixes: #42775
Reviewed-By: Antoine du Hamel duhamelantoine1995@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com

vmorozand others added 4 commits June 22, 2022 21:05
PR-URL: nodejs#42459
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Backport-PR-URL: nodejs#43293
 [Compiler] Remove untrusted code mitigations.
These are no longer enabled, so remove the code mitigation logic from
the codebase.
BUG=chromium:1003890
Change-Id: I536bb1732e8463281c21da446bbba8f47ede8ebe
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3045704
Commit-Queue: Ross McIlroy <rmcilroy@chromium.org>
Reviewed-by: Jakob Gruber <jgruber@chromium.org>
Reviewed-by: Clemens Backes <clemensb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#76256}
Refs: v8/v8@4ab70f6Fixes: nodejs#43296
PR-URL: nodejs#43328
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
This updates all sources in deps/openssl/openssl by:
$ git clone https://github.com/quictls/openssl
$ cd openssl
$ git checkout OpenSSL_1_1_1p+quic
$ cd ../node/deps/openssl
$ rm -rf openssl
$ cp -R ../openssl openssl
$ rm -rf openssl/.git* openssl/.travis*
$ git add --all openssl
$ git commit openssl
PR-URL: nodejs#43535
Refs: https://mta.openssl.org/pipermail/openssl-announce/2022-June/000228.html
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Danielle Adams <adamzdanielle@gmail.com>
 After an OpenSSL source update, all the config files need to be
regenerated and committed by:
$ make -C deps/openssl/config
$ git add deps/openssl/config/archs
$ git add deps/openssl/openssl/include/crypto/bn_conf.h
$ git add deps/openssl/openssl/include/crypto/dso_conf.h
$ git add deps/openssl/openssl/include/openssl/opensslconf.h
$ git commit
PR-URL: nodejs#43535
Refs: https://mta.openssl.org/pipermail/openssl-announce/2022-June/000228.html
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Danielle Adams <adamzdanielle@gmail.com>
@nodejs-github-botnodejs-github-bot added debugger Issues and PRs related to the debugger subsystem. needs-ci PRs that need a full CI run. v16.x labels Jun 27, 2022
@danielleadams

Copy link
Copy Markdown
Contributor

@cola119 I had to update the base branch (v16.x-staging) and its commit history. Do you mind rebasing? Sorry for the extra effort, and thank you.

PR-URL: nodejs#42776Fixes: nodejs#42775
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@cola119
cola119force-pushed the backport-42776-to-v16.x branch from 56b136f to 04b2f33CompareJune 28, 2022 03:00
@cola119

Copy link
Copy Markdown
MemberAuthor

@danielleadams rebasing done. Thank you for your support!

@targos

Copy link
Copy Markdown
Member

The original commit ended up working fine so I cherry-picked it. Thank anyway @cola119

@targostargos closed this Jul 18, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debuggerIssues and PRs related to the debugger subsystem.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@cola119@danielleadams@targos@nodejs-github-bot@vmoroz@XadillaX@RafaelGSS