Skip to content
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.

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

TLSSocket.setMaxSendFragment results in an abort #38325

Closed
zyscoder opened this issue Apr 21, 2021 · 3 comments
Closed

TLSSocket.setMaxSendFragment results in an abort #38325

zyscoder opened this issue Apr 21, 2021 · 3 comments
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs. tls Issues and PRs related to the tls subsystem.

Comments

@zyscoder
Copy link

What steps will reproduce the bug?

Setup a node instance,

» node

and run the following javascript code.

new tls.TLSSocket().setMaxSendFragment();

Then the node instance occurs an abort.

How often does it reproduce? Is there a required condition?

This abort can always be triggered following the steps above.

What is the expected behavior?

If any error occurs, an exception or other similar error-reporting stuff should be thrown. There is no reason to abort the whole node process.

What do you see instead?

» node
Welcome to Node.js v16.0.0-pre.
Type ".help" for more information.
> new tls.TLSSocket().setMaxSendFragment();
/home/zys/Toolchains/node/node[97452]: ../src/crypto/crypto_tls.cc:2010:static void node::crypto::TLSWrap::SetMaxSendFragment(const v8::FunctionCallbackInfo<v8::Value>&): Assertion `args.Length() >= 1 && args[0]->IsNumber()' failed.
 1: 0x565026d33af4 node::Abort() [/home/zys/Toolchains/node/node]
 2: 0x565026d33b88  [/home/zys/Toolchains/node/node]
 3: 0x565026ef3edc node::crypto::TLSWrap::SetMaxSendFragment(v8::FunctionCallbackInfo<v8::Value> const&) [/home/zys/Toolchains/node/node]
 4: 0x565026fb2167 v8::internal::FunctionCallbackArguments::Call(v8::internal::CallHandlerInfo) [/home/zys/Toolchains/node/node]
 5: 0x565026fb2f10  [/home/zys/Toolchains/node/node]
 6: 0x565026fb3507  [/home/zys/Toolchains/node/node]
 7: 0x565026fb379a v8::internal::Builtin_HandleApiCall(int, unsigned long*, v8::internal::Isolate*) [/home/zys/Toolchains/node/node]
 8: 0x565027951259  [/home/zys/Toolchains/node/node]
[1]    97452 abort (core dumped)  /home/zys/Toolchains/node/node                                                                                                                                                                                       

Additional information

@Ayase-252 Ayase-252 added the tls Issues and PRs related to the tls subsystem. label Apr 21, 2021
@Ayase-252
Copy link
Member

Since it is a duplicate of #38169 and will be fixed if #38170 lands, I'm closing the issue for now. Feel free to reopen it if you have further concern about the issue

@Ayase-252 Ayase-252 added the duplicate Issues and PRs that are duplicates of other issues or PRs. label Apr 21, 2021
@zyscoder
Copy link
Author

Sorry for my carelessness.

@Ayase-252
Copy link
Member

It is always appreciated to send bug reports. Just do what you want 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Issues and PRs that are duplicates of other issues or PRs. tls Issues and PRs related to the tls subsystem.
Projects
None yet
Development

No branches or pull requests

2 participants