diff --git a/camera_hdf/interfaces/hdi_ipc/server/host/dcamera_host_stub.cpp b/camera_hdf/interfaces/hdi_ipc/server/host/dcamera_host_stub.cpp
index e75663686536048583e5e21b5cd410e0cff0a783..ba7e633fd5b0a737a6f0bc1944a07231b2da6090 100644
--- a/camera_hdf/interfaces/hdi_ipc/server/host/dcamera_host_stub.cpp
+++ b/camera_hdf/interfaces/hdi_ipc/server/host/dcamera_host_stub.cpp
@@ -110,13 +110,17 @@ int32_t DCameraHostStub::DCHostStubGetCameraAbility(MessageParcel &data, Message
         return HDF_FAILURE;
     }
 
-    if (ret == CamRetCode::NO_ERROR) {
-        bool bRet = Camera::MetadataUtils::EncodeCameraMetadata(ability, reply);
-        if (!bRet) {
-            DHLOGE("Write ability failed.");
-            return HDF_FAILURE;
-        }
+    if (ret != CamRetCode::NO_ERROR) {
+        DHLOGE("Getting camera ability failed.");
+        return HDF_FAILURE;
     }
+
+    bool bRet = Camera::MetadataUtils::EncodeCameraMetadata(ability, reply);
+    if (!bRet) {
+        DHLOGE("Write ability failed.");
+        return HDF_FAILURE;
+    }
+
     return HDF_SUCCESS;
 }