[v16.x backport] src,inspector: fix empty MaybeLocal crash (!42967) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/Dossar/src,inspector/fix-empty-MaybeLocal-crash into v16.x-staging May 04, 2022

Based off of https://github.com/nodejs/node/pull/42409 to backport into node 16. Below credit of RaisinTen

Return early when the Inspector StringView to V8 String conversion fails and returns an empty MaybeLocal instead of running the invalid ToLocalChecked() assertion. Fixes: https://github.com/nodejs/node/issues/42407 Signed-off-by: Darshan Sen raisinten@gmail.com