From 39df36c2472bc271b3bc294d83b43d9d203e1298 Mon Sep 17 00:00:00 2001
From: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
Date: Mon, 9 Sep 2019 18:06:29 +0200
Subject: [PATCH] Use return type to prevent warning and determine chown
 success. Added TODO

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>
---
 src/libsync/propagatedownload.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/libsync/propagatedownload.cpp b/src/libsync/propagatedownload.cpp
index da2d88b82..a0d1bd6bc 100644
--- a/src/libsync/propagatedownload.cpp
+++ b/src/libsync/propagatedownload.cpp
@@ -798,7 +798,10 @@ namespace { // Anonymous namespace for the recall feature
     static void preserveGroupOwnership(const QString &fileName, const QFileInfo &fi)
     {
 #ifdef Q_OS_UNIX
-        chown(fileName.toLocal8Bit().constData(), -1, fi.groupId());
+        int chownErr = chown(fileName.toLocal8Bit().constData(), -1, fi.groupId());
+        if (chownErr) {
+            // TODO: Error handling!
+        }
 #else
         Q_UNUSED(fileName);
         Q_UNUSED(fi);