From c28ecbb7ed8404b7e29b1d1b30882c865d87e34d Mon Sep 17 00:00:00 2001 From: Oswald Buddenhagen Date: Thu, 5 Sep 2013 18:30:05 +0200 Subject: [PATCH] make setting a bad $$[QMAKEFEATURES] not crash qmake Task-number: QTBUG-29642 Change-Id: I9cc209eb313f03bf342bcb64b1de3005755700a7 (cherry picked from qtbase/50a90720be7d67dcdfe704a82fb3b1138edd2aab) Reviewed-by: Joerg Bornemann Reviewed-by: Daniel Teske --- src/linguist/shared/qmakeevaluator.cpp | 4 ++-- src/linguist/shared/qmakeglobals.cpp | 24 ++++++++++++++---------- src/linguist/shared/qmakeglobals.h | 1 + 3 files changed, 17 insertions(+), 12 deletions(-) diff --git a/src/linguist/shared/qmakeevaluator.cpp b/src/linguist/shared/qmakeevaluator.cpp index 0162f6e..e5715fa 100644 --- a/src/linguist/shared/qmakeevaluator.cpp +++ b/src/linguist/shared/qmakeevaluator.cpp @@ -1478,8 +1478,8 @@ void QMakeEvaluator::updateFeaturePaths() feature_roots += m_qmakefeatures; - feature_roots += m_option->propertyValue(ProKey("QMAKEFEATURES")).toQString(m_mtmp).split( - m_option->dirlist_sep, QString::SkipEmptyParts); + feature_roots += m_option->splitPathList( + m_option->propertyValue(ProKey("QMAKEFEATURES")).toQString(m_mtmp)); QStringList feature_bases; if (!m_buildRoot.isEmpty()) { diff --git a/src/linguist/shared/qmakeglobals.cpp b/src/linguist/shared/qmakeglobals.cpp index 6f2390f..a00eb1f 100644 --- a/src/linguist/shared/qmakeglobals.cpp +++ b/src/linguist/shared/qmakeglobals.cpp @@ -270,6 +270,19 @@ QString QMakeGlobals::shadowedPath(const QString &fileName) const return QString(); } +QStringList QMakeGlobals::splitPathList(const QString &val) const +{ + QStringList ret; + if (!val.isEmpty()) { + QDir bdir; + QStringList vals = val.split(dirlist_sep); + ret.reserve(vals.length()); + foreach (const QString &it, vals) + ret << QDir::cleanPath(bdir.absoluteFilePath(it)); + } + return ret; +} + QString QMakeGlobals::getEnv(const QString &var) const { #ifdef PROEVALUATOR_SETENV @@ -281,16 +294,7 @@ QString QMakeGlobals::getEnv(const QString &var) const QStringList QMakeGlobals::getPathListEnv(const QString &var) const { - QStringList ret; - QString val = getEnv(var); - if (!val.isEmpty()) { - QDir bdir; - QStringList vals = val.split(dirlist_sep); - ret.reserve(vals.length()); - foreach (const QString &it, vals) - ret << QDir::cleanPath(bdir.absoluteFilePath(it)); - } - return ret; + return splitPathList(getEnv(var)); } QString QMakeGlobals::expandEnvVars(const QString &str) const diff --git a/src/linguist/shared/qmakeglobals.h b/src/linguist/shared/qmakeglobals.h index 25d3697..8fcffab 100644 --- a/src/linguist/shared/qmakeglobals.h +++ b/src/linguist/shared/qmakeglobals.h @@ -148,6 +148,7 @@ public: QString expandEnvVars(const QString &str) const; QString shadowedPath(const QString &fileName) const; + QStringList splitPathList(const QString &value) const; private: QString getEnv(const QString &) const; -- 2.7.4