Added smb.conf file

Reviewed-on: #2

configs/ncsambawatcher.service

@@ -1,6 +1,7 @@
 [Unit]
 Description=A service which scan Nextcloud folders
-After=network.target
+After=network.target docker.service
+Requires=docker.service
 
 [Service]
 ExecStart=/usr/bin/ncsambawatcher

configs/smb.24.04.conf

@@ -0,0 +1,18 @@
+[global]
+
+   vfs objects = full_audit
+   full_audit:prefix = %u|%I|%m|%S
+   full_audit:success = mkdirat unlinkat renameat write
+   full_audit:failure = none
+   full_audit:facility = local5
+   full_audit:priority = NOTICE
+
+# Put this line only for the groupfolder's share
+[Some gorupfolder share]
+
+   full_audit:prefix = %u|%I|%m|__groupfolders/<group-folders-id>
+
+# To disable logs for a specific share
+[A share]
+
+    vfs objects = 

src/guarder.h

@@ -4,26 +4,37 @@
 #include <sys/types.h>
 #include <sys/shm.h>
 #include <sys/ipc.h>
+#include <sys/sem.h>
+#include <unistd.h>
 
 class guarder{
 private:
     const int shmid;
+    const int semid;
+
+    void semaphoreOp(int op) {
+        struct sembuf sb{};
+        sb.sem_num = 0;
+        sb.sem_op = op;
+        sb.sem_flg = 0;
+        semop(semid, &sb, 1);
+    }
 
     void setFlag(bool value)
     {
+        semaphoreOp(-1);
         bool *flag = static_cast<bool *>(shmat(shmid, nullptr, 0));
         *flag = value;
         shmdt(flag);
+        semaphoreOp(1);
     }
 
 public:
-    guarder() : shmid(shmget(IPC_PRIVATE, sizeof(bool), 0666 | IPC_CREAT))
+    guarder() : shmid(shmget(IPC_PRIVATE, sizeof(bool), 0666 | IPC_CREAT)), semid(semget(IPC_PRIVATE, 1, 0666 | IPC_CREAT))
     {
         setFlagOff();
     }
 
-    guarder(guarder &g) : shmid(g.shmid) {}
-
     void setFlagOff()
     {
         setFlag(false);
@@ -36,15 +47,18 @@ public:
 
     bool isFlagOn()
     {
+        semaphoreOp(-1);
         bool *flag = static_cast<bool *>(shmat(shmid, nullptr, 0));
         bool ret = *flag;
         shmdt(flag);
+        semaphoreOp(1);
         return ret;
     }
 
     ~guarder()
     {
         shmctl(shmid, IPC_RMID, nullptr);
+        semctl(semid, 0, IPC_RMID);
     }
 
 };

src/main.cpp

@@ -1,63 +1,25 @@
+#include <stdio.h>
+#include <unistd.h>
+#include <sys/wait.h>
 #include <iostream>
+#include <thread>
+#include <vector>
+#include <set>
 #include <array>
 #include <string>
-#include <vector>
+#include <mutex>
-#include <unistd.h>
+#include <condition_variable>
-#include <sys/types.h>
+#include <cstdio>
-#include <syslog.h>
-#include <fcntl.h>
-#include <signal.h>
-#include "locations.h"
 #include "usermanager.h"
-#include "guarder.h"
-#include "processManager.h"
 
-#define MAXNAMESIZE 255
-#define SCAN_DONE_SIG SIGRTMIN
 #define SCAN_CMD_USR "docker exec --user www-data nextcloud /var/www/html/occ files:scan --path="
 
-int p1[2];
-guarder guard;
 userManager manager;
+std::condition_variable cv;
+std::mutex mtx;
 
-void flushManagerToPipe()
+void readingThreadFunc()
 {
-    if (!guard.isFlagOn())
-    {
-        std::vector<std::string> users = manager.getFlaggedUsers();
-
-        for (std::vector<std::string>::iterator it = users.begin(); it != users.end(); ++it)
-        {
-            int size = it->size();
-            write(p1[1], &size, sizeof(int));
-            write(p1[1], it->data(), size * sizeof(char));
-        }
-
-        manager.unflagAllUsers();
-    }
-}
-
-void handler(int sig) 
-{
-    if (sig == SCAN_DONE_SIG)
-    {
-        flushManagerToPipe();
-    }
-}
-
-int main()
-{
-    openlog("ncsambawatcher", LOG_PID | LOG_CONS, LOG_USER);
-    pipe(p1);
-
-    pid_t parent = getpid();
-    pid_t child = fork();
-
-    if (child > 0) // parent
-    {
-        signal(SCAN_DONE_SIG, handler);
-        close(p1[0]); // read
-
     FILE *logpipe = popen(LOGFILE, "r");
     std::array<char, 256> buffer;
 
@@ -70,51 +32,71 @@ int main()
 
         std::vector<std::string> x = splitLogFile(line, '|');
         std::string user(x.at(USER_LOG_LOCATION));
+
+        {
+            std::lock_guard<std::mutex> lock(mtx);
             manager.addUser(user);
             manager.setUserFlagged(user);
+        }
+
+        cv.notify_one();
 
         std::cout << "User find: " << user << std::endl;
-
-            flushManagerToPipe();
     }
-
+}
-        fclose(logpipe);
+
-        close(p1[1]); // write
+void scannerThreadFunc()
-    }
+{
-    else // child
+    std::unique_lock<std::mutex> lock(mtx);
-    {
+    std::vector<pid_t> childrens;
-        close(p1[1]); // write
+    while (true)
-
+    {
-        int size;
+        cv.wait(lock, []
-        char *buffer = nullptr;
+                { return manager.isAnybodyFlagged(); });
-        processManager pm;
+
-
+        std::set<std::string> scanUsers = manager.getFlaggedUsers();
-        while (read(p1[0], &size, sizeof(int)))
+        manager.unflagAllUsers();
-        {
+
-            if (buffer == nullptr)
+        lock.unlock();
-                buffer = new char[size];
+        childrens.clear();
-
+
-            read(p1[0], buffer, size * sizeof(char));
+        for (const std::string& user : scanUsers)
-            std::string name(buffer);
+        {
-            std::string cmd = std::string(SCAN_CMD_USR) + name;
+            pid_t child = fork();
-
+
-            std::cout << "Scan received for: " << name << std::endl;
+            if (child < 0)
-
+            {
-            pm.runTask(name, cmd);
+                std::cerr << "Fork failed for: " << user << std::endl;
-
+            }
-            kill(parent, SCAN_DONE_SIG);
+            else if (child == 0) // child
-
+            {
-            if (buffer != nullptr)
+                std::string cmd = (std::string(SCAN_CMD_USR) + user);
-            {
+                execl("/bin/sh", "sh", "-c", cmd.c_str(), static_cast<char *>(nullptr));
-                delete[] buffer;
+                std::cerr << "Scan failed" << std::endl;
-                buffer = nullptr;
+                _exit(EXIT_FAILURE);
-            }
+            }
-        }
+            else // parent
-
+            {
-        close(p1[0]); // read
+                childrens.push_back(child);
-    }
+            }
-
+        }
-    closelog();
+
-
+        for (const pid_t& pid : childrens)
-    return EXIT_SUCCESS;
+        {
+            waitpid(pid, nullptr, 0);
+        }
+
+        lock.lock();
+    }
+}
+
+int main()
+{
+    std::thread readingThread(readingThreadFunc);
+    std::thread scannerThread(scannerThreadFunc);
+
+    readingThread.join();
+    scannerThread.join();
+
+    return 0;
 }

src/processManager.h

@@ -1,41 +0,0 @@
-#ifndef _PROCCESSMANAGER_H
-#define _PROCCESSMANAGER_H
-
-#include <map>
-#include <string>
-#include <unistd.h>
-#include <sys/types.h>
-#include "guarder.h"
-#include "usermanager.h"
-
-class processManager
-{
-private:
-    std::map<std::string, guarder> running;
-
-    bool getIdTag(std::string &id)
-    {
-        return running[id].isFlagOn();
-    }
-
-
-public:  
-
-    void runTask(std::string &id, std::string &cmdCommand)
-    {
-        if (!getIdTag(id))
-        {
-            running[id].setFlagOn();
-            pid_t child = fork();
-
-            if (child == 0) // child
-            {
-                system(cmdCommand.c_str());
-                running[id].setFlagOff();
-                exit(EXIT_SUCCESS);
-            }
-        }
-    }
-};
-
-#endif // _PROCCESSMANAGER_H

src/usermanager.h

@@ -4,7 +4,9 @@
 #include <string>
 #include <map>
 #include <vector>
+#include <set>
 #include <sstream>
+#include <mutex>
 #include "locations.h"
 
 std::vector<std::string> splitLogFile(const std::string& input, char delimiter);
@@ -13,6 +15,7 @@ class userManager
 {
 private:
     std::map<std::string, bool> users;
+    std::mutex mtx;
 
 public:
     
@@ -23,6 +26,7 @@ public:
 
     void addUser(std::string &user)
     {
+        std::lock_guard<std::mutex> lock(mtx);
         if (users.count(user) == 0)
         {
             users[user] = false;
@@ -31,16 +35,19 @@ public:
 
     void removeUser(std::string &user)
     {
+        std::lock_guard<std::mutex> lock(mtx);
         users.erase(user);
     }
 
     bool isContains(std::string &user)
     {
+        std::lock_guard<std::mutex> lock(mtx);
         return users.count(user) == 1;
     }
 
     void setUserFlagged(std::string &user)
     {
+        std::lock_guard<std::mutex> lock(mtx);
         if (users.count(user) == 1)
         {
             users[user] = true;
@@ -49,6 +56,7 @@ public:
 
     void setUserUnflagged(std::string &user)
     {
+        std::lock_guard<std::mutex> lock(mtx);
         if (users.count(user) == 1)
         {
             users[user] = false;
@@ -57,38 +65,56 @@ public:
 
     void unflagAllUsers()
     {
+        std::lock_guard<std::mutex> lock(mtx);
         for (std::map<std::string, bool>::iterator it = users.begin(); it != users.end(); ++it)
         {
             it->second = false;
         }
     }
 
-    std::vector<std::string> getUsers()
+    std::set<std::string> getUsers()
     {
-        std::vector<std::string> ret;
+        std::set<std::string> ret;
+
+        std::lock_guard<std::mutex> lock(mtx);
 
         for (std::map<std::string, bool>::iterator it = users.begin(); it != users.end(); ++it)
         {
-            ret.push_back(it->first);
+            ret.insert(it->first);
         }
 
         return ret;
     }
 
-    std::vector<std::string> getFlaggedUsers()
+    std::set<std::string> getFlaggedUsers()
     {
-        std::vector<std::string> ret;
+        std::set<std::string> ret;
+
+        std::lock_guard<std::mutex> lock(mtx);
 
         for (std::map<std::string, bool>::iterator it = users.begin(); it != users.end(); ++it)
         {
             if (it->second)
             {
-                ret.push_back(it->first);
+                ret.insert(it->first);
             }
         }
 
         return ret;
     }
+
+    bool isAnybodyFlagged()
+    {
+        std::lock_guard<std::mutex> lock(mtx);
+    
+        for (std::map<std::string, bool>::iterator it = users.begin(); it != users.end(); ++it)
+        {
+            if (it->second)
+                return true;
+        }
+
+        return false;
+    }
 };
 
 #endif // _USERMAN_H