diff --git a/services/cameraservice/cameraoperator/client/include/dcamera_client.h b/services/cameraservice/cameraoperator/client/include/dcamera_client.h
index 1bf9d23b3350d546183d7d4ceda0ce88eacce82b..d0b1df0da7aec34bdc17ac65a1774b4f08adf261 100644
--- a/services/cameraservice/cameraoperator/client/include/dcamera_client.h
+++ b/services/cameraservice/cameraoperator/client/include/dcamera_client.h
@@ -66,6 +66,7 @@ private:
         std::shared_ptr<CameraStandard::PhotoCaptureSetting>& photoCaptureSetting);
     void ReleaseCaptureSession();
     int32_t CameraServiceErrorType(const int32_t errorType);
+    void UpdateSettingCache(const std::string& metadataStr);
 
 private:
     constexpr static uint32_t DCAMERA_MAX_METADATA_SIZE = 20;
diff --git a/services/cameraservice/cameraoperator/client/src/dcamera_client.cpp b/services/cameraservice/cameraoperator/client/src/dcamera_client.cpp
index 985abdf6309c4426533309fe7bc6c5aff779d0e9..d44c313b5a58c682d1d75df9b93ee3d857a510de 100644
--- a/services/cameraservice/cameraoperator/client/src/dcamera_client.cpp
+++ b/services/cameraservice/cameraoperator/client/src/dcamera_client.cpp
@@ -102,12 +102,7 @@ int32_t DCameraClient::UpdateSettings(std::vector<std::shared_ptr<DCameraSetting
 
                 if (cameraInput_ == nullptr) {
                     DHLOGE("DCameraClient::UpdateSettings %s cameraInput is null", GetAnonyString(cameraId_).c_str());
-                    if (cameraMetadatas_.size() == DCAMERA_MAX_METADATA_SIZE) {
-                        DHLOGE("DCameraClient::UpdateSettings %s camera metadata oversize",
-                            GetAnonyString(cameraId_).c_str());
-                        cameraMetadatas_.pop();
-                    }
-                    cameraMetadatas_.push(metadataStr);
+                    UpdateSettingCache(metadataStr);
                     return DCAMERA_OK;
                 }
 
@@ -129,6 +124,16 @@ int32_t DCameraClient::UpdateSettings(std::vector<std::shared_ptr<DCameraSetting
     return DCAMERA_OK;
 }
 
+void DCameraClient::UpdateSettingCache(const std::string& metadataStr)
+{
+    if (cameraMetadatas_.size() == DCAMERA_MAX_METADATA_SIZE) {
+        DHLOGE("DCameraClient::UpdateSettings %s camera metadata oversize",
+            GetAnonyString(cameraId_).c_str());
+        cameraMetadatas_.pop();
+    }
+    cameraMetadatas_.push(metadataStr);
+}
+
 void DCameraClient::FindCameraMetadata(const std::string& metadataStr)
 {
     std::shared_ptr<Camera::CameraMetadata> cameraMetadata = Camera::MetadataUtils::DecodeFromString(metadataStr);
diff --git a/services/cameraservice/cameraoperator/client/src/dcamera_client_common.cpp b/services/cameraservice/cameraoperator/client/src/dcamera_client_common.cpp
index f1c0bc19be38b316c4596e8b081bb4f95142507c..c2f351d856e483d432e4b221decc8d7a6c9f9a2d 100644
--- a/services/cameraservice/cameraoperator/client/src/dcamera_client_common.cpp
+++ b/services/cameraservice/cameraoperator/client/src/dcamera_client_common.cpp
@@ -103,6 +103,7 @@ int32_t DCameraClient::UpdateSettings(std::vector<std::shared_ptr<DCameraSetting
                 if (cameraInput_ == nullptr) {
                     DHLOGE("DCameraClientCommon::UpdateSettings %s cameraInput is null",
                         GetAnonyString(cameraId_).c_str());
+                    UpdateSettingCache(metadataStr);
                     return DCAMERA_BAD_VALUE;
                 }
 
@@ -124,6 +125,11 @@ int32_t DCameraClient::UpdateSettings(std::vector<std::shared_ptr<DCameraSetting
     return DCAMERA_OK;
 }
 
+void DCameraClient::UpdateSettingCache(const std::string& metadataStr)
+{
+    DHLOGE("DCameraClientCommon::UpdateSettingCache metadataStr %s", GetAnonyString(metadataStr).c_str());
+}
+
 void DCameraClient::FindCameraMetadata(const std::string& metadataStr)
 {
     DHLOGI("DCameraClient::FindCameraMetadata cameraId: %s", GetAnonyString(cameraId_).c_str());