diff --git a/interfaces/inner_kits/native_cpp/camera_source/include/dcamera_source_handler.h b/interfaces/inner_kits/native_cpp/camera_source/include/dcamera_source_handler.h index 09d278f6fb3e9826aca767348635762238bff26a..c5f85e5f87d4bc5a4638729441eaf19ffb62504f 100644 --- a/interfaces/inner_kits/native_cpp/camera_source/include/dcamera_source_handler.h +++ b/interfaces/inner_kits/native_cpp/camera_source/include/dcamera_source_handler.h @@ -45,7 +45,7 @@ private: DCAMERA_SA_STATE_STOP = 0, DCAMERA_SA_STATE_START = 1, } DCameraSAState; - DCameraSourceHandler() = default; + DCameraSourceHandler(); ~DCameraSourceHandler(); void FinishStartSA(const std::string ¶ms); void FinishStartSAFailed(int32_t systemAbilityId); diff --git a/interfaces/inner_kits/native_cpp/camera_source/src/dcamera_source_handler.cpp b/interfaces/inner_kits/native_cpp/camera_source/src/dcamera_source_handler.cpp index 0cbb17a8dbe61c7d0e435278ec8386c9a28beb84..5c520b2b0d311cf42b60d9b98fb995fb940df6c4 100644 --- a/interfaces/inner_kits/native_cpp/camera_source/src/dcamera_source_handler.cpp +++ b/interfaces/inner_kits/native_cpp/camera_source/src/dcamera_source_handler.cpp @@ -30,6 +30,12 @@ namespace OHOS { namespace DistributedHardware { IMPLEMENT_SINGLE_INSTANCE(DCameraSourceHandler); +DCameraSourceHandler::DCameraSourceHandler() +{ + DHLOGI("DCameraSourceHandler construct."); + callback_ = new DCameraSourceCallback(); +} + DCameraSourceHandler::~DCameraSourceHandler() { DHLOGI("~DCameraSourceHandler"); @@ -72,11 +78,6 @@ void DCameraSourceHandler::FinishStartSA(const std::string ¶ms) return; } - callback_ = new DCameraSourceCallback(); - if (callback_ == nullptr) { - DHLOGE("DCameraSourceHandler InitSource init callback failed"); - return; - } dCameraSourceSrv->InitSource(params, callback_); std::unique_lock lock(producerMutex_); state_ = DCAMERA_SA_STATE_START; @@ -122,10 +123,6 @@ int32_t DCameraSourceHandler::RegisterDistributedHardware(const std::string& dev std::lock_guard autoLock(optLock_); std::string reqId = GetRandomID(); - if (callback_ == nullptr) { - DHLOGE("DCameraSourceHandler RegisterDistributedHardware DCameraSourceCallback is null."); - return DCAMERA_BAD_VALUE; - } callback_->PushRegCallback(reqId, callback); int32_t ret = dCameraSourceSrv->RegisterDistributedHardware(devId, dhId, reqId, param); if (ret != DCAMERA_OK) { @@ -150,10 +147,6 @@ int32_t DCameraSourceHandler::UnregisterDistributedHardware(const std::string& d std::lock_guard autoLock(optLock_); std::string reqId = GetRandomID(); - if (callback_ == nullptr) { - DHLOGE("DCameraSourceHandler RegisterDistributedHardware DCameraSourceCallback is null."); - return DCAMERA_BAD_VALUE; - } callback_->PushUnregCallback(reqId, callback); int32_t ret = dCameraSourceSrv->UnregisterDistributedHardware(devId, dhId, reqId); if (ret != DCAMERA_OK) {