CTF: Use access to check if location is writable

Pick-to: 6.5
Change-Id: Ic6afcf74432f176374d47e828b480e4d2499c670
Reviewed-by: Antti Määttä <antti.maatta@qt.io>
Reviewed-by: Janne Koskinen <janne.p.koskinen@qt.io>
Reviewed-by: Hatem ElKharashy <hatem.elkharashy@qt.io>

1 files changed, 2 insertions, 5 deletions

diff --git a/src/plugins/tracing/qctflib.cpp b/src/plugins/tracing/qctflib.cpp
index 28016982dd..9311bb1fdd 100644
--- a/src/plugins/tracing/qctflib.cpp
+++ b/src/plugins/tracing/qctflib.cpp
@@ -62,16 +62,13 @@ QCtfLibImpl::QCtfLibImpl()
     }
 
     // Check if the location is writable
-    FILE *file = nullptr;
-    file = fopen(qPrintable(location + "/metadata"_L1), "w+b");
-    if (!file) {
+    if (QT_ACCESS(qPrintable(location), W_OK) != 0) {
         qCWarning (lcDebugTrace) << "Unable to write to location";
         return;
     }
-    fclose(file);
 
     const QString filename = location + QStringLiteral("/session.json");
-    file = fopen(qPrintable(filename), "rb");
+    FILE *file = fopen(qPrintable(filename), "rb");
     if (!file) {
         qCWarning (lcDebugTrace) << "unable to open session file: " << filename;
         m_location = location;