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

nodejs / node Public

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

• Code
• Issues 632
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit cc5875b

Browse filesBrowse the repository at this point in the historyBrowse files

lundibundi

authored and

addaleax

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 1f10681 commit cc5875b
Copy full SHA for cc5875b

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
@@ -983,6 +983,7 @@ napi_status napi_has_own_property(napi_env env,
983	983	bool* result) {
984	984	NAPI_PREAMBLE(env);
985	985	CHECK_ARG(env, key);
	986	+ CHECK_ARG(env, result);
986	987
987	988	v8::Local<v8::Context> context = env->context();
988	989	v8::Local<v8::Object> obj;

0 commit comments

Comments

0 (0)