Xargon/ES-DE
1
0
Fork 0
mirror of https://github.com/RetroDECK/ES-DE.git synced 2025-02-16 20:15:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix getHomePath crash when the environment variable isn't set

Browse source
This commit is contained in:
Tomas Jakobsson 2018-02-20 19:15:34 +01:00
parent 6f44ef9011
commit 33086042f9
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
es-core/src/utils/FileSystemUtil.cpp
View file

@ -143,21 +143,24 @@ namespace Utils
if(!path.length()) if(!path.length())
{ {
// this should give us something like "/home/YOUR_USERNAME" on Linux and "C:/Users/YOUR_USERNAME/" on Windows // this should give us something like "/home/YOUR_USERNAME" on Linux and "C:/Users/YOUR_USERNAME/" on Windows
std::string envHome(getenv("HOME")); char* envHome = getenv("HOME");
if(envHome.length()) if(envHome)
path = getGenericPath(envHome); path = getGenericPath(envHome);
#if defined(_WIN32) #if defined(_WIN32)
// but does not seem to work for Windows XP or Vista, so try something else // but does not seem to work for Windows XP or Vista, so try something else
if(!path.length()) if(!path.length())
{ {
std::string envDir(getenv("HOMEDRIVE")); char* envHomeDrive = getenv("HOMEDRIVE");
std::string envPath(getenv("HOMEPATH")); char* envHomePath = getenv("HOMEPATH");
if(envDir.length() && envPath.length()) if(envHomeDrive && envHomePath)
path = getGenericPath(envDir + "/" + envPath); path = getGenericPath(std::string(envHomeDrive) + "/" + envHomePath);
} }
#endif // _WIN32 #endif // _WIN32
// no homepath found, fall back to current working directory
if(!path.length())
path = getCWDPath();
} }
// return constructed homepath // return constructed homepath