Another take at fixing symlinks
This commit is contained in:
parent
ded74dedef
commit
dccdef1327
|
@ -211,36 +211,38 @@ Error DirAccessUnix::make_dir(String p_dir) {
|
|||
Error DirAccessUnix::change_dir(String p_dir) {
|
||||
|
||||
GLOBAL_LOCK_FUNCTION
|
||||
p_dir = fix_path(p_dir);
|
||||
|
||||
char real_current_dir_name[2048];
|
||||
getcwd(real_current_dir_name, 2048);
|
||||
String prev_dir;
|
||||
if (prev_dir.parse_utf8(real_current_dir_name))
|
||||
prev_dir = real_current_dir_name; //no utf8, maybe latin?
|
||||
|
||||
chdir(current_dir.utf8().get_data()); //ascii since this may be unicode or wathever the host os wants
|
||||
bool worked = (chdir(p_dir.utf8().get_data()) == 0); // we can only give this utf8
|
||||
|
||||
String base = _get_root_path();
|
||||
if (base != "") {
|
||||
|
||||
// make sure current_dir is valid absolute path
|
||||
if (current_dir == "." || current_dir == "") {
|
||||
char real_current_dir_name[2048];
|
||||
getcwd(real_current_dir_name, 2048);
|
||||
String new_dir;
|
||||
new_dir.parse_utf8(real_current_dir_name);
|
||||
if (!new_dir.begins_with(base))
|
||||
worked = false;
|
||||
current_dir.parse_utf8(real_current_dir_name);
|
||||
}
|
||||
|
||||
if (worked) {
|
||||
if (p_dir == ".") {
|
||||
return OK;
|
||||
}
|
||||
|
||||
getcwd(real_current_dir_name, 2048);
|
||||
if (current_dir.parse_utf8(real_current_dir_name))
|
||||
current_dir = real_current_dir_name; //no utf8, maybe latin?
|
||||
p_dir = fix_path(p_dir);
|
||||
|
||||
String prev_dir = current_dir;
|
||||
|
||||
if (p_dir.is_rel_path()) {
|
||||
String next_dir = current_dir + "/" + p_dir;
|
||||
next_dir = next_dir.simplify_path();
|
||||
current_dir = next_dir;
|
||||
} else {
|
||||
current_dir = p_dir;
|
||||
}
|
||||
|
||||
bool worked = (chdir(current_dir.utf8().get_data()) == 0); // we can only give this utf8
|
||||
if (!worked) {
|
||||
current_dir = prev_dir;
|
||||
return ERR_INVALID_PARAMETER;
|
||||
}
|
||||
|
||||
chdir(prev_dir.utf8().get_data());
|
||||
return worked ? OK : ERR_INVALID_PARAMETER;
|
||||
return OK;
|
||||
}
|
||||
|
||||
String DirAccessUnix::get_current_dir() {
|
||||
|
|
Loading…
Reference in New Issue