diff --git a/services/engine/histreamer/player/dragging_player_agent.cpp b/services/engine/histreamer/player/dragging_player_agent.cpp
index 0bfd0e5f165c140575b493b9d9f62edb65bb8e1f..4e5bbaa20b25301f162c062e9afaa30faa3af497 100644
--- a/services/engine/histreamer/player/dragging_player_agent.cpp
+++ b/services/engine/histreamer/player/dragging_player_agent.cpp
@@ -286,11 +286,9 @@ void SeekContinuousDelegator::Release()
     if (draggingPlayer_ != nullptr) {
         draggingPlayer_->Release();
     }
-    if (draggingPlayer_ != nullptr) {
+    if (demuxer_ != nullptr) {
         auto res = demuxer_->PauseDragging();
         FALSE_LOG_MSG(res == Status::OK, "PauseDragging failed");
-    }
-    if (demuxer_ != nullptr) {
         demuxer_->DeregisterVideoStreamReadyCallback();
     }
     if (decoder_ != nullptr) {
diff --git a/services/services/player/server/player_server.cpp b/services/services/player/server/player_server.cpp
index 0155679a80f126dd7172c51fc581913e3ab3ede1..a0a9309e87879788f2bc4d42ce98ffba9cfc888e 100644
--- a/services/services/player/server/player_server.cpp
+++ b/services/services/player/server/player_server.cpp
@@ -1497,7 +1497,7 @@ int32_t PlayerServer::SetVideoSurface(sptr<Surface> surface)
         std::lock_guard<std::mutex> surfaceLock(surfaceMutex_);
         surface_ = surface;
     }
-    CHECK_AND_RETURN_RET_LOG(switchSurface || playerEngine_ != nullptr, MSERR_OK,
+    CHECK_AND_RETURN_RET_LOG(switchSurface && playerEngine_ != nullptr, MSERR_OK,
         "current state: %{public}s, playerEngine == nullptr: %{public}d, can not SetVideoSurface",
         GetStatusDescription(lastOpStatus_).c_str(), playerEngine_ == nullptr);
     auto task = std::make_shared<TaskHandler<void>>([this]() {