Fixed various PlatSec violations when app had no AllFiles capability.

Fixed QtCore in various places that caused Platform Security
violations in Symbian if AllFiles capability was missing from the
application. All of these these were caused by trying to access /private
folder unnecessarily, either by Qt code or Open C.

Task-number: 249008
Reviewed-by: Janne Koskinen

1 files changed, 24 insertions, 7 deletions

diff --git a/src/corelib/kernel/qcoreapplication.cpp b/src/corelib/kernel/qcoreapplication.cpp
index 423bdf8..25098cc 100644
--- a/src/corelib/kernel/qcoreapplication.cpp
+++ b/src/corelib/kernel/qcoreapplication.cpp
@@ -1806,18 +1806,35 @@ QString QCoreApplication::applicationDirPath()
         RProcess proc;
         TInt err = proc.Open(proc.Id());
         if (err == KErrNone) {
+            QChar driveChar;
 #if defined(Q_CC_NOKIAX86)
             // In emulator, always resolve the private dir on C-drive
-            appPath.append(QChar('C'));
+            driveChar = QLatin1Char('C');
 #else
-            appPath.append(QChar((proc.FileName())[0]));
+            driveChar = QLatin1Char((proc.FileName())[0]);
 #endif
-            appPath.append(QLatin1String(":\\private\\"));
-            QString sid;
-            sid.setNum(proc.SecureId().iId, 16);
-            appPath.append(sid);
-            appPath.append(QLatin1Char('\\'));
             proc.Close();
+
+            driveChar = driveChar.toUpper();
+
+            TFileName privatePath;
+            RFs& fs = qt_s60GetRFs();
+            fs.PrivatePath(privatePath);
+            appPath = qt_TDesC2QString(privatePath);
+            appPath.prepend(QLatin1Char(':')).prepend(driveChar);
+
+            // Create the appPath if it doesn't exist. Non-existing appPath will cause
+            // Platform Security violations later on if the app doesn't have AllFiles capability.
+            // Can't create appPath for ROM unfortunately, so applications meant for
+            // ROM should always deploy something to their private dir to ensure appPath exists,
+            // if the PlatSec violations are an issue.
+            char driveDiff = QLatin1Char('Z').toLatin1() - driveChar.toLatin1();
+            TInt driveId = EDriveZ - static_cast<TInt>(driveDiff);
+            if (driveId != EDriveZ) {
+                TInt err = fs.CreatePrivatePath(driveId);
+                if (err != KErrNone)
+                    qWarning("QCoreApplication::applicationDirPath: Failed to create private path.");
+            }
         }
 
         QFileInfo fi(appPath);