]> arthur.barton.de Git - netatalk.git/blobdiff - etc/cnid_dbd/cmd_dbd.c
Merge sf/master
[netatalk.git] / etc / cnid_dbd / cmd_dbd.c
index ccc7eada645aa11416e1beceadc349bd775277f3..5493a28059461cb525f49f99ecaa0cbf16dbb15d 100644 (file)
 #include <atalk/logger.h>
 #include <atalk/cnid_dbd_private.h>
 #include <atalk/volinfo.h>
+#include <atalk/util.h>
+
 #include "cmd_dbd.h"
 #include "dbd.h"
 #include "dbif.h"
 #include "db_param.h"
 
-#define LOCKFILENAME  "lock"
 #define DBOPTIONS (DB_CREATE | DB_INIT_LOCK | DB_INIT_LOG | DB_INIT_MPOOL | DB_INIT_TXN)
 
 int nocniddb = 0;               /* Dont open CNID database, only scan filesystem */
+struct volinfo volinfo; /* needed by pack.c:idxname() */
 volatile sig_atomic_t alarmed;  /* flags for signals */
 int db_locked;                  /* have we got the fcntl lock on lockfile ? */
 
@@ -91,6 +93,8 @@ static struct db_param db_param = {
     NULL,                       /* Volume dirpath */
     1,                          /* bdb logfile autoremove */
     64 * 1024,                  /* bdb cachesize (64 MB) */
+    DEFAULT_MAXLOCKS,           /* maxlocks */
+    DEFAULT_MAXLOCKOBJS,        /* maxlockobjs */
     0,                          /* flush_interval */
     0,                          /* flush_frequency */
     0,                          /* usock_file */
@@ -98,7 +102,7 @@ static struct db_param db_param = {
     -1,                         /* idle_timeout */
     -1                          /* max_vols */
 };
-static char dbpath[PATH_MAX];   /* Path to the dbd database */
+static char dbpath[MAXPATHLEN+1];   /* Path to the dbd database */
 
 /* 
    Provide some logging
@@ -164,84 +168,6 @@ static void set_signal(void)
     }        
 }
 
-
-/*!
- * Get lock on db lock file
- *
- * @args cmd       (r) !=0: lock, 0: unlock
- * @args dbpath    (r) path to lockfile, only used on first call,
- *                     later the stored fd is used
- * @returns            1 if lock was acquired, 0 if file is already locked, -1 on error
- */
-int get_lock(int cmd, const char *dbpath)
-{
-    static int lockfd = -1;
-    char lockpath[PATH_MAX];
-    struct flock lock;
-    struct stat st;
-
-    if (cmd == 0) {
-        if (lockfd == -1)
-            return -1;
-
-        lock.l_start  = 0;
-        lock.l_whence = SEEK_SET;
-        lock.l_len    = 0;
-        lock.l_type = F_UNLCK;
-        fcntl(lockfd, F_SETLK, &lock);
-        close(lockfd);
-        lockfd = -1;
-        return 0;
-    }
-
-    if (lockfd == -1) {
-        if ( (strlen(dbpath) + strlen(LOCKFILENAME+1)) > (PATH_MAX - 1) ) {
-            dbd_log( LOGSTD, ".AppleDB pathname too long");
-            return -1;
-        }
-        strncpy(lockpath, dbpath, PATH_MAX - 1);
-        strcat(lockpath, "/");
-        strcat(lockpath, LOCKFILENAME);
-
-        if ((lockfd = open(lockpath, O_RDWR | O_CREAT, 0644)) < 0) {
-            dbd_log( LOGSTD, "Error opening lockfile: %s", strerror(errno));
-            return -1;
-        }
-
-        if ((stat(dbpath, &st)) != 0) {
-            dbd_log( LOGSTD, "Error statting lockfile: %s", strerror(errno));
-            return -1;
-        }
-
-        if ((chown(lockpath, st.st_uid, st.st_gid)) != 0) {
-            dbd_log( LOGSTD, "Error inheriting lockfile permissions: %s", strerror(errno));
-            return -1;
-        }
-    }
-    
-    lock.l_start  = 0;
-    lock.l_whence = SEEK_SET;
-    lock.l_len    = 0;
-    lock.l_type   = F_WRLCK;
-
-    if (fcntl(lockfd, F_SETLK, &lock) < 0) {
-        if (errno == EACCES || errno == EAGAIN) {
-            if (exclusive) {
-                dbd_log(LOGSTD, "Database is in use and exlusive was requested");
-                return -1;
-            };
-            dbd_log(LOGDEBUG, "get_lock: couldn't lock");
-            return 0;
-        } else {
-            dbd_log( LOGSTD, "Error getting fcntl F_WRLCK on lockfile: %s", strerror(errno));
-            return -1;
-       }
-    }
-
-    dbd_log(LOGDEBUG, "get_lock: got lock");    
-    return 1;
-}
-
 static void usage (void)
 {
     printf("Usage: dbd [-e|-t|-v|-x] -d [-i] | -s [-c|-n]| -r [-c|-f] | -u <path to netatalk volume>\n"
@@ -295,7 +221,6 @@ int main(int argc, char **argv)
     int dump=0, scan=0, rebuild=0, prep_upgrade=0, rebuildindexes=0, dumpindexes=0, force=0;
     dbd_flags_t flags = 0;
     char *volpath;
-    struct volinfo volinfo;
     int cdir;
 
     if (geteuid() != 0) {
@@ -399,12 +324,25 @@ int main(int argc, char **argv)
         exit(EXIT_FAILURE);        
     }
 
+    /* Enuser dbpath is there, create if necessary */
+    struct stat st;
+    if (stat(volinfo.v_dbpath, &st) != 0) {
+        if (errno != ENOENT) {
+            dbd_log( LOGSTD, "Can't stat dbpath \"%s\": %s", volinfo.v_dbpath, strerror(errno));
+            exit(EXIT_FAILURE);        
+        }
+        if ((mkdir(volinfo.v_dbpath, 0755)) != 0) {
+            dbd_log( LOGSTD, "Can't create dbpath \"%s\": %s", dbpath, strerror(errno));
+            exit(EXIT_FAILURE);
+        }        
+    }
+
     /* Put "/.AppleDB" at end of volpath, get path from volinfo file */
-    if ( (strlen(volinfo.v_dbpath) + strlen("/.AppleDB")) > (PATH_MAX - 1) ) {
+    if ( (strlen(volinfo.v_dbpath) + strlen("/.AppleDB")) > MAXPATHLEN ) {
         dbd_log( LOGSTD, "Volume pathname too long");
         exit(EXIT_FAILURE);        
     }
-    strncpy(dbpath, volinfo.v_dbpath, PATH_MAX - 9 - 1);
+    strncpy(dbpath, volinfo.v_dbpath, MAXPATHLEN - strlen("/.AppleDB"));
     strcat(dbpath, "/.AppleDB");
 
     /* Check or create dbpath */
@@ -423,13 +361,22 @@ int main(int argc, char **argv)
         close(dbdirfd);
     }
 
-    /* Get db lock, which exits if exclusive was requested and it already is locked */
-    if ((db_locked = get_lock(1, dbpath)) == -1)
+    /* Get db lock */
+    if ((db_locked = get_lock(LOCK_EXCL, dbpath)) == -1)
         goto exit_failure;
+    if (db_locked != LOCK_EXCL) {
+        /* Couldn't get exclusive lock, try shared lock if -e wasn't requested */
+        if (exclusive) {
+            dbd_log(LOGSTD, "Database is in use and exlusive was requested");
+            goto exit_failure;
+        }
+        if ((db_locked = get_lock(LOCK_SHRD, NULL)) != LOCK_SHRD)
+            goto exit_failure;
+    }
 
     /* Prepare upgrade ? */
     if (prep_upgrade) {
-        if (dbif_prep_upgrade(dbpath))
+        if (dbif_env_remove(dbpath))
             goto exit_failure;
         goto exit_success;
     }        
@@ -437,13 +384,18 @@ int main(int argc, char **argv)
     /* Check if -f is requested and wipe db if yes */
     if ((flags & DBD_FLAGS_FORCE) && rebuild && (volinfo.v_flags & AFPVOL_CACHE)) {
         char cmd[8 + MAXPATHLEN];
-        if ((db_locked = get_lock(0, NULL)) != 0)
+        if ((db_locked = get_lock(LOCK_FREE, NULL)) != 0)
             goto exit_failure;
-        snprintf(cmd, 8 + MAXPATHLEN, "rm -f %s/*", dbpath);
+
+        snprintf(cmd, 8 + MAXPATHLEN, "rm -rf \"%s\"", dbpath);
         dbd_log( LOGDEBUG, "Removing old database of volume: '%s'", volpath);
         system(cmd);
+        if ((mkdir(dbpath, 0755)) != 0) {
+            dbd_log( LOGSTD, "Can't create dbpath \"%s\": %s", dbpath, strerror(errno));
+            exit(EXIT_FAILURE);
+        }
         dbd_log( LOGDEBUG, "Removed old database.");
-        if ((db_locked = get_lock(1, dbpath)) == -1)
+        if ((db_locked = get_lock(LOCK_EXCL, dbpath)) == -1)
             goto exit_failure;
     }
 
@@ -456,23 +408,26 @@ int main(int argc, char **argv)
         
         if (dbif_env_open(dbd,
                           &db_param,
-                          exclusive ? (DBOPTIONS | DB_RECOVER) : DBOPTIONS) < 0) {
+                          (db_locked == LOCK_EXCL) ? (DBOPTIONS | DB_RECOVER) : DBOPTIONS) < 0) {
             dbd_log( LOGSTD, "error opening database!");
             goto exit_failure;
         }
 
-        if (exclusive)
+        if (db_locked == LOCK_EXCL)
             dbd_log( LOGDEBUG, "Finished recovery.");
 
         if (dbif_open(dbd, NULL, rebuildindexes) < 0) {
             dbif_close(dbd);
             goto exit_failure;
         }
+    }
 
-        if (dbd_stamp(dbd) < 0) {
-            dbif_close(dbd);
+    /* Downgrade db lock if not running exclusive */
+    if (!exclusive && (db_locked == LOCK_EXCL)) {
+        if (get_lock(LOCK_UNLOCK, NULL) != 0)
+            goto exit_failure;
+        if (get_lock(LOCK_SHRD, NULL) != LOCK_SHRD)
             goto exit_failure;
-        }
     }
 
     /* Now execute given command scan|rebuild|dump */