diff --git a/display/composer/v1_0/display_command/display_cmd_responser.h b/display/composer/v1_0/display_command/display_cmd_responser.h
index aeea58c53dc006c7d25b392dfb57fed9bd3b8bbc..1987f60eb8c24fe1ae56691ee65cc79d1285df7c 100755
--- a/display/composer/v1_0/display_command/display_cmd_responser.h
+++ b/display/composer/v1_0/display_command/display_cmd_responser.h
@@ -164,6 +164,8 @@ public:
         int32_t ret = request_->Read(reinterpret_cast<int32_t *>(requestData.get()), inEleCnt,
             CmdUtils::TRANSFER_WAIT_TIME);
 
+        DISPLAY_CHK_RETURN(ret != HDF_SUCCESS, HDF_FAILURE,
+            HDF_LOGE("%{public}s: error: Read requestData failed", __func__));
         CommandDataUnpacker unpacker;
         unpacker.Init(requestData.get(), inEleCnt << CmdUtils::MOVE_SIZE);
 #ifdef DEBUG_DISPLAY_CMD_RAW_DATA
@@ -178,7 +180,7 @@ public:
             HDF_LOGI("error: unpacker PackBegin cmd not match, cmd(%{public}d)=%{public}s.", unpackCmd,
             CmdUtils::CommandToString(unpackCmd)));
 
-        while (ret == HDF_SUCCESS && unpacker.NextSection()) {
+        while (unpacker.NextSection()) {
             if (!unpacker.BeginSection(unpackCmd)) {
                 HDF_LOGE("error: PackSection failed, unpackCmd=%{public}s.",
                     CmdUtils::CommandToString(unpackCmd));