summaryrefslogtreecommitdiff
path: root/src/fs/paths.cpp
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2017-03-11 21:39:42 +0300
committerAndrei Karas <akaras@inbox.ru>2017-03-11 21:39:42 +0300
commita721e4898f71a68ecb916940864599d462880e5b (patch)
tree50442e8b86707221f498ea0f304a5314a0e67430 /src/fs/paths.cpp
parent81b97b30afa0c72e5ff17162c7d1fa88dd6187fb (diff)
downloadmv-a721e4898f71a68ecb916940864599d462880e5b.tar.gz
mv-a721e4898f71a68ecb916940864599d462880e5b.tar.bz2
mv-a721e4898f71a68ecb916940864599d462880e5b.tar.xz
mv-a721e4898f71a68ecb916940864599d462880e5b.zip
Fix different issues found by automatic checks.
Diffstat (limited to 'src/fs/paths.cpp')
-rw-r--r--src/fs/paths.cpp14
1 files changed, 6 insertions, 8 deletions
diff --git a/src/fs/paths.cpp b/src/fs/paths.cpp
index 06e87c2f0..050b9d00c 100644
--- a/src/fs/paths.cpp
+++ b/src/fs/paths.cpp
@@ -78,11 +78,11 @@ std::string getRealPath(const std::string &str)
// defined(__native_client__)
char *realPath = realpath(str.c_str(), nullptr);
+ if (!realPath)
+ return "";
#endif // defined(__OpenBSD__) || defined(__ANDROID__) ||
// defined(__native_client__)
- if (!realPath)
- return "";
std::string path = realPath;
free(realPath);
return path;
@@ -105,7 +105,7 @@ bool checkPath(const std::string &path)
void prepareFsPath(std::string &path)
{
- std::string path2 = path;
+// std::string path2 = path;
sanitizePath(path);
// can be enabled for debugging
// if (path != path2)
@@ -231,11 +231,9 @@ std::string getHomePath()
const char *path = getenv("HOME");
if (path == nullptr)
{
- uid_t uid = getuid();
- struct passwd *pw;
-
- pw = getpwuid(uid);
- if (pw != NULL &&
+ const uid_t uid = getuid();
+ const struct passwd *const pw = getpwuid(uid);
+ if (pw != nullptr &&
pw->pw_dir != nullptr)
{
path = pw->pw_dir;