diff options
| author | Anna Henningsen <anna@addaleax.net> | 2019-06-02 15:47:53 +0200 |
|---|---|---|
| committer | Anna Henningsen <anna@addaleax.net> | 2019-06-09 22:34:42 +0200 |
| commit | 626f24374741c4ee83c7da45542b99e67855707d (patch) | |
| tree | 4b940f03d3dcc00de289c647c62bdecceca9cc36 /src/node_util.cc | |
| parent | 890223dede339622e7e66e8629805f7904840870 (diff) | |
| download | android-node-v8-626f24374741c4ee83c7da45542b99e67855707d.tar.gz android-node-v8-626f24374741c4ee83c7da45542b99e67855707d.tar.bz2 android-node-v8-626f24374741c4ee83c7da45542b99e67855707d.zip | |
src: handle exceptions from ToDetailString()
These methods may fail if execution is terminating.
PR-URL: https://github.com/nodejs/node/pull/28019
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Diffstat (limited to 'src/node_util.cc')
| -rw-r--r-- | src/node_util.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/node_util.cc b/src/node_util.cc index 9e506f0e65..ab54c84379 100644 --- a/src/node_util.cc +++ b/src/node_util.cc @@ -125,8 +125,10 @@ static void PreviewEntries(const FunctionCallbackInfo<Value>& args) { // Side effect-free stringification that will never throw exceptions. static void SafeToString(const FunctionCallbackInfo<Value>& args) { - auto context = args.GetIsolate()->GetCurrentContext(); - args.GetReturnValue().Set(args[0]->ToDetailString(context).ToLocalChecked()); + Local<Context> context = args.GetIsolate()->GetCurrentContext(); + Local<String> detail_string; + if (args[0]->ToDetailString(context).ToLocal(&detail_string)) + args.GetReturnValue().Set(detail_string); } inline Local<Private> IndexToPrivateSymbol(Environment* env, uint32_t index) { |