n-api: add missed nullptr check in napi_has_own_property · nodejs/node@4d396fd · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 638
• Pull requests 751
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 4d396fd

Browse filesBrowse the repository at this point in the historyBrowse files

lundibundi

authored and

MylesBorins

committed

n-api: add missed nullptr check in napi_has_own_property

PR-URL: #30626 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>

1 parent 0e68f55 commit 4d396fd
Copy full SHA for 4d396fd

1 file changed

+1Lines changed: 1 addition & 0 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• src
  • js_native_api_v8.cc

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎src/js_native_api_v8.cc‎

Copy file name to clipboardExpand all lines: src/js_native_api_v8.cc

+1Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -981,6 +981,7 @@ napi_status napi_has_own_property(napi_env env,
981	981	bool* result) {
982	982	NAPI_PREAMBLE(env);
983	983	CHECK_ARG(env, key);
	984	+ CHECK_ARG(env, result);
984	985
985	986	v8::Local<v8::Context> context = env->context();
986	987	v8::Local<v8::Object> obj;

0 commit comments

Comments

0 (0)