Partial fix for T43113: Filebrowser: Empty folders do not contain go back arrow.

Do not allow going into un-readable directories at all.
Note we might want to reflect that 'state' in UI for users too, but that will be
for later.

Also, not quite sure this fix the windows case, will have to start my VM... :/
This commit is contained in:
Bastien Montagne 2015-01-04 17:54:12 +01:00
parent 8abdc89912
commit a08c5e1183
Notes: blender-bot 2023-02-14 09:17:53 +01:00
Referenced by issue #44235, UNC Path Fails in open
2 changed files with 12 additions and 4 deletions

View File

@ -1087,18 +1087,22 @@ void BLI_char_switch(char *string, char from, char to)
}
/**
* Strips off nonexistent subdirectories from the end of *dir, leaving the path of
* the lowest-level directory that does exist.
* Strips off nonexistent (or non-accessible) subdirectories from the end of *dir, leaving the path of
* the lowest-level directory that does exist and we can read.
*/
void BLI_make_exist(char *dir)
{
int a;
char par_path[PATH_MAX + 3];
BLI_char_switch(dir, ALTSEP, SEP);
a = strlen(dir);
while (!BLI_is_dir(dir)) {
for (BLI_join_dirfile(par_path, sizeof(par_path), dir, "..");
!(BLI_is_dir(dir) && BLI_exists(par_path));
BLI_join_dirfile(par_path, sizeof(par_path), dir, ".."))
{
a--;
while (dir[a] != SEP) {
a--;

View File

@ -973,9 +973,14 @@ static void filelist_read_dir(struct FileList *filelist)
filelist->fidx = NULL;
filelist->filelist = NULL;
BLI_make_exist(filelist->dir);
BLI_cleanup_dir(G.main->name, filelist->dir);
filelist->numfiles = BLI_filelist_dir_contents(filelist->dir, &(filelist->filelist));
/* We shall *never* get an empty list here, since we now the dir exists and is readable
* (ensured by BLI_make_exist()). So we expect at the very least the parent '..' entry. */
BLI_assert(filelist->numfiles != 0);
filelist_setfiletypes(filelist);
}
@ -994,7 +999,6 @@ static void filelist_read_library(struct FileList *filelist)
int num;
struct direntry *file;
BLI_make_exist(filelist->dir);
filelist_read_dir(filelist);
file = filelist->filelist;
for (num = 0; num < filelist->numfiles; num++, file++) {