diff --git a/frameworks/innerkits/file_access/include/ifile_access_ext_base.h b/frameworks/innerkits/file_access/include/ifile_access_ext_base.h
index 5c79a87b62b8fea0fa0cdb801b4171aeacd876d4..fdc837b8ce8d9612634cd5570e0555cb1e7845a8 100644
--- a/frameworks/innerkits/file_access/include/ifile_access_ext_base.h
+++ b/frameworks/innerkits/file_access/include/ifile_access_ext_base.h
@@ -32,13 +32,13 @@ public:
 
     enum {
         CMD_OPEN_FILE = 1,
-        CMD_CREATE_FILE = 2,
-        CMD_MKDIR = 3,
-        CMD_DELETE = 4,
-        CMD_MOVE = 5,
-        CMD_RENAME = 6,
-        CMD_LIST_FILE = 7,
-        CMD_GET_ROOTS = 8
+        CMD_CREATE_FILE,
+        CMD_MKDIR,
+        CMD_DELETE,
+        CMD_MOVE,
+        CMD_RENAME,
+        CMD_LIST_FILE,
+        CMD_GET_ROOTS
     };
 
     virtual int OpenFile(const Uri &uri, int flags) = 0;
diff --git a/frameworks/innerkits/file_access/src/file_access_helper.cpp b/frameworks/innerkits/file_access/src/file_access_helper.cpp
index 39dd4886d3e169e81db4e2939bdfe224255cb2bc..9a3a497a74c6272f60f5fba99355966be7e97a64 100644
--- a/frameworks/innerkits/file_access/src/file_access_helper.cpp
+++ b/frameworks/innerkits/file_access/src/file_access_helper.cpp
@@ -63,6 +63,11 @@ FileAccessHelper::FileAccessHelper(const sptr<IRemoteObject> &token,
 
 std::shared_ptr<FileAccessHelper> FileAccessHelper::Creator(const sptr<IRemoteObject> &token, const AAFwk::Want &want)
 {
+    if (token == nullptr) {
+        HILOG_ERROR("FileAccessHelper::Creator failed, token is nullptr");
+        return nullptr;
+    }
+
     sptr<IFileAccessExtBase> fileAccessExtProxy = nullptr;
 
     sptr<FileAccessExtConnection> fileAccessExtConnection = FileAccessExtConnection::GetInstance();
@@ -88,7 +93,7 @@ std::shared_ptr<FileAccessHelper> FileAccessHelper::Creator(
     const std::shared_ptr<OHOS::AbilityRuntime::Context> &context, const AAFwk::Want &want)
 {
     if (context == nullptr) {
-        HILOG_ERROR("failed, context == nullptr");
+        HILOG_ERROR("FileAccessHelper::Creator failed, context is nullptr");
         return nullptr;
     }
 
@@ -114,6 +119,10 @@ std::shared_ptr<FileAccessHelper> FileAccessHelper::Creator(
 
 bool FileAccessHelper::Release()
 {
+    if (fileAccessExtConnection_ == nullptr) {
+        return false;
+    }
+
     if (fileAccessExtConnection_->IsExtAbilityConnected()) {
         fileAccessExtConnection_->DisconnectFileExtAbility();
     }
@@ -123,6 +132,10 @@ bool FileAccessHelper::Release()
 
 bool FileAccessHelper::GetProxy()
 {
+    if (fileAccessExtConnection_ == nullptr) {
+        return false;
+    }
+
     if (!fileAccessExtConnection_->IsExtAbilityConnected()) {
         fileAccessExtConnection_->ConnectFileExtAbility(want_, token_);
     }
diff --git a/frameworks/innerkits/file_access/src/napi_common_fileaccess.cpp b/frameworks/innerkits/file_access/src/napi_common_fileaccess.cpp
index b8a02f7b60e53a1d3b7c3b7a1f612b26bc68ac7b..7f0075c38ebe18554355c3d9f90d00a95242b2ad 100644
--- a/frameworks/innerkits/file_access/src/napi_common_fileaccess.cpp
+++ b/frameworks/innerkits/file_access/src/napi_common_fileaccess.cpp
@@ -112,27 +112,21 @@ napi_value WrapFileInfo(napi_env env, const FileInfo &fileInfo)
 
     NAPI_CALL(env, napi_create_object(env, &jsObject));
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapStringToJS(env, fileInfo.uri.ToString());
     SetPropertyValueByPropertyName(env, jsObject, "uri", jsValue);
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapStringToJS(env, fileInfo.fileName);
     SetPropertyValueByPropertyName(env, jsObject, "fileName", jsValue);
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapStringToJS(env, fileInfo.mode);
     SetPropertyValueByPropertyName(env, jsObject, "mode", jsValue);
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapInt64ToJS(env, fileInfo.size);
     SetPropertyValueByPropertyName(env, jsObject, "size", jsValue);
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapInt64ToJS(env, fileInfo.mtime);
     SetPropertyValueByPropertyName(env, jsObject, "mtime", jsValue);
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapStringToJS(env, fileInfo.mimeType);
     SetPropertyValueByPropertyName(env, jsObject, "mimeType", jsValue);
 
@@ -150,12 +144,10 @@ bool UnwrapFileInfo(napi_env env, napi_value param, FileInfo &fileInfo)
         fileInfo.uri = Uri(natValueString);
     }
 
-    natValueString = "";
     if (OHOS::AppExecFwk::UnwrapStringByPropertyName(env, param, "fileName", natValueString)) {
         fileInfo.fileName = natValueString;
     }
 
-    natValueString = "";
     if (OHOS::AppExecFwk::UnwrapStringByPropertyName(env, param, "mode", natValueString)) {
         fileInfo.mode = natValueString;
     }
@@ -165,12 +157,10 @@ bool UnwrapFileInfo(napi_env env, napi_value param, FileInfo &fileInfo)
         fileInfo.size = natValueInt64;
     }
 
-    natValueInt64 = ERR_OK;
     if (UnwrapInt64ByPropertyName(env, param, "mtime", natValueInt64)) {
         fileInfo.mtime = natValueInt64;
     }
 
-    natValueString = "";
     if (OHOS::AppExecFwk::UnwrapStringByPropertyName(env, param, "mimeType", natValueString)) {
         fileInfo.mimeType = natValueString;
     }
@@ -226,19 +216,15 @@ napi_value WrapDeviceInfo(napi_env env, const DeviceInfo &deviceInfo)
 
     NAPI_CALL(env, napi_create_object(env, &jsObject));
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapStringToJS(env, deviceInfo.uri.ToString());
     SetPropertyValueByPropertyName(env, jsObject, "uri", jsValue);
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapStringToJS(env, deviceInfo.displayName);
     SetPropertyValueByPropertyName(env, jsObject, "displayName", jsValue);
 
-    jsValue = nullptr;
     jsValue = OHOS::AppExecFwk::WrapStringToJS(env, deviceInfo.deviceId);
     SetPropertyValueByPropertyName(env, jsObject, "deviceId", jsValue);
 
-    jsValue = nullptr;
     jsValue = WrapUint32ToJS(env, deviceInfo.flags);
     SetPropertyValueByPropertyName(env, jsObject, "flags", jsValue);
 
@@ -256,12 +242,10 @@ bool UnwrapDeviceInfo(napi_env env, napi_value param, DeviceInfo &deviceInfo)
         deviceInfo.uri = Uri(natValueString);
     }
 
-    natValueString = "";
     if (OHOS::AppExecFwk::UnwrapStringByPropertyName(env, param, "displayName", natValueString)) {
         deviceInfo.displayName = natValueString;
     }
 
-    natValueString = "";
     if (OHOS::AppExecFwk::UnwrapStringByPropertyName(env, param, "deviceId", natValueString)) {
         deviceInfo.deviceId = natValueString;
     }