diff --git a/interfaces/innerkits/native_cpp/screen_sink/src/dscreen_sink_handler.cpp b/interfaces/innerkits/native_cpp/screen_sink/src/dscreen_sink_handler.cpp
index 8f4472cc86dbe81d3017d4c9235957fd3d9e33f3..8efd4b3fb96fe54e03e7e9f761762a1bde936749 100644
--- a/interfaces/innerkits/native_cpp/screen_sink/src/dscreen_sink_handler.cpp
+++ b/interfaces/innerkits/native_cpp/screen_sink/src/dscreen_sink_handler.cpp
@@ -121,7 +121,7 @@ void DScreenSinkHandler::OnRemoteSinkSvrDied(const wptr<IRemoteObject> &remote)
         return;
     }
     std::lock_guard<std::mutex> lock(mutex_);
-    if (!dScreenSinkProxy_) {
+    if (dScreenSinkProxy_) {
         dScreenSinkProxy_->AsObject()->RemoveDeathRecipient(sinkSvrRecipient_);
         dScreenSinkProxy_ = nullptr;
     }
diff --git a/interfaces/innerkits/native_cpp/screen_source/src/dscreen_source_handler.cpp b/interfaces/innerkits/native_cpp/screen_source/src/dscreen_source_handler.cpp
index ed5e3c0f89c79b46a453b839d4241335ae20c355..825f04a7b6277c01331a0d96ebd4e60c1147652c 100644
--- a/interfaces/innerkits/native_cpp/screen_source/src/dscreen_source_handler.cpp
+++ b/interfaces/innerkits/native_cpp/screen_source/src/dscreen_source_handler.cpp
@@ -151,7 +151,7 @@ void DScreenSourceHandler::OnRemoteSourceSvrDied(const wptr<IRemoteObject> &remo
         return;
     }
     std::lock_guard<std::mutex> lock(mutex_);
-    if (!dScreenSourceProxy_) {
+    if (dScreenSourceProxy_) {
         dScreenSourceProxy_->AsObject()->RemoveDeathRecipient(sourceSvrRecipient_);
         dScreenSourceProxy_ = nullptr;
     }
diff --git a/services/screenservice/sinkservice/screenregionmgr/src/screenregion.cpp b/services/screenservice/sinkservice/screenregionmgr/src/screenregion.cpp
index 9fdf73b13ddc1d91c21ed0ffaf6ef88eee83c99f..25bb4114b5d88e28c6643855e00caa4d4fe12f15 100644
--- a/services/screenservice/sinkservice/screenregionmgr/src/screenregion.cpp
+++ b/services/screenservice/sinkservice/screenregionmgr/src/screenregion.cpp
@@ -37,7 +37,7 @@ ScreenRegion::ScreenRegion(const std::string &remoteDevId, uint64_t screenId, ui
 ScreenRegion::~ScreenRegion()
 {
     DHLOGD("ScreenRegion deConstruct");
-    if (!sinkTrans_) {
+    if (sinkTrans_) {
         sinkTrans_->Release();
     }
 }