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
Next Next commit
fixup! src: throw ERR_INVALID_ARG_TYPE in C++ argument checks
  • Loading branch information
@joyeecheung
joyeecheung committed Apr 18, 2018
commit 834764ec6f3fafce90461d218955a9d0223de24f
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 @@ -977,7 +977,7 @@ void SecureContext::SetOptions(const FunctionCallbackInfo<Value>& args) {
ASSIGN_OR_RETURN_UNWRAP(&sc, args.Holder());

if (args.Length() != 1 || !args[0]->IntegerValue()) {
return node::THROW_ERR_INVALID_ARG_TYPE(
return THROW_ERR_INVALID_ARG_TYPE(
sc->env(), "Options must be an integer value");
}

Expand Down Expand Up @@ -1033,7 +1033,7 @@ void SecureContext::SetSessionTimeout(const FunctionCallbackInfo<Value>& args) {
ASSIGN_OR_RETURN_UNWRAP(&sc, args.Holder());

if (args.Length() != 1 || !args[0]->IsInt32()) {
return node::THROW_ERR_INVALID_ARG_TYPE(
return THROW_ERR_INVALID_ARG_TYPE(
sc->env(), "Session timeout must be a 32-bit integer");
}

Expand Down Expand Up @@ -4311,7 +4311,7 @@ void DiffieHellman::SetKey(const v8::FunctionCallbackInfo<v8::Value>& args,

if (!Buffer::HasInstance(args[0])) {
snprintf(errmsg, sizeof(errmsg), "%s must be a buffer", what);
return node::THROW_ERR_INVALID_ARG_TYPE(env, errmsg);
return THROW_ERR_INVALID_ARG_TYPE(env, errmsg);
}

BIGNUM* num =
Expand Down