Skip to content

v4.8.3 proposal #12499

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MylesBorins merged 15 commits into from
May 2, 2017
Merged

v4.8.3 proposal #12499

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
6d3aaa7
tls: TLSSocket emits 'error' on handshake failure
lekoder Sep 27, 2016
07b92a3
doc: add supported platforms list for v4.x
mhdawson Mar 17, 2017
1dc6b38
test: add test for loading from global folders
richardlau Oct 20, 2016
ba91c41
module: fix loading from global folders on Windows
richardlau Oct 25, 2016
99749dc
tls: keep track of stream that is closed
jBarz Mar 9, 2017
ff1d61c
stream_base,tls_wrap: notify on destruct
trevnorris Oct 31, 2016
4426080
Partial revert "tls: keep track of stream that is closed"
trevnorris Mar 20, 2017
54f5258
tls: fix segfault on destroy after partial read
bnoordhuis Mar 17, 2017
ab3fdf5
deps: cherry-pick ca0f9573 from V8 upstream
ofrobots Mar 20, 2017
8ed18a1
src: ensure that fd 0-2 are valid on windows
bzoz Mar 14, 2017
7a1920d
test: add hasCrypto check to tls-socket-close
danbev Mar 8, 2017
eb393f9
src: fix base64 decoding
seishun Mar 22, 2017
b5b78b1
src: add fcntl.h include to node.cc
bzoz Apr 20, 2017
6040efd
test: fix flaky test-tls-wrap-timeout
Trott Jul 23, 2016
788f280
2017-05-02, Version 4.8.3 'Argon' (Maintenance)
MylesBorins Apr 19, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
deps: cherry-pick ca0f9573 from V8 upstream 
Original commit message:
  Trigger OOM crash if no memory returned in v8::ArrayBuffer::New and v…
  …8::SharedArrayBuffer::New.

  This API does not allow reporting failure, but we should crash rather than have
  the caller get an ArrayBuffer that isn't properly set up.

  BUG=chromium:681843

  Review-Url: https://codereview.chromium.org/2641953002
  Cr-Commit-Position: refs/heads/master@{#42511}

PR-URL: #11940
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
  • Loading branch information
@ofrobots @MylesBorins
ofrobots authored and MylesBorins committed Apr 19, 2017
commit ab3fdf531fe0536ae6b3a657d7fb73adc0881a99
2 changes: 1 addition & 1 deletion deps/v8/include/v8-version.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
#define V8_MAJOR_VERSION 4
#define V8_MINOR_VERSION 5
#define V8_BUILD_NUMBER 103
#define V8_PATCH_LEVEL 46
#define V8_PATCH_LEVEL 47

// Use 1 for candidates and 0 otherwise.
// (Boolean macro values are not supported by all preprocessors.)
Expand Down
14 changes: 11 additions & 3 deletions deps/v8/src/api.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6580,7 +6580,11 @@ Local<ArrayBuffer> v8::ArrayBuffer::New(Isolate* isolate, size_t byte_length) {
ENTER_V8(i_isolate);
i::Handle<i::JSArrayBuffer> obj =
i_isolate->factory()->NewJSArrayBuffer(i::SharedFlag::kNotShared);
i::Runtime::SetupArrayBufferAllocatingData(i_isolate, obj, byte_length);
// TODO(jbroman): It may be useful in the future to provide a MaybeLocal
// version that throws an exception or otherwise does not crash.
if (!i::Runtime::SetupArrayBufferAllocatingData(i_isolate, obj, byte_length)) {
i::FatalProcessOutOfMemory("v8::ArrayBuffer::New");
}
return Utils::ToLocal(obj);
}

Expand Down Expand Up @@ -6775,8 +6779,12 @@ Local<SharedArrayBuffer> v8::SharedArrayBuffer::New(Isolate* isolate,
ENTER_V8(i_isolate);
i::Handle<i::JSArrayBuffer> obj =
i_isolate->factory()->NewJSArrayBuffer(i::SharedFlag::kShared);
i::Runtime::SetupArrayBufferAllocatingData(i_isolate, obj, byte_length, true,
i::SharedFlag::kShared);
// TODO(jborman): It may be useful in the future to provide a MaybeLocal
// version that throws an exception or otherwise does not crash.
if (!i::Runtime::SetupArrayBufferAllocatingData(i_isolate, obj, byte_length, true,
i::SharedFlag::kShared)) {
i::FatalProcessOutOfMemory("v8::SharedArrayBuffer::New");
}
return Utils::ToLocalShared(obj);
}

Expand Down