diff --git a/src/js_native_api_v8.cpp b/src/js_native_api_v8.cpp index bdda5f36119fe7dd3a901949dd3ed6e1f69a8b1d..44a1d3c1539d88745d10932d8d9f594691292029 100644 --- a/src/js_native_api_v8.cpp +++ b/src/js_native_api_v8.cpp @@ -5561,14 +5561,14 @@ JSVM_Status OH_JSVM_RemoveHandlerForGC(JSVM_VM vm, if (it == handlers.end()) { return JSVM_INVALID_ARG; } - v8impl::GCHandlerWrapper wraper = *it; + v8impl::GCHandlerWrapper* wrapper = *it; handlers.erase(it); if (triggerTime == JSVM_CB_TRIGGER_BEFORE_GC) { - isolate->RemoveGCPrologueCallback(OnBeforeGC, wraper); + isolate->RemoveGCPrologueCallback(OnBeforeGC, wrapper); } else { - isolate->RemoveGCEpilogueCallback(OnAfterGC, wraper); + isolate->RemoveGCEpilogueCallback(OnAfterGC, wrapper); } - delete wraper; + delete wrapper; return JSVM_OK; } diff --git a/test/unittest/test_jsvm.cpp b/test/unittest/test_jsvm.cpp index 54cab73f84f83dc3446fe803f85868e54fe4c1a0..dd8af68a8b874cce33ab23d518809479a2ec0a45 100644 --- a/test/unittest/test_jsvm.cpp +++ b/test/unittest/test_jsvm.cpp @@ -630,8 +630,8 @@ HWTEST_F(JSVMTest, JSVMTraceStop003, TestSize.Level1) status = OH_JSVM_TraceStop(OutputStream, nullptr); ASSERT_EQ(status, JSVM_INVALID_ARG); std::string data; - JSVM_Status status = OH_JSVM_TraceStop(OutputStream, (void*)&data); - ASSERT_EQ(status, JSVM_INVALID_ARG); + status = OH_JSVM_TraceStop(OutputStream, (void*)&data); + ASSERT_EQ(status, JSVM_OK); } HWTEST_F(JSVMTest, JSVMTraceStop004, TestSize.Level1) @@ -641,8 +641,8 @@ HWTEST_F(JSVMTest, JSVMTraceStop004, TestSize.Level1) std::string data; status = OH_JSVM_TraceStop(nullptr, (void*)&data); ASSERT_EQ(status, JSVM_INVALID_ARG); - JSVM_Status status = OH_JSVM_TraceStop(OutputStream, (void*)&data); - ASSERT_EQ(status, JSVM_INVALID_ARG); + status = OH_JSVM_TraceStop(OutputStream, (void*)&data); + ASSERT_EQ(status, JSVM_OK); } HWTEST_F(JSVMTest, JSVMIsNumberObject001, TestSize.Level1)