From 4b5348ba04a62335757d439aa1acb8a3579c267d Mon Sep 17 00:00:00 2001 From: Dominik Riebeling Date: Fri, 11 May 2012 22:05:09 +0200 Subject: [PATCH] Don't try to resolve an empty path. An empty path is not resolvable, so immediately return an empty result. Fixes an empty path resolve to "/" on Linux / OS X. Change-Id: I22c26a7716becbf46850a04e18ee581084546263 --- rbutil/rbutilqt/base/utils.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/rbutil/rbutilqt/base/utils.cpp b/rbutil/rbutilqt/base/utils.cpp index 5c80e1225c..60bacd709e 100644 --- a/rbutil/rbutilqt/base/utils.cpp +++ b/rbutil/rbutilqt/base/utils.cpp @@ -90,11 +90,12 @@ bool Utils::recursiveRmdir( const QString &dirName ) //! @return returns exact casing of path, empty string if path not found. QString Utils::resolvePathCase(QString path) { - QStringList elems; - QString realpath; - - elems = path.split("/", QString::SkipEmptyParts); int start; + QString realpath; + QStringList elems = path.split("/", QString::SkipEmptyParts); + + if(path.isEmpty()) + return QString(); #if defined(Q_OS_WIN32) // on windows we must make sure to start with the first entry (i.e. the // drive letter) instead of a single / to make resolving work.