diff --git a/camera_hdf/hdi_impl/include/dstream_operator/dstream_operator.h b/camera_hdf/hdi_impl/include/dstream_operator/dstream_operator.h
index 174958d8df6ff2159496cc7bc65fc5da020c1360..8e012fa6e69c19a29d84bec3355b9241d86634bc 100644
--- a/camera_hdf/hdi_impl/include/dstream_operator/dstream_operator.h
+++ b/camera_hdf/hdi_impl/include/dstream_operator/dstream_operator.h
@@ -33,7 +33,7 @@ using namespace std;
 class DCameraProvider;
 class DStreamOperator : public DStreamOperatorStub {
 public:
-    DStreamOperator(std::shared_ptr<DMetadataProcessor> &dMetadataProcessor);
+    explicit DStreamOperator(std::shared_ptr<DMetadataProcessor> &dMetadataProcessor);
     DStreamOperator() = default;
     virtual ~DStreamOperator() = default;
     DStreamOperator(const DStreamOperator &other) = delete;
diff --git a/camera_hdf/hdi_impl/src/dcamera_device/dmetadata_processor.cpp b/camera_hdf/hdi_impl/src/dcamera_device/dmetadata_processor.cpp
index 154d47fdb6b2241ff116716e0e1559aa0a1ea435..20bfaf9eb8c3841585cb5572cc56e85838037098 100644
--- a/camera_hdf/hdi_impl/src/dcamera_device/dmetadata_processor.cpp
+++ b/camera_hdf/hdi_impl/src/dcamera_device/dmetadata_processor.cpp
@@ -529,8 +529,8 @@ std::map<int, std::vector<DCResolution>> DMetadataProcessor::GetDCameraSupported
                 if (reso.size() != SIZE_FMT_LEN) {
                     continue;
                 }
-                uint32_t width = static_cast<uint32_t>(std::atoi(reso[0].c_str()));
-                uint32_t height = static_cast<uint32_t>(std::atoi(reso[1].c_str()));
+                uint32_t width = static_cast<uint32_t>(std::stoi(reso[0]));
+                uint32_t height = static_cast<uint32_t>(std::stoi(reso[1]));
                 if (height == 0 || width == 0 ||
                     (isPhotoFormat && (width > MAX_SUPPORT_PHOTO_WIDTH || height > MAX_SUPPORT_PHOTO_HEIGHT)) ||
                     (!isPhotoFormat &&
diff --git a/camera_hdf/hdi_impl/src/dstream_operator/dcamera_steam.cpp b/camera_hdf/hdi_impl/src/dstream_operator/dcamera_steam.cpp
index 112e85359c5446f961936686c0d85021b15cc00b..b28b740cb070657b69578448e6b7bc3e3b2255ff 100644
--- a/camera_hdf/hdi_impl/src/dstream_operator/dcamera_steam.cpp
+++ b/camera_hdf/hdi_impl/src/dstream_operator/dcamera_steam.cpp
@@ -160,7 +160,7 @@ DCamRetCode DCameraStream::FinishCommitStream()
 
 DCamRetCode DCameraStream::GetNextRequest()
 {
-    if (isBufferMgrInited_ == false) {
+    if (!isBufferMgrInited_) {
         DHLOGE("BufferManager not be init.");
         return DCamRetCode::INVALID_ARGUMENT;
     }
@@ -353,7 +353,7 @@ DCamRetCode DCameraStream::FlushDCameraBuffer()
 
 bool DCameraStream::HasBufferQueue()
 {
-    if (dcStreamProducer_ == nullptr || isBufferMgrInited_ == false) {
+    if (dcStreamProducer_ == nullptr || !isBufferMgrInited_) {
         return false;
     }
     return true;
diff --git a/camera_hdf/hdi_impl/src/dstream_operator/dstream_operator.cpp b/camera_hdf/hdi_impl/src/dstream_operator/dstream_operator.cpp
index 4655598b108f0882d619cc0ba17ef5a000bafb52..ff6b8783f024383f6348002ef08c5a96ed62e2eb 100644
--- a/camera_hdf/hdi_impl/src/dstream_operator/dstream_operator.cpp
+++ b/camera_hdf/hdi_impl/src/dstream_operator/dstream_operator.cpp
@@ -65,7 +65,7 @@ CamRetCode DStreamOperator::CreateStreams(const std::vector<std::shared_ptr<Stre
         if (halStreamMap_.find(info->streamId_) != halStreamMap_.end()) {
             return CamRetCode::INVALID_ARGUMENT;
         }
-        if (info->tunneledMode_ != true) {
+        if (!info->tunneledMode_) {
             return CamRetCode::METHOD_NOT_SUPPORTED;
         }
 
@@ -418,8 +418,8 @@ DCamRetCode DStreamOperator::InitOutputConfigurations(const std::shared_ptr<DHBa
                 if (reso.size() != SIZE_FMT_LEN) {
                     continue;
                 }
-                uint32_t width = static_cast<uint32_t>(std::atoi(reso[0].c_str()));
-                uint32_t height = static_cast<uint32_t>(std::atoi(reso[1].c_str()));
+                uint32_t width = static_cast<uint32_t>(std::stoi(reso[0]));
+                uint32_t height = static_cast<uint32_t>(std::stoi(reso[1]));
                 if (height == 0 || width == 0 ||
                     (isPhotoFormat && (width > MAX_SUPPORT_PHOTO_WIDTH || height > MAX_SUPPORT_PHOTO_HEIGHT)) ||
                     (!isPhotoFormat &&
diff --git a/camera_hdf/hdi_impl/test/common.cpp b/camera_hdf/hdi_impl/test/common.cpp
index a0b805e74f507f299ae2c65b5e4e8fdc444fae4b..ec92151a6120afeff9a40388d33bb8f027244ec1 100644
--- a/camera_hdf/hdi_impl/test/common.cpp
+++ b/camera_hdf/hdi_impl/test/common.cpp
@@ -354,7 +354,7 @@ OHOS::sptr<OHOS::IBufferProducer> Test::StreamConsumer::CreateProducer(std::func
         int32_t flushFence = 0;
         int64_t timestamp = 0;
         OHOS::Rect damage;
-        while (running_ == true) {
+        while (running_) {
             OHOS::sptr<OHOS::SurfaceBuffer> buffer = nullptr;
             consumer_->AcquireBuffer(buffer, flushFence, timestamp, damage);
             if (buffer != nullptr) {
@@ -380,7 +380,7 @@ OHOS::sptr<OHOS::IBufferProducer> Test::StreamConsumer::CreateProducer(std::func
                     cv_.notify_one();
                 }
             }
-            if (running_ == false) {
+            if (!running_) {
                 break;
             }
             std::this_thread::sleep_for(1ms);
diff --git a/common/include/utils/data_buffer.h b/common/include/utils/data_buffer.h
index fa94f3d127bb00156536ac8f2f3257cb7a75f6c1..c884ff05c7355a69feea0906acc623350ec62039 100644
--- a/common/include/utils/data_buffer.h
+++ b/common/include/utils/data_buffer.h
@@ -26,7 +26,7 @@ namespace OHOS {
 namespace DistributedHardware {
 class DataBuffer {
 public:
-    DataBuffer(size_t capacity);
+    explicit DataBuffer(size_t capacity);
 
     size_t Size() const;
     size_t Offset() const;
diff --git a/services/cameraservice/base/include/dcamera_open_info_cmd.h b/services/cameraservice/base/include/dcamera_open_info_cmd.h
index d40a414e37f3327a6a4d16bb1ac5fc9ebe5a2c90..1f22e22d953d1e9b695f326045a63b73241f0692 100644
--- a/services/cameraservice/base/include/dcamera_open_info_cmd.h
+++ b/services/cameraservice/base/include/dcamera_open_info_cmd.h
@@ -25,7 +25,7 @@ namespace DistributedHardware {
 class DCameraOpenInfo {
 public:
     DCameraOpenInfo() {}
-    DCameraOpenInfo(std::string sourceDevId) : sourceDevId_(sourceDevId) {}
+    explicit DCameraOpenInfo(std::string sourceDevId) : sourceDevId_(sourceDevId) {}
     ~DCameraOpenInfo() = default;
     std::string sourceDevId_;
 };
diff --git a/services/cameraservice/sourceservice/include/distributedcamera/dcamera_service_state_listener.h b/services/cameraservice/sourceservice/include/distributedcamera/dcamera_service_state_listener.h
index 92e7530aff51b9ddaffb93f432d3267aecab78ac..967bd17a3122957e488d62f8cc86509ee168e4d9 100644
--- a/services/cameraservice/sourceservice/include/distributedcamera/dcamera_service_state_listener.h
+++ b/services/cameraservice/sourceservice/include/distributedcamera/dcamera_service_state_listener.h
@@ -23,7 +23,7 @@ namespace OHOS {
 namespace DistributedHardware {
 class DCameraServiceStateListener : public ICameraStateListener {
 public:
-    DCameraServiceStateListener(sptr<IDCameraSourceCallback> callback);
+    explicit DCameraServiceStateListener(sptr<IDCameraSourceCallback> callback);
     ~DCameraServiceStateListener();
 
     int32_t OnRegisterNotify(const std::string& devId, const std::string& dhId, const std::string& reqId,
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamera_source_event.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamera_source_event.h
index d28efd3f517db05a075508056064a105fdf745ef..b9285416a87493cf9abf2df419ddb3678b64cc01 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamera_source_event.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamera_source_event.h
@@ -56,7 +56,7 @@ class DCameraSourceEvent : public Event {
     TYPEINDENT(DCameraSourceEvent)
 
 public:
-    DCameraSourceEvent(EventSender& sender) : Event(sender) {}
+    explicit DCameraSourceEvent(EventSender& sender) : Event(sender) {}
     ~DCameraSourceEvent() = default;
     explicit DCameraSourceEvent(EventSender& sender, DCAMERA_EVENT eventType)
         : Event(sender), eventType_(eventType) {}
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcameracontrol/dcamera_source_controller_channel_listener.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcameracontrol/dcamera_source_controller_channel_listener.h
index 0a4e1fe155bd0d0a78e6cdc04372b993acaf8e87..8a7e7ace9d724085729d3c9f316c8e19aaef43d3 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcameracontrol/dcamera_source_controller_channel_listener.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcameracontrol/dcamera_source_controller_channel_listener.h
@@ -25,7 +25,7 @@ namespace DistributedHardware {
 class DCameraSourceController;
 class DCameraSourceControllerChannelListener : public ICameraChannelListener {
 public:
-    DCameraSourceControllerChannelListener(std::shared_ptr<DCameraSourceController>& controller);
+    explicit DCameraSourceControllerChannelListener(std::shared_ptr<DCameraSourceController>& controller);
     ~DCameraSourceControllerChannelListener();
     void OnSessionState(int32_t state) override;
     void OnSessionError(int32_t eventType, int32_t eventReason, std::string detail) override;
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcameradata/dcamera_stream_data_process_pipeline_listener.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcameradata/dcamera_stream_data_process_pipeline_listener.h
index 2c6cf8a475696d3f0a7cbc7535bf587e1c9ce756..c1832c081402c3918f8b6a3dac6b738226e124f1 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcameradata/dcamera_stream_data_process_pipeline_listener.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcameradata/dcamera_stream_data_process_pipeline_listener.h
@@ -26,7 +26,7 @@ namespace DistributedHardware {
 class DCameraStreamDataProcess;
 class DCameraStreamDataProcessPipelineListener : public DataProcessListener {
 public:
-    DCameraStreamDataProcessPipelineListener(std::shared_ptr<DCameraStreamDataProcess>& process);
+    explicit DCameraStreamDataProcessPipelineListener(std::shared_ptr<DCameraStreamDataProcess>& process);
     ~DCameraStreamDataProcessPipelineListener();
 
     void OnProcessedVideoBuffer(const std::shared_ptr<DataBuffer>& videoResult) override;
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_capture_state.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_capture_state.h
index 9614f132950dff33daaf9c1090e480da3d3019d0..f9a35cb0248b2cf13166e3b604903d6134bc2226 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_capture_state.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_capture_state.h
@@ -23,7 +23,7 @@ namespace OHOS {
 namespace DistributedHardware {
 class DCameraSourceCaptureState : public DCameraSourceState {
 public:
-    DCameraSourceCaptureState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
+    explicit DCameraSourceCaptureState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
     ~DCameraSourceCaptureState() {}
 
     int32_t Execute(std::shared_ptr<DCameraSourceDev>& camDev, DCAMERA_EVENT eventType,
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_config_stream_state.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_config_stream_state.h
index ab7a1f30a0be3249410123633dee172697cd8060..5d9186c48ae6e3d19e50fe79640ed398b26a19bd 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_config_stream_state.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_config_stream_state.h
@@ -23,7 +23,7 @@ namespace OHOS {
 namespace DistributedHardware {
 class DCameraSourceConfigStreamState : public DCameraSourceState {
 public:
-    DCameraSourceConfigStreamState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
+    explicit DCameraSourceConfigStreamState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
     DCameraSourceConfigStreamState() {}
     ~DCameraSourceConfigStreamState() {}
 
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_init_state.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_init_state.h
index 1fd0459340d0db428ca33b15d4167354f8a3c405..5ecbb096169d92654738a9e945ea2370822195d3 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_init_state.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_init_state.h
@@ -23,7 +23,7 @@ namespace OHOS {
 namespace DistributedHardware {
 class DCameraSourceInitState : public DCameraSourceState {
 public:
-    DCameraSourceInitState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
+    explicit DCameraSourceInitState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
     ~DCameraSourceInitState() {}
 
     int32_t Execute(std::shared_ptr<DCameraSourceDev>& camDev, DCAMERA_EVENT eventType,
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_opened_state.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_opened_state.h
index 51d68b2765ea7837894ee93c51aae9f34e8c7e84..339e497b3d27b20f41e5f2323abf47c95f78d9fd 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_opened_state.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_opened_state.h
@@ -23,7 +23,7 @@ namespace OHOS {
 namespace DistributedHardware {
 class DCameraSourceOpenedState : public DCameraSourceState {
 public:
-    DCameraSourceOpenedState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
+    explicit DCameraSourceOpenedState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
     ~DCameraSourceOpenedState() {}
 
     int32_t Execute(std::shared_ptr<DCameraSourceDev>& camDev, DCAMERA_EVENT eventType,
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_regist_state.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_regist_state.h
index 01053d33b5cbeae6a1172bcc60beea4a3519a78f..6b01d437997ec0d203c55eb1d603a8eef36a5dbc 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_regist_state.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_regist_state.h
@@ -23,7 +23,7 @@ namespace OHOS {
 namespace DistributedHardware {
 class DCameraSourceRegistState : public DCameraSourceState {
 public:
-    DCameraSourceRegistState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
+    explicit DCameraSourceRegistState(std::shared_ptr<DCameraSourceStateMachine>& stateMachine);
     ~DCameraSourceRegistState() {}
 
     int32_t Execute(std::shared_ptr<DCameraSourceDev>& camDev, DCAMERA_EVENT eventType,
diff --git a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_state_machine.h b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_state_machine.h
index 5056a9e300243e76731f41187ef903a42b271c7d..8d5d30347e959fa901224af733b5087c33fa361b 100644
--- a/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_state_machine.h
+++ b/services/cameraservice/sourceservice/include/distributedcameramgr/dcamerastate/dcamera_source_state_machine.h
@@ -23,7 +23,7 @@ namespace DistributedHardware {
 class DCameraSourceDev;
 class DCameraSourceStateMachine : public std::enable_shared_from_this<DCameraSourceStateMachine> {
 public:
-    DCameraSourceStateMachine(std::shared_ptr<DCameraSourceDev>& camDev);
+    explicit DCameraSourceStateMachine(std::shared_ptr<DCameraSourceDev>& camDev);
     ~DCameraSourceStateMachine();
     int32_t Execute(DCAMERA_EVENT eventType, DCameraSourceEvent& event);
     void UpdateState(DCameraStateType stateType);
diff --git a/services/cameraservice/sourceservice/src/distributedcameramgr/dcamera_source_dev.cpp b/services/cameraservice/sourceservice/src/distributedcameramgr/dcamera_source_dev.cpp
index 994e488f1ec1e396a821f36f3352aec2647edb36..ff476e1329f45510671e70832482c4c90758e6fd 100644
--- a/services/cameraservice/sourceservice/src/distributedcameramgr/dcamera_source_dev.cpp
+++ b/services/cameraservice/sourceservice/src/distributedcameramgr/dcamera_source_dev.cpp
@@ -72,6 +72,10 @@ int32_t DCameraSourceDev::InitDCameraSourceDev()
     controller_ = std::make_shared<DCameraSourceController>(devId_, dhId_, stateMachine_, eventBus_);
     input_ = std::make_shared<DCameraSourceInput>(devId_, dhId_, eventBus_);
     hdiCallback_ = new (std::nothrow) DCameraProviderCallbackImpl(devId_, dhId_, cameraSourceDev);
+    if (hdiCallback_ == nullptr) {
+        DHLOGE("DCameraSourceDev InitDCameraSourceDev failed, hdiCallback is null.");
+        return DCAMERA_BAD_VALUE;
+    }
     DHLOGI("DCameraSourceDev InitDCameraSourceDev end devId %s dhId %s", GetAnonyString(devId_).c_str(),
         GetAnonyString(dhId_).c_str());
     return DCAMERA_OK;