diff --git a/code/ryzom/tools/client/ryzom_installer/src/configfile.cpp b/code/ryzom/tools/client/ryzom_installer/src/configfile.cpp index 88fae2388..8815396f7 100644 --- a/code/ryzom/tools/client/ryzom_installer/src/configfile.cpp +++ b/code/ryzom/tools/client/ryzom_installer/src/configfile.cpp @@ -502,6 +502,8 @@ bool CConfigFile::isRyzomInstalledIn(const QString &directory) const bool CConfigFile::areRyzomDataInstalledIn(const QString &directory) const { + if (directory.isEmpty()) return false; + QDir dir(directory); // directory doesn't exist @@ -528,6 +530,8 @@ bool CConfigFile::areRyzomDataInstalledIn(const QString &directory) const bool CConfigFile::isRyzomClientInstalledIn(const QString &directory) const { + if (directory.isEmpty()) return false; + QDir dir(directory); // directory doesn't exist @@ -557,6 +561,8 @@ bool CConfigFile::isRyzomClientInstalledIn(const QString &directory) const bool CConfigFile::foundTemporaryFiles(const QString &directory) const { + if (directory.isEmpty()) return false; + QDir dir(directory); // directory doesn't exist diff --git a/code/ryzom/tools/client/ryzom_installer/src/filescleaner.cpp b/code/ryzom/tools/client/ryzom_installer/src/filescleaner.cpp index e1481fcf9..79c42634a 100644 --- a/code/ryzom/tools/client/ryzom_installer/src/filescleaner.cpp +++ b/code/ryzom/tools/client/ryzom_installer/src/filescleaner.cpp @@ -52,6 +52,8 @@ void CFilesCleaner::setDirectory(const QString &src) bool CFilesCleaner::exec() { + if (m_directory.isEmpty()) return false; + if (m_listener) m_listener->operationPrepare(); QDir dir(m_directory); diff --git a/code/ryzom/tools/client/ryzom_installer/src/utils.cpp b/code/ryzom/tools/client/ryzom_installer/src/utils.cpp index b07db1e43..5ab73f298 100644 --- a/code/ryzom/tools/client/ryzom_installer/src/utils.cpp +++ b/code/ryzom/tools/client/ryzom_installer/src/utils.cpp @@ -37,23 +37,27 @@ QString qBytesToHumanReadable(qint64 bytes) qint64 getDirectorySize(const QString &directory) { qint64 size = 0; - QDir dir(directory); - if (dir.exists()) + if (!directory.isEmpty()) { - QFileInfoList list = dir.entryInfoList(QDir::Files | QDir::Dirs | QDir::Hidden | QDir::NoSymLinks | QDir::NoDotAndDotDot); + QDir dir(directory); - for (int i = 0; i < list.size(); ++i) + if (dir.exists()) { - QFileInfo fileInfo = list.at(i); + QFileInfoList list = dir.entryInfoList(QDir::Files | QDir::Dirs | QDir::Hidden | QDir::NoSymLinks | QDir::NoDotAndDotDot); - if (fileInfo.isDir()) + for (int i = 0; i < list.size(); ++i) { - size += getDirectorySize(fileInfo.absoluteFilePath()); - } - else - { - size += fileInfo.size(); + QFileInfo fileInfo = list.at(i); + + if (fileInfo.isDir()) + { + size += getDirectorySize(fileInfo.absoluteFilePath()); + } + else + { + size += fileInfo.size(); + } } } }