Fix broken random name generation for temporary directories.

Task-number: QTIFW-354

The function was correctly calculating the initial part of the
name using QTemporaryFile, though the appended "meta" was not
checked and could result in duplicated names later on.

Change-Id: I00eeebbb01fbfcc5a626e4bdfe04013f8e520ed5
Reviewed-by: Niels Weber <niels.weber@digia.com>
Reviewed-by: Tim Jenssen <tim.jenssen@digia.com>

1 files changed, 12 insertions, 8 deletions

diff --git a/src/libs/installer/fileutils.cpp b/src/libs/installer/fileutils.cpp
index 1fd5bf35c..6f8c44622 100644
--- a/src/libs/installer/fileutils.cpp
+++ b/src/libs/installer/fileutils.cpp
@@ -453,14 +453,18 @@ QString QInstaller::generateTemporaryFileName(const QString &templ)
     return f.fileName();
 }
 
-QString QInstaller::createTemporaryDirectory(const QString &templ)
-{
-    const QString t = QDir::tempPath() + QLatin1String("/") + templ + QLatin1String("XXXXXX");
-    QTemporaryFile f(t);
-    if (!f.open())
-        throw Error(QObject::tr("Could not create temporary folder for template %1: %2").arg(t, f.errorString()));
-    const QString path = f.fileName() + QLatin1String("meta");
-    qDebug() << "\nCreating meta data directory at" << path;
+QString QInstaller::createTemporaryDirectory(const QString &templateName)
+{
+    QString path = QDir::tempPath() + QLatin1String("/") + templateName + QLatin1String("XXXXXX");
+    {
+        QTemporaryFile f(path);
+        if (!f.open()) {
+            throw Error(QObject::tr("Could not create temporary directory %1: %2").arg(f.fileName(),
+                f.errorString()));
+        }
+        path = f.fileName();
+    }
+    qDebug() << "\nCreating temporary directory at:" << path;
 
     QInstaller::mkpath(path);
     return path;