From b7f91d164e64dcbf20f4b2eb3d6c04e4ceb1c0d9 Mon Sep 17 00:00:00 2001 From: liyuke Date: Wed, 2 Jul 2025 11:24:43 +0800 Subject: [PATCH] movefile and movedir cancel access Signed-off-by: liyuke Change-Id: I5dbec63f1a02a5937c3a258bd34395f72487c7b5 --- interfaces/kits/js/src/mod_fs/properties/move_core.cpp | 8 +------- interfaces/kits/js/src/mod_fs/properties/movedir_core.cpp | 4 ---- 2 files changed, 1 insertion(+), 11 deletions(-) diff --git a/interfaces/kits/js/src/mod_fs/properties/move_core.cpp b/interfaces/kits/js/src/mod_fs/properties/move_core.cpp index 56961567c..a0b823ea1 100644 --- a/interfaces/kits/js/src/mod_fs/properties/move_core.cpp +++ b/interfaces/kits/js/src/mod_fs/properties/move_core.cpp @@ -147,14 +147,8 @@ static int RenameFile(const string &src, const string &dest) static int MoveFile(const string &src, const string &dest, int mode) { uv_fs_t access_req; - int ret = uv_fs_access(nullptr, &access_req, src.c_str(), W_OK, nullptr); - if (ret < 0) { - HILOGE("Failed to move src file due to doesn't exist or hasn't write permission"); - uv_fs_req_cleanup(&access_req); - return ret; - } if (mode == MODE_THROW_ERR) { - ret = uv_fs_access(nullptr, &access_req, dest.c_str(), 0, nullptr); + int ret = uv_fs_access(nullptr, &access_req, dest.c_str(), 0, nullptr); uv_fs_req_cleanup(&access_req); if (ret == 0) { HILOGE("Failed to move file due to existing destPath with MODE_THROW_ERR."); diff --git a/interfaces/kits/js/src/mod_fs/properties/movedir_core.cpp b/interfaces/kits/js/src/mod_fs/properties/movedir_core.cpp index fb357ff05..f882c1aa2 100644 --- a/interfaces/kits/js/src/mod_fs/properties/movedir_core.cpp +++ b/interfaces/kits/js/src/mod_fs/properties/movedir_core.cpp @@ -226,10 +226,6 @@ static int MoveDirFunc(const string &src, const string &dest, const int mode, de if (found == std::string::npos) { return EINVAL; } - if (access(src.c_str(), W_OK) != 0) { - HILOGE("Failed to move src directory due to doesn't exist or hasn't write permission"); - return errno; - } string dirName = string(src).substr(found); string destStr = dest + dirName; auto [destStrExist, destStrEmpty] = JudgeExistAndEmpty(destStr); -- Gitee