diff --git a/services/sensor/src/sensor_dump.cpp b/services/sensor/src/sensor_dump.cpp
index 7ad601c9cc4367f98722d25e9c7dd33641ffe5f2..d1d298b904f82bf1703aa9447056f37af4612768 100644
--- a/services/sensor/src/sensor_dump.cpp
+++ b/services/sensor/src/sensor_dump.cpp
@@ -139,7 +139,6 @@ void SensorDump::ParseCommand(int32_t fd, const std::vector<std::string> &args,
         if (strcpy_s(argv[i], args[i].size() + 1, args[i].c_str()) != EOK) {
             SEN_HILOGE("strcpy_s error");
             goto RELEASE_RES;
-            return;
         }
     }
     optind = 1;
diff --git a/services/sensor/src/sensor_suspend_policy.cpp b/services/sensor/src/sensor_suspend_policy.cpp
index 0fe24efa7808624071c845ead677966d86c7159b..217e25400b86fc58a93063c824e6f6db0e60c39d 100644
--- a/services/sensor/src/sensor_suspend_policy.cpp
+++ b/services/sensor/src/sensor_suspend_policy.cpp
@@ -168,7 +168,7 @@ void SensorSuspendPolicy::DoActive(const std::shared_ptr<ResourceSchedule::Suspe
             }
             auto ret = EnableSensor(sensorId, appInfo.pid, samplePeriod, maxReportDelay);
             if (ret != ERR_OK) {
-                SEN_HILOGE("sensorId:%{public}u,pid:%{public}d,ret:%{public}d"), sensorId, appInfo.pid, ret);
+                SEN_HILOGE("sensorId:%{public}u,pid:%{public}d,ret:%{public}d", sensorId, appInfo.pid, ret);
             }
         }
     }
diff --git a/utils/src/sensor_basic_data_channel.cpp b/utils/src/sensor_basic_data_channel.cpp
index 6ac276c1effb553795eb22a7caa796611d24b536..0e7ef0777eaf5c1609e4b19e50d3a17995c80444 100755
--- a/utils/src/sensor_basic_data_channel.cpp
+++ b/utils/src/sensor_basic_data_channel.cpp
@@ -174,9 +174,6 @@ int32_t SensorBasicDataChannel::ReceiveData(void *vaddr, size_t size)
     do {
         length = recv(receiveFd_, vaddr, size, MSG_DONTWAIT);
     } while (errno == EINTR);
-    if (length < 0) {
-        return 0;
-    }
     return length;
 }