Skip to content

Commit

Permalink
src: check for empty maybe local
Browse files Browse the repository at this point in the history
Using ToLocalChecked on MaybeLocal without verifying it's empty
can lead to unattempted crash.

PR-URL: #32339
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
Xstoudi authored and targos committed Apr 22, 2020
1 parent aaf94fd commit 13a7e05
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/udp_wrap.cc
Original file line number Diff line number Diff line change
Expand Up @@ -477,7 +477,8 @@ void UDPWrap::DoSend(const FunctionCallbackInfo<Value>& args, int family) {

// construct uv_buf_t array
for (size_t i = 0; i < count; i++) {
Local<Value> chunk = chunks->Get(env->context(), i).ToLocalChecked();
Local<Value> chunk;
if (!chunks->Get(env->context(), i).ToLocal(&chunk)) return;

size_t length = Buffer::Length(chunk);

Expand Down

0 comments on commit 13a7e05

Please sign in to comment.