diff --git a/interfaces/innerkits/form_manager/src/form_js_info.cpp b/interfaces/innerkits/form_manager/src/form_js_info.cpp
index 77f95e996a33d0392a0571afee7cbd63e6d78e0d..cec9f67fecc0f65e334cb5a91f1941a84c01d813 100644
--- a/interfaces/innerkits/form_manager/src/form_js_info.cpp
+++ b/interfaces/innerkits/form_manager/src/form_js_info.cpp
@@ -151,12 +151,12 @@ bool FormJsInfo::WriteImageData(Parcel &parcel) const
                 HILOG_INFO("%{public}s unexpected image number %{public}zu", __func__, size);
                 break;
             }
-            auto messageParcel = static_cast<MessageParcel*>(&parcel);
             for (auto entry : sharedImageMap) {
+                auto messageParcel = static_cast<MessageParcel*>(&parcel);
                 if (!messageParcel->WriteAshmem(entry.second.first)) {
                     return false;
                 }
-                if (parcel.WriteString16(Str8ToStr16(entry.first))) {
+                if (!parcel.WriteString16(Str8ToStr16(entry.first))) {
                     return false;
                 }
             }
@@ -188,8 +188,8 @@ void FormJsInfo::ReadImageData(Parcel &parcel)
                 HILOG_WARN("%{public}s unexpected image number %{public}d", __func__, size);
                 break;
             }
-            auto messageParcel = static_cast<MessageParcel*>(&parcel);
             for (auto i = 0; i < size; i++) {
+                auto messageParcel = static_cast<MessageParcel*>(&parcel);
                 auto fd = messageParcel->ReadFileDescriptor();
                 auto len = parcel.ReadInt32();
                 auto picName = Str16ToStr8(parcel.ReadString16());