openharmony/communication_ipc
1
0
Fork 0
mirror of https://gitee.com/openharmony/communication_ipc synced 2024-11-30 11:30:30 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

修改NAPIDeathRecipient将napi_delete_reference放在OnRemoteDied中

Browse Source 
Signed-off-by: wanderer-dl122 <dengliang21@huawei.com>
This commit is contained in:
wanderer-dl122 2023-05-03 22:28:40 +08:00
parent aeeeda8f52
commit 3005d41484
2 changed files with 4 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ipc/native/src/napi_common/include/napi_remote_proxy_holder.h
View File

@ -34,7 +34,7 @@ public:
bool Matches(napi_value object);
protected:
virtual ~NAPIDeathRecipient();
virtual ~NAPIDeathRecipient() = default;
private:
struct OnRemoteDiedParam {

14
ipc/native/src/napi_common/source/napi_remote_proxy_holder.cpp
View File

@ -33,17 +33,6 @@ NAPIDeathRecipient::NAPIDeathRecipient(napi_env env, napi_value jsDeathRecipient
NAPI_ASSERT_RETURN_VOID(env, status == napi_ok, "failed to create ref to js death recipient");
}
NAPIDeathRecipient::~NAPIDeathRecipient()
{
if (env_ != nullptr) {
if (deathRecipientRef_ != nullptr) {
napi_status status = napi_delete_reference(env_, deathRecipientRef_);
NAPI_ASSERT_RETURN_VOID(env_, status == napi_ok, "failed to delete ref to js death recipient");
deathRecipientRef_ = nullptr;
}
}
}
void NAPIDeathRecipient::OnRemoteDied(const wptr<IRemoteObject> &object)
{
if (deathRecipientRef_ == nullptr) {
@ -78,6 +67,9 @@ void NAPIDeathRecipient::OnRemoteDied(const wptr<IRemoteObject> &object)
if (returnVal == nullptr) {
ZLOGE(LOG_LABEL, "failed to call function onRemoteDied");
}
napi_status napiStatus = napi_delete_reference(param->env, param->deathRecipientRef);
NAPI_ASSERT_RETURN_VOID(param->env, napiStatus == napi_ok, "failed to delete ref to js death recipient");
delete param;
delete work;
});