Check if file is writable before making a copy for writing

1 files changed, 8 insertions, 0 deletions

diff --git a/src/sources/metadatasourcetaglib.cpp b/src/sources/metadatasourcetaglib.cpp
index 607e9881cf..b1660e6441 100644
--- a/src/sources/metadatasourcetaglib.cpp
+++ b/src/sources/metadatasourcetaglib.cpp
@@ -630,6 +630,14 @@ class SafelyWritableFile final {
         // See also: https://bugs.launchpad.net/mixxx/+bug/1815305
         DEBUG_ASSERT(m_origFileName.isNull());
         DEBUG_ASSERT(m_tempFileName.isNull());
+        if (!QFileInfo(origFileName).isWritable()) {
+            kLogger.warning()
+                    << "Failed to prepare file for writing:"
+                    << origFileName
+                    << "is not writable.";
+            // Abort constructor
+            return;
+        }
         if (useTemporaryFile) {
             QString tempFileName = origFileName + kSafelyWritableTempFileSuffix;
             QFile origFile(origFileName);