Skip to content

src: assign error codes to more errors thrown in C++ #20121

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

Closed
joyeecheung wants to merge 10 commits into from
Closed

src: assign error codes to more errors thrown in C++ #20121

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
034ddaf
src: add THROW_ERR_* helpers
joyeecheung Apr 16, 2018
495ba98
src: migrate ERR_INDEX_OUT_OF_RANGE in C++
joyeecheung Apr 16, 2018
0a9f327
src: throw ERR_INVALID_ARG_TYPE in C++ argument checks
joyeecheung Apr 16, 2018
185c60d
src: throw ERR_BUFFER_OUT_OF_BOUNDS in node_buffer.cc
joyeecheung Apr 16, 2018
b2e7ff3
src: throw ERR_MISSING_MODULE in module_wrap.cc
joyeecheung Apr 16, 2018
46935ab
src: throw ERR_INVALID_ARG_VALUE in node_crypto.cc
joyeecheung Apr 18, 2018
0b4364e
src: throw ERR_MISSING_ARGS in node_crypto.cc
joyeecheung Apr 18, 2018
184e8ea
fixup! src: throw ERR_MISSING_ARGS in node_crypto.cc
joyeecheung Apr 18, 2018
834764e
fixup! src: throw ERR_INVALID_ARG_TYPE in C++ argument checks
joyeecheung Apr 18, 2018
60a0c36
fixup! src: throw ERR_INVALID_ARG_VALUE in node_crypto.cc
joyeecheung Apr 18, 2018
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
fixup! src: throw ERR_INVALID_ARG_VALUE in node_crypto.cc
  • Loading branch information
@joyeecheung
joyeecheung committed Apr 18, 2018
commit 60a0c3618bc4cc41c0bc2215e1b90d254b0c2131
6 changes: 3 additions & 3 deletions src/node_crypto.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -956,7 +956,7 @@ void SecureContext::SetDHParam(const FunctionCallbackInfo<Value>& args) {
DH_get0_pqg(dh, &p, nullptr, nullptr);
const int size = BN_num_bits(p);
if (size < 1024) {
return node::THROW_ERR_INVALID_ARG_VALUE(
return THROW_ERR_INVALID_ARG_VALUE(
env, "DH parameter is less than 1024 bits");
} else if (size < 2048) {
args.GetReturnValue().Set(FIXED_ONE_BYTE_STRING(
Expand Down Expand Up @@ -1205,7 +1205,7 @@ void SecureContext::SetTicketKeys(const FunctionCallbackInfo<Value>& args) {
THROW_AND_RETURN_IF_NOT_BUFFER(env, args[0], "Ticket keys");

if (Buffer::Length(args[0]) != 48) {
return node::THROW_ERR_INVALID_ARG_VALUE(
return THROW_ERR_INVALID_ARG_VALUE(
env, "Ticket keys length must be 48 bytes");
}

Expand Down Expand Up @@ -4394,7 +4394,7 @@ void ECDH::New(const FunctionCallbackInfo<Value>& args) {

int nid = OBJ_sn2nid(*curve);
if (nid == NID_undef)
return node::THROW_ERR_INVALID_ARG_VALUE(env,
return THROW_ERR_INVALID_ARG_VALUE(env,
"First argument should be a valid curve name");

EC_KEY* key = EC_KEY_new_by_curve_name(nid);
Expand Down