Commit 3aaf013d authored by Max Kellermann's avatar Max Kellermann

ConfigPath: convert "home" variable to Path object

Use Path::FromUTF8() for the g_get_home_dir() return value instead of assuming it's already FS charset.
parent 5f2705ab
...@@ -34,37 +34,39 @@ ...@@ -34,37 +34,39 @@
/** /**
* Determine a given user's home directory. * Determine a given user's home directory.
*/ */
static const char * static Path
GetHome(const char *user, Error &error) GetHome(const char *user, Error &error)
{ {
passwd *pw = getpwnam(user); passwd *pw = getpwnam(user);
if (pw == nullptr) { if (pw == nullptr) {
error.Format(path_domain, error.Format(path_domain,
"no such user: %s", user); "no such user: %s", user);
return nullptr; return Path::Null();
} }
return pw->pw_dir; return Path::FromFS(pw->pw_dir);
} }
/** /**
* Determine the current user's home directory. * Determine the current user's home directory.
*/ */
static const char * static Path
GetHome(Error &error) GetHome(Error &error)
{ {
const char *home = g_get_home_dir(); const char *home = g_get_home_dir();
if (home == nullptr) if (home == nullptr) {
error.Set(path_domain, error.Set(path_domain,
"problems getting home for current user"); "problems getting home for current user");
return Path::Null();
}
return home; return Path::FromUTF8(home, error);
} }
/** /**
* Determine the configured user's home directory. * Determine the configured user's home directory.
*/ */
static const char * static Path
GetConfiguredHome(Error &error) GetConfiguredHome(Error &error)
{ {
const char *user = config_get_string(CONF_USER, nullptr); const char *user = config_get_string(CONF_USER, nullptr);
...@@ -90,7 +92,7 @@ ParsePath(const char *path, Error &error) ...@@ -90,7 +92,7 @@ ParsePath(const char *path, Error &error)
"not an absolute path: %s", path); "not an absolute path: %s", path);
return Path::Null(); return Path::Null();
} else if (path[0] == '~') { } else if (path[0] == '~') {
const char *home; Path home = Path::Null();
if (path[1] == '/' || path[1] == '\0') { if (path[1] == '/' || path[1] == '\0') {
home = GetConfiguredHome(error); home = GetConfiguredHome(error);
...@@ -110,7 +112,7 @@ ParsePath(const char *path, Error &error) ...@@ -110,7 +112,7 @@ ParsePath(const char *path, Error &error)
path = slash; path = slash;
} }
if (home == nullptr) if (home.IsNull())
return Path::Null(); return Path::Null();
return Path::Build(home, path2); return Path::Build(home, path2);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment