Fixed bug in 'os.rename()' function

Author: loboris

firmware/MaixPy.bin

@@ -658,10 +658,12 @@ STATIC MP_DEFINE_CONST_FUN_OBJ_2(littlefs_vfs_remove_obj, littlefs_vfs_remove);
 STATIC mp_obj_t littlefs_vfs_rename(mp_obj_t vfs_in, mp_obj_t path_in, mp_obj_t path_out)
 {
     littlefs_user_mount_t *self = MP_OBJ_TO_PTR(vfs_in);
+    char lnew_path[LITTLEFS_CFG_MAX_NAME_LEN] = {'\0'};
+    char lold_path[LITTLEFS_CFG_MAX_NAME_LEN] = {'\0'};
     const char *old_path = mp_obj_str_get_str(path_in);
     const char *new_path = mp_obj_str_get_str(path_out);
-    const char *lold_path = littlefs_local_path(old_path);
-    const char *lnew_path = littlefs_local_path(new_path);
+    strcpy(lold_path, littlefs_local_path(old_path));
+    strcpy(lnew_path, littlefs_local_path(new_path));
     int res;
 
     res = lfs_rename(&self->fs->lfs, lold_path, lnew_path);

firmware/MaixPy.kfpkg

@@ -435,10 +435,12 @@ STATIC MP_DEFINE_CONST_FUN_OBJ_2(sdcard_vfs_rmdir_obj, sdcard_vfs_rmdir);
 STATIC mp_obj_t sdcard_vfs_rename(mp_obj_t vfs_in, mp_obj_t path_in, mp_obj_t path_out)
 {
     mp_obj_sdcard_vfs_t *self = MP_OBJ_TO_PTR(vfs_in);
+    static char local_newpath[FF_MAX_LFN] = {'\0'};
+    static char local_oldpath[FF_MAX_LFN] = {'\0'};
     const char *old_path = mp_obj_str_get_str(path_in);
     const char *new_path = mp_obj_str_get_str(path_out);
-    const char *local_oldpath = sdcard_local_path(old_path, self);
-    const char *local_newpath = sdcard_local_path(new_path, self);
+    strcpy(local_oldpath, sdcard_local_path(old_path, self));
+    strcpy(local_newpath, sdcard_local_path(new_path, self));
 
     FRESULT res = f_rename(local_oldpath, local_newpath);
     if (res == FR_EXIST) {

firmware/MaixPy_firmware.zip

@@ -490,10 +490,12 @@ STATIC MP_DEFINE_CONST_FUN_OBJ_2(spiffs_vfs_remove_obj, spiffs_vfs_remove);
 STATIC mp_obj_t spiffs_vfs_rename(mp_obj_t vfs_in, mp_obj_t path_in, mp_obj_t path_out)
 {
     spiffs_user_mount_t *self = MP_OBJ_TO_PTR(vfs_in);
+    char lnew_path[SPIFFS_OBJ_NAME_LEN] = {'\0'};
+    char lold_path[SPIFFS_OBJ_NAME_LEN] = {'\0'};
     const char *old_path = mp_obj_str_get_str(path_in);
     const char *new_path = mp_obj_str_get_str(path_out);
-    const char *lold_path = spiffs_local_path(old_path);
-    const char *lnew_path = spiffs_local_path(new_path);
+    strcpy(lold_path, spiffs_local_path(old_path));
+    strcpy(lnew_path, spiffs_local_path(new_path));
     int res;
 
     res = SPIFFS_rename(&(self->fs), lold_path, lnew_path);