openharmony/ability_ability_runtime
1
0
Fork 0
mirror of https://gitee.com/openharmony/ability_ability_runtime synced 2024-11-27 01:10:52 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

补充napi_wrap调用异常处理，避免内存泄漏

Browse Source 
Signed-off-by: hanchenZz <wanghanchen1@huawei.com>
This commit is contained in:
hanchenZz 2024-11-08 14:34:44 +08:00
parent dd5d48d1ea
commit e5b900dcdc
1 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
frameworks/native/ability/native/auto_fill_extension_ability/js_auto_fill_extension.cpp
View File

@ -83,12 +83,17 @@ napi_value AttachAutoFillExtensionContext(napi_env env, void *value, void *)
auto workContext = new (std::nothrow) std::weak_ptr<AutoFillExtensionContext>(ptr);
if (workContext != nullptr) {
napi_wrap(env, contextObj, workContext,
auto status = napi_wrap(env, contextObj, workContext,
[](napi_env, void *data, void *) {
TAG_LOGD(AAFwkTag::AUTOFILL_EXT, "Finalizer for weak_ptr ui extension context is called");
delete static_cast<std::weak_ptr<AutoFillExtensionContext> *>(data);
},
nullptr, nullptr);
if (status != napi_ok) {
TAG_LOGE(AAFwkTag::AUTOFILL_EXT, "wrap ui extension context failed: %{public}d", status);
delete workContext;
return nullptr;
}
}
return contextObj;
@ -201,12 +206,16 @@ void JsAutoFillExtension::BindContext(napi_env env, napi_value obj)
env, contextObj, DetachCallbackFunc, AttachAutoFillExtensionContext, workContext, nullptr);
context->Bind(jsRuntime_, shellContextRef_.get());
napi_set_named_property(env, obj, "context", contextObj);
napi_wrap(env, contextObj, workContext,
auto status = napi_wrap(env, contextObj, workContext,
[](napi_env, void* data, void*) {
TAG_LOGD(AAFwkTag::AUTOFILL_EXT, "Finalizer for weak_ptr ui extension context is called");
delete static_cast<std::weak_ptr<AutoFillExtensionContext>*>(data);
},
nullptr, nullptr);
if (status != napi_ok) {
TAG_LOGE(AAFwkTag::AUTOFILL_EXT, "wrap ui extension context failed: %{public}d", status);
delete workContext;
}
}
void JsAutoFillExtension::OnStart(const AAFwk::Want &want, sptr<AAFwk::SessionInfo> sessionInfo)