]> arthur.barton.de Git - netatalk.git/commitdiff
Merge 2-1
authorFrank Lahm <franklahm@googlemail.com>
Wed, 20 Apr 2011 10:58:53 +0000 (12:58 +0200)
committerFrank Lahm <franklahm@googlemail.com>
Wed, 20 Apr 2011 10:58:53 +0000 (12:58 +0200)
1  2 
etc/cnid_dbd/cmd_dbd.c
etc/cnid_dbd/cmd_dbd.h
etc/cnid_dbd/cmd_dbd_scanvol.c
etc/cnid_dbd/main.c

diff --combined etc/cnid_dbd/cmd_dbd.c
index ed67a9a54bc14aecf0d38dd3e0a9c8610714a431,ccc7eada645aa11416e1beceadc349bd775277f3..2359defde738e2bb9770483c840f27cfe841b1fd
@@@ -81,8 -81,8 +81,9 @@@
  #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 */
- volatile sig_atomic_t alarmed;
 +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 ? */
  
  static DBD *dbd;
  static int verbose;             /* Logging flag */
@@@ -91,8 -91,6 +92,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 */
      -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
@@@ -166,34 -164,59 +167,59 @@@ static void set_signal(void
      }        
  }
  
- static int get_lock(const char *dbpath)
+ /*!
+  * 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)
  {
-     int lockfd;
+     static int lockfd = -1;
      char lockpath[PATH_MAX];
      struct flock lock;
      struct stat st;
  
-     if ( (strlen(dbpath) + strlen(LOCKFILENAME+1)) > (PATH_MAX - 1) ) {
-         dbd_log( LOGSTD, ".AppleDB pathname too long");
-         exit(EXIT_FAILURE);
-     }
-     strncpy(lockpath, dbpath, PATH_MAX - 1);
-     strcat(lockpath, "/");
-     strcat(lockpath, LOCKFILENAME);
+     if (cmd == 0) {
+         if (lockfd == -1)
+             return -1;
  
-     if ((lockfd = open(lockpath, O_RDWR | O_CREAT, 0644)) < 0) {
-         dbd_log( LOGSTD, "Error opening lockfile: %s", strerror(errno));
-         exit(EXIT_FAILURE);
+         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 ((stat(dbpath, &st)) != 0) {
-         dbd_log( LOGSTD, "Error statting lockfile: %s", strerror(errno));
-         exit(EXIT_FAILURE);
-     }
+     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 ((chown(lockpath, st.st_uid, st.st_gid)) != 0) {
-         dbd_log( LOGSTD, "Error inheriting lockfile permissions: %s", strerror(errno));
-         exit(EXIT_FAILURE);
+         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;
      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", strerror(errno));        
-                 exit(EXIT_FAILURE);
+                 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));
-             exit(EXIT_FAILURE);
+             return -1;
         }
      }
-     
-     return lockfd;
- }
- static void free_lock(int lockfd)
- {
-     struct flock lock;
  
-     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);
+     dbd_log(LOGDEBUG, "get_lock: got lock");    
+     return 1;
  }
  
  static void usage (void)
@@@ -281,6 -295,7 +298,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) {
          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 */
          close(dbdirfd);
      }
  
-     /* 
-        Before we do anything else, check if there is an instance of cnid_dbd
-        running already and silently exit if yes.
-     */
-     lockfd = get_lock(dbpath);
+     /* Get db lock, which exits if exclusive was requested and it already is locked */
+     if ((db_locked = get_lock(1, dbpath)) == -1)
+         goto exit_failure;
  
      /* Prepare upgrade ? */
      if (prep_upgrade) {
 -        if (dbif_prep_upgrade(dbpath))
 +        if (dbif_env_remove(dbpath))
              goto exit_failure;
          goto exit_success;
      }        
      /* 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];
-         close(lockfd);
++
+         if ((db_locked = get_lock(0, 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.");
-         lockfd = get_lock(dbpath);
+         if ((db_locked = get_lock(1, dbpath)) == -1)
+             goto exit_failure;
      }
  
      /* 
          if ((dbd = dbif_init(dbpath, "cnid2.db")) == NULL)
              goto exit_failure;
          
-         if (dbif_env_open(dbd, &db_param, exclusive ? (DBOPTIONS | DB_RECOVER) : DBOPTIONS) < 0) {
+         if (dbif_env_open(dbd,
+                           &db_param,
+                           exclusive ? (DBOPTIONS | DB_RECOVER) : DBOPTIONS) < 0) {
              dbd_log( LOGSTD, "error opening database!");
              goto exit_failure;
          }
              dbif_close(dbd);
              goto exit_failure;
          }
 -
 -        if (dbd_stamp(dbd) < 0) {
 -            dbif_close(dbd);
 -            goto exit_failure;
 -        }
      }
  
      /* Now execute given command scan|rebuild|dump */
@@@ -494,7 -499,7 +515,7 @@@ exit_success
      ret = 0;
  
  exit_failure:
-     free_lock(lockfd);
+     get_lock(0, NULL);
      
      if ((fchdir(cdir)) < 0)
          dbd_log(LOGSTD, "fchdir: %s", strerror(errno));
diff --combined etc/cnid_dbd/cmd_dbd.h
index 425fb400cd3f4d1e13a1b671b249278c1eba0c0a,0a4c2a8446eeb15b3db3cb26d3bf712f72d69a31..7caf634f5f1ac093a0f818b6bb2063923f417cb0
@@@ -25,10 -25,14 +25,12 @@@ typedef unsigned int dbd_flags_t
          (strcmp(a,c) b 0)
  
  extern int nocniddb; /* Dont open CNID database, only scan filesystem */
+ extern int db_locked; /* have we got the fcntl lock on lockfd ? */
  extern volatile sig_atomic_t alarmed;
 -extern struct volinfo *volinfo;
 -extern char cwdbuf[MAXPATHLEN+1];
  
  extern void dbd_log(enum logtype lt, char *fmt, ...);
  extern int cmd_dbd_scanvol(DBD *dbd, struct volinfo *volinfo, dbd_flags_t flags);
+ extern int get_lock(int cmd, const char *dbpath);
  
  /*
    Functions for querying the database which couldn't be reused from the existing
index 49938af36e8a7e6e04d9b6700b92a718fcb64c71,2ec332690a2c01d800830bf87458c88236f99e8a..588da0db777591d0e25a4000d03b1d1b1ffb1ef2
  #define ADDIR_OK (addir_ok == 0)
  #define ADFILE_OK (adfile_ok == 0)
  
 -/* These must be accessible for cmd_dbd_* funcs */
 -struct volinfo        *volinfo;
 -char                  cwdbuf[MAXPATHLEN+1];
  
 -/* Some static vars */
 +static struct volinfo *myvolinfo;
 +static char           cwdbuf[MAXPATHLEN+1];
  static DBD            *dbd;
  static DBD            *dbd_rebuild;
  static dbd_flags_t    dbd_flags;
@@@ -83,22 -85,22 +83,22 @@@ static char *utompath(char *upath
      u = upath;
      outlen = strlen(upath);
  
 -    if ((volinfo->v_casefold & AFPVOL_UTOMUPPER))
 +    if ((myvolinfo->v_casefold & AFPVOL_UTOMUPPER))
          flags |= CONV_TOUPPER;
 -    else if ((volinfo->v_casefold & AFPVOL_UTOMLOWER))
 +    else if ((myvolinfo->v_casefold & AFPVOL_UTOMLOWER))
          flags |= CONV_TOLOWER;
  
 -    if ((volinfo->v_flags & AFPVOL_EILSEQ)) {
 +    if ((myvolinfo->v_flags & AFPVOL_EILSEQ)) {
          flags |= CONV__EILSEQ;
      }
  
      /* convert charsets */
 -    if ((size_t)-1 == ( outlen = convert_charset(volinfo->v_volcharset,
 +    if ((size_t)-1 == ( outlen = convert_charset(myvolinfo->v_volcharset,
                                                   CH_UTF8_MAC,
 -                                                 volinfo->v_maccharset,
 +                                                 myvolinfo->v_maccharset,
                                                   u, outlen, mpath, MAXPATHLEN, &flags)) ) {
          dbd_log( LOGSTD, "Conversion from %s to %s for %s failed.",
 -                 volinfo->v_volcodepage, volinfo->v_maccodepage, u);
 +                 myvolinfo->v_volcodepage, myvolinfo->v_maccodepage, u);
          return NULL;
      }
  
@@@ -124,17 -126,17 +124,17 @@@ static char *mtoupath(char *mpath
      }
  
      /* set conversion flags */
 -    if (!(volinfo->v_flags & AFPVOL_NOHEX))
 +    if (!(myvolinfo->v_flags & AFPVOL_NOHEX))
          flags |= CONV_ESCAPEHEX;
 -    if (!(volinfo->v_flags & AFPVOL_USEDOTS))
 +    if (!(myvolinfo->v_flags & AFPVOL_USEDOTS))
          flags |= CONV_ESCAPEDOTS;
  
 -    if ((volinfo->v_casefold & AFPVOL_MTOUUPPER))
 +    if ((myvolinfo->v_casefold & AFPVOL_MTOUUPPER))
          flags |= CONV_TOUPPER;
 -    else if ((volinfo->v_casefold & AFPVOL_MTOULOWER))
 +    else if ((myvolinfo->v_casefold & AFPVOL_MTOULOWER))
          flags |= CONV_TOLOWER;
  
 -    if ((volinfo->v_flags & AFPVOL_EILSEQ)) {
 +    if ((myvolinfo->v_flags & AFPVOL_EILSEQ)) {
          flags |= CONV__EILSEQ;
      }
  
      outlen = MAXPATHLEN;
  
      if ((size_t)-1 == (outlen = convert_charset(CH_UTF8_MAC,
 -                                                volinfo->v_volcharset,
 -                                                volinfo->v_maccharset,
 +                                                myvolinfo->v_volcharset,
 +                                                myvolinfo->v_maccharset,
                                                  m, inplen, u, outlen, &flags)) ) {
          dbd_log( LOGSTD, "conversion from UTF8-MAC to %s for %s failed.",
 -                 volinfo->v_volcodepage, mpath);
 +                 myvolinfo->v_volcodepage, mpath);
          return NULL;
      }
  
@@@ -222,8 -224,8 +222,8 @@@ static int check_symlink(const char *na
        and can compare it with the currents volume path
      */
      int i = 0;
 -    while (volinfo->v_path[i]) {
 -        if ((pathbuf[i] == 0) || (volinfo->v_path[i] != pathbuf[i])) {
 +    while (myvolinfo->v_path[i]) {
 +        if ((pathbuf[i] == 0) || (myvolinfo->v_path[i] != pathbuf[i])) {
              dbd_log( LOGDEBUG, "extra-share symlink '%s/%s', following", cwdbuf, name);
              return 1;
          }
@@@ -304,7 -306,7 +304,7 @@@ static int check_adfile(const char *fna
      else
          adflags = ADFLAGS_DIR;
  
 -    adname = volinfo->ad_path(fname, adflags);
 +    adname = myvolinfo->ad_path(fname, adflags);
  
      if ((ret = access( adname, F_OK)) != 0) {
          if (errno != ENOENT) {
              return -1;
  
          /* Create ad file */
 -        ad_init(&ad, volinfo->v_adouble, volinfo->v_ad_options);
 +        ad_init(&ad, myvolinfo->v_adouble, myvolinfo->v_ad_options);
  
          if ((ret = ad_open_metadata( fname, adflags, O_CREAT, &ad)) != 0) {
              dbd_log( LOGSTD, "Error creating AppleDouble file '%s/%s': %s",
          chmod(adname, st->st_mode);
  #endif
      } else {
 -        ad_init(&ad, volinfo->v_adouble, volinfo->v_ad_options);
 +        ad_init(&ad, myvolinfo->v_adouble, myvolinfo->v_ad_options);
          if (ad_open_metadata( fname, adflags, O_RDONLY, &ad) != 0) {
              dbd_log( LOGSTD, "Error opening AppleDouble file for '%s/%s'", cwdbuf, fname);
              return -1;
@@@ -470,10 -472,10 +470,10 @@@ static int check_addir(int volroot
      }
  
      /* Check for ".Parent" */
 -    if ( (adpar_ok = access(volinfo->ad_path(".", ADFLAGS_DIR), F_OK)) != 0) {
 +    if ( (adpar_ok = access(myvolinfo->ad_path(".", ADFLAGS_DIR), F_OK)) != 0) {
          if (errno != ENOENT) {
              dbd_log(LOGSTD, "Access error on '%s/%s': %s",
 -                    cwdbuf, volinfo->ad_path(".", ADFLAGS_DIR), strerror(errno));
 +                    cwdbuf, myvolinfo->ad_path(".", ADFLAGS_DIR), strerror(errno));
              return -1;
          }
          dbd_log(LOGSTD, "Missing .AppleDouble/.Parent for '%s'", cwdbuf);
          }
  
          /* Create ad dir and set name */
 -        ad_init(&ad, volinfo->v_adouble, volinfo->v_ad_options);
 +        ad_init(&ad, myvolinfo->v_adouble, myvolinfo->v_ad_options);
  
          if (ad_open_metadata( ".", ADFLAGS_DIR, O_CREAT, &ad) != 0) {
              dbd_log( LOGSTD, "Error creating AppleDouble dir in %s: %s", cwdbuf, strerror(errno));
              return -1;
          }
          chown(ADv2_DIRNAME, st.st_uid, st.st_gid);
 -        chown(volinfo->ad_path(".", ADFLAGS_DIR), st.st_uid, st.st_gid);
 +        chown(myvolinfo->ad_path(".", ADFLAGS_DIR), st.st_uid, st.st_gid);
      }
  
      return 0;
@@@ -532,7 -534,7 +532,7 @@@ static int check_eafile_in_adouble(cons
      char *namep, *namedup = NULL;
  
      /* Check if this is an AFPVOL_EA_AD vol */
 -    if (volinfo->v_vfs_ea == AFPVOL_EA_AD) {
 +    if (myvolinfo->v_vfs_ea == AFPVOL_EA_AD) {
          /* Does the filename contain "::EA" ? */
          namedup = strdup(name);
          if ((namep = strstr(namedup, "::EA")) == NULL) {
@@@ -666,8 -668,6 +666,8 @@@ static int read_addir(void
  /*
    Check CNID for a file/dir, both from db and from ad-file.
    For detailed specs see intro.
 +
 +  @return Correct CNID of object or CNID_INVALID (ie 0) on error
  */
  static cnid_t check_cnid(const char *name, cnid_t did, struct stat *st, int adfile_ok, int adflags)
  {
          cnidcount = 0;
          if (dbif_txn_checkpoint(dbd, 0, 0, 0) < 0) {
              dbd_log(LOGSTD, "Error checkpointing!");
 -            return 0;
 +            return CNID_INVALID;
          }
      }
  
      /* Get CNID from ad-file if volume is using AFPVOL_CACHE */
      ad_cnid = 0;
 -    if ( (volinfo->v_flags & AFPVOL_CACHE) && ADFILE_OK) {
 -        ad_init(&ad, volinfo->v_adouble, volinfo->v_ad_options);
 +    if ( (myvolinfo->v_flags & AFPVOL_CACHE) && ADFILE_OK) {
 +        ad_init(&ad, myvolinfo->v_adouble, myvolinfo->v_ad_options);
          if (ad_open_metadata( name, adflags, O_RDWR, &ad) != 0) {
              
              if (dbd_flags & DBD_FLAGS_CLEANUP)
 -                return 0;
 +                return CNID_INVALID;
  
              dbd_log( LOGSTD, "Error opening AppleDouble file for '%s/%s': %s", cwdbuf, name, strerror(errno));
 -            return 0;
 +            return CNID_INVALID;
          }
  
          if (dbd_flags & DBD_FLAGS_FORCE) {
      memset(&rply, 0, sizeof(struct cnid_dbd_rply));
      rqst.did = did;
      rqst.cnid = ad_cnid;
 -    if ( ! (volinfo->v_flags & AFPVOL_NODEV))
 +    if ( ! (myvolinfo->v_flags & AFPVOL_NODEV))
          rqst.dev = st->st_dev;
      rqst.ino = st->st_ino;
      rqst.type = S_ISDIR(st->st_mode)?1:0;
  
      /* Query the database */
      ret = dbd_lookup(dbd, &rqst, &rply, (dbd_flags & DBD_FLAGS_SCAN) ? 1 : 0);
 -    dbif_txn_close(dbd, ret);
 +    if (dbif_txn_close(dbd, ret) != 0)
 +        return CNID_INVALID;
      if (rply.result == CNID_DBD_RES_OK) {
          db_cnid = rply.cnid;
      } else if (rply.result == CNID_DBD_RES_NOTFOUND) {
          if ( ! (dbd_flags & DBD_FLAGS_SCAN)) {
              rqst.cnid = db_cnid;
              ret = dbd_delete(dbd, &rqst, &rply, DBIF_CNID);
 -            dbif_txn_close(dbd, ret);
 +            if (dbif_txn_close(dbd, ret) != 0)
 +                return CNID_INVALID;
  
              rqst.cnid = ad_cnid;
              ret = dbd_delete(dbd, &rqst, &rply, DBIF_CNID);
 -            dbif_txn_close(dbd, ret);
 +            if (dbif_txn_close(dbd, ret) != 0)
 +                return CNID_INVALID;
  
              ret = dbd_rebuild_add(dbd, &rqst, &rply);
 -            dbif_txn_close(dbd, ret);
 +            if (dbif_txn_close(dbd, ret) != 0)
 +                return CNID_INVALID;
          }
          return ad_cnid;
      } else if (ad_cnid && (db_cnid == 0)) {
              if (ret == CNID_DBD_RES_OK) {
                  /* Occupied! Choose another, update ad-file */
                  ret = dbd_add(dbd, &rqst, &rply, 1);
 -                dbif_txn_close(dbd, ret);
 +                if (dbif_txn_close(dbd, ret) != 0)
 +                    return CNID_INVALID;
                  db_cnid = rply.cnid;
                  dbd_log(LOGSTD, "New CNID for '%s/%s': %u", cwdbuf, name, ntohl(db_cnid));
  
 -                if ((volinfo->v_flags & AFPVOL_CACHE)
 +                if ((myvolinfo->v_flags & AFPVOL_CACHE)
                      && ADFILE_OK
                      && ( ! (dbd_flags & DBD_FLAGS_SCAN))) {
                      dbd_log(LOGSTD, "Writing CNID data for '%s/%s' to AppleDouble file",
                              cwdbuf, name, ntohl(db_cnid));
 -                    ad_init(&ad, volinfo->v_adouble, volinfo->v_ad_options);
 +                    ad_init(&ad, myvolinfo->v_adouble, myvolinfo->v_ad_options);
                      if (ad_open_metadata( name, adflags, O_RDWR, &ad) != 0) {
                          dbd_log(LOGSTD, "Error opening AppleDouble file for '%s/%s': %s",
                                  cwdbuf, name, strerror(errno));
 -                        return 0;
 +                        return CNID_INVALID;
                      }
                      ad_setid( &ad, st->st_dev, st->st_ino, db_cnid, did, stamp);
                      ad_flush(&ad);
                      cwdbuf, name, ntohl(ad_cnid));
              rqst.cnid = ad_cnid;
              ret = dbd_rebuild_add(dbd, &rqst, &rply);
 -            dbif_txn_close(dbd, ret);
 +            if (dbif_txn_close(dbd, ret) != 0)
 +                return CNID_INVALID;
          }
          return ad_cnid;
      } else if ((db_cnid == 0) && (ad_cnid == 0)) {
          if ( ! (dbd_flags & DBD_FLAGS_SCAN)) {
              /* add to db */
              ret = dbd_add(dbd, &rqst, &rply, 1);
 -            dbif_txn_close(dbd, ret);
 +            if (dbif_txn_close(dbd, ret) != 0)
 +                return CNID_INVALID;
              db_cnid = rply.cnid;
              dbd_log(LOGSTD, "New CNID for '%s/%s': %u", cwdbuf, name, ntohl(db_cnid));
          }
  
      if ((ad_cnid == 0) && db_cnid) {
          /* in db but zeroID in ad-file, write it to ad-file if AFPVOL_CACHE */
 -        if ((volinfo->v_flags & AFPVOL_CACHE) && ADFILE_OK) {
 +        if ((myvolinfo->v_flags & AFPVOL_CACHE) && ADFILE_OK) {
              if ( ! (dbd_flags & DBD_FLAGS_SCAN)) {
                  dbd_log(LOGSTD, "Writing CNID data for '%s/%s' to AppleDouble file",
                          cwdbuf, name, ntohl(db_cnid));
 -                ad_init(&ad, volinfo->v_adouble, volinfo->v_ad_options);
 +                ad_init(&ad, myvolinfo->v_adouble, myvolinfo->v_ad_options);
                  if (ad_open_metadata( name, adflags, O_RDWR, &ad) != 0) {
                      dbd_log(LOGSTD, "Error opening AppleDouble file for '%s/%s': %s",
                              cwdbuf, name, strerror(errno));
 -                    return 0;
 +                    return CNID_INVALID;
                  }
                  ad_setid( &ad, st->st_dev, st->st_ino, db_cnid, did, stamp);
                  ad_flush(&ad);
          return db_cnid;
      }
  
 -    return 0;
 +    return CNID_INVALID;
  }
  
  /*
@@@ -862,6 -855,11 +862,11 @@@ static int dbd_readdir(int volroot, cni
      struct dirent *ep;
      static struct stat st;      /* Save some stack space */
  
+     /* keep trying to get the lock */
+     if (!db_locked)
+         if ((db_locked = get_lock(1, NULL)) == -1)
+             return -1;
      /* Check again for .AppleDouble folder, check_adfile also checks/creates it */
      if ((addir_ok = check_addir(volroot)) != 0)
          if ( ! (dbd_flags & DBD_FLAGS_SCAN))
                  static uint count = 0;
                  rqst.cnid = rply.cnid;
                  ret = dbd_rebuild_add(dbd_rebuild, &rqst, &rply);
 -                dbif_txn_close(dbd_rebuild, ret);
 +                if (dbif_txn_close(dbd_rebuild, ret) != 0)
 +                    return -1;
                  if (rply.result != CNID_DBD_RES_OK) {
 -                    dbd_log( LOGDEBUG, "Fatal error adding CNID: %u for '%s/%s' to in-memory rebuild-db",
 +                    dbd_log( LOGSTD, "Fatal error adding CNID: %u for '%s/%s' to in-memory rebuild-db",
                               cnid, cwdbuf, ep->d_name);
 -                    longjmp(jmp, 1); /* this jumps back to cmd_dbd_scanvol() */
 +                    return -1;
                  }
                  count++;
                  if (count == 10000) {
          }
  
          /* Check EA files */
 -        if (volinfo->v_vfs_ea == AFPVOL_EA_AD)
 +        if (myvolinfo->v_vfs_ea == AFPVOL_EA_AD)
              check_eafiles(ep->d_name);
  
          /**************************************************************************
@@@ -1048,22 -1045,22 +1053,22 @@@ static int scanvol(struct volinfo *vi, 
      }
  
      /* Make this stuff accessible from all funcs easily */
 -    volinfo = vi;
 +    myvolinfo = vi;
      dbd_flags = flags;
  
      /* Init a fake struct vol with just enough so we can call ea_open and friends */
      volume.v_adouble = AD_VERSION2;
 -    volume.v_vfs_ea = volinfo->v_vfs_ea;
 +    volume.v_vfs_ea = myvolinfo->v_vfs_ea;
      initvol_vfs(&volume);
  
      /* Run with umask 0 */
      umask(0);
  
      /* Remove trailing slash from volume, chdir to vol */
 -    if (volinfo->v_path[strlen(volinfo->v_path) - 1] == '/')
 -        volinfo->v_path[strlen(volinfo->v_path) - 1] = 0;
 -    strcpy(cwdbuf, volinfo->v_path);
 -    chdir(volinfo->v_path);
 +    if (myvolinfo->v_path[strlen(myvolinfo->v_path) - 1] == '/')
 +        myvolinfo->v_path[strlen(myvolinfo->v_path) - 1] = 0;
 +    strcpy(cwdbuf, myvolinfo->v_path);
 +    chdir(myvolinfo->v_path);
  
      /* Start recursion */
      if (dbd_readdir(1, htonl(2)) < 0)  /* 2 = volumeroot CNID */
@@@ -1118,14 -1115,8 +1123,14 @@@ static void delete_orphaned_cnids(DBD *
                      dbd_log(LOGSTD, "Orphaned CNID in database: %u", dbd_cnid);
                      if ( ! (dbd_flags & DBD_FLAGS_SCAN)) {
                          rqst.cnid = htonl(dbd_cnid);
 -                        ret = dbd_delete(dbd, &rqst, &rply, DBIF_CNID);
 -                        dbif_txn_close(dbd, ret);
 +                        if ((ret = dbd_delete(dbd, &rqst, &rply, DBIF_CNID)) == -1) {
 +                            dbd_log(LOGSTD, "Error deleting CNID %u", dbd_cnid);
 +                            (void)dbif_txn_abort(dbd);
 +                            goto cleanup;
 +                        }
 +                        
 +                        if (dbif_txn_close(dbd, ret) != 0)
 +                            return;
                          deleted++;
                      }
                      /* Check if we got a termination signal */
  
          if (dbd_cnid < rebuild_cnid) {
              /* CNID is orphaned -> delete */
 -            dbd_log(LOGSTD, "Orphaned CNID in database: %u.", dbd_cnid);
 +            dbd_log(LOGSTD, "One orphaned CNID in database: %u.", dbd_cnid);
              if ( ! (dbd_flags & DBD_FLAGS_SCAN)) {
                  rqst.cnid = htonl(dbd_cnid);
 -                ret = dbd_delete(dbd, &rqst, &rply, DBIF_CNID);
 -                dbif_txn_close(dbd, ret);
 +                if ((ret = dbd_delete(dbd, &rqst, &rply, DBIF_CNID)) == -1) {
 +                    dbd_log(LOGSTD, "Error deleting CNID %u", dbd_cnid);
 +                    (void)dbif_txn_abort(dbd);
 +                    goto cleanup;
 +                }
 +                if (dbif_txn_close(dbd, ret) != 0)
 +                    return;
                  deleted++;
              }
              continue;
          if (dbd_cnid > rebuild_cnid) {
              dbif_idwalk(dbd, NULL, 1); /* Close cursor */
              dbif_idwalk(dbd_rebuild, NULL, 1); /* Close cursor */
 -            dbif_txn_close(dbd, 2);
 -            dbif_txn_close(dbd_rebuild, 2);
 +            (void)dbif_txn_close(dbd, 2);
 +            (void)dbif_txn_close(dbd_rebuild, 2);                
              dbd_log(LOGSTD, "Ghost CNID: %u. This is fatal! Dumping rebuild db:\n", rebuild_cnid);
              dbif_dump(dbd_rebuild, 0);
              dbd_log(LOGSTD, "Send this dump and a `dbd -d ...` dump to the Netatalk Dev team!");
              goto cleanup;
          }
 -    }
 +    } /* while ((dbif_idwalk(dbd, &dbd_cnid, 0)) == 1) */
  
  cleanup:
      dbif_idwalk(dbd, NULL, 1); /* Close cursor */
@@@ -1186,7 -1172,7 +1191,7 @@@ static const char *get_tmpdb_path(void
  /*
    Main func called from cmd_dbd.c
  */
 -int cmd_dbd_scanvol(DBD *dbd_ref, struct volinfo *volinfo, dbd_flags_t flags)
 +int cmd_dbd_scanvol(DBD *dbd_ref, struct volinfo *vi, dbd_flags_t flags)
  {
      int ret = 0;
      struct db_param db_param = { 0 };
  
      /* Set cachesize for in-memory rebuild db */
      db_param.cachesize = 64 * 1024;         /* 64 MB */
 +    db_param.maxlocks = DEFAULT_MAXLOCKS;
 +    db_param.maxlockobjs = DEFAULT_MAXLOCKOBJS;
      db_param.logfile_autoremove = 1;
  
      /* Make it accessible for all funcs */
      dbd = dbd_ref;
  
      /* We only support unicode volumes ! */
 -    if ( volinfo->v_volcharset != CH_UTF8) {
 -        dbd_log( LOGSTD, "Not a Unicode volume: %s, %u != %u", volinfo->v_volcodepage, volinfo->v_volcharset, CH_UTF8);
 +    if ( vi->v_volcharset != CH_UTF8) {
 +        dbd_log( LOGSTD, "Not a Unicode volume: %s, %u != %u", vi->v_volcodepage, vi->v_volcharset, CH_UTF8);
          return -1;
      }
  
      }
  
      /* scanvol */
 -    if ( (scanvol(volinfo, flags)) != 0) {
 +    if ( (scanvol(vi, flags)) != 0) {
          ret = -1;
          goto exit;
      }
  
  exit:
      if (! nocniddb) {
 -        dbif_txn_close(dbd, 1);
 +        if (dbif_txn_close(dbd, 2) != 0)
 +            ret = -1;
          if (dbd_rebuild)
 -            dbif_txn_close(dbd_rebuild, 1);
 -        if ((flags & DBD_FLAGS_EXCL) && !(flags & DBD_FLAGS_FORCE))
 +            if (dbif_txn_close(dbd_rebuild, 2) != 0)
 +                ret = -1;
 +        if ((ret == 0) && dbd_rebuild && (flags & DBD_FLAGS_EXCL) && !(flags & DBD_FLAGS_FORCE))
              /* We can only do this in exclusive mode, otherwise we might delete CNIDs added from
                 other clients in between our pass 1 and 2 */
              delete_orphaned_cnids(dbd, dbd_rebuild, flags);
diff --combined etc/cnid_dbd/main.c
index da4cd9a4e7085d764590140786580a3e82a3bb9d,5cafb2ef12f9a0de4e70098892fd61ee073fddf6..9b9d963134c4a02b5feee5bc0f248ef2e689c1bc
@@@ -1,4 -1,6 +1,4 @@@
  /*
 - * $Id: main.c,v 1.16 2009-11-25 14:59:15 franklahm Exp $
 - *
   * Copyright (C) Joerg Lenneis 2003
   * Copyright (c) Frank Lahm 2009
   * All Rights Reserved.  See COPYING.
@@@ -32,7 -34,6 +32,7 @@@
  #include <netatalk/endian.h>
  #include <atalk/cnid_dbd_private.h>
  #include <atalk/logger.h>
 +#include <atalk/volinfo.h>
  
  #include "db_param.h"
  #include "dbif.h"
     Note: DB_INIT_LOCK is here so we can run the db_* utilities while netatalk is running.
     It's a likey performance hit, but it might we worth it.
   */
- #define DBOPTIONS (DB_CREATE | DB_INIT_LOG | DB_INIT_MPOOL | DB_INIT_LOCK | DB_INIT_TXN | DB_RECOVER)
+ #define DBOPTIONS (DB_CREATE | DB_INIT_LOG | DB_INIT_MPOOL | DB_INIT_LOCK | DB_INIT_TXN)
  
 +/* Global, needed by pack.c:idxname() */
 +struct volinfo volinfo;
 +
  static DBD *dbd;
  static int exit_sig = 0;
+ static int db_locked;
  
  static void sig_exit(int signo)
  {
@@@ -73,6 -72,57 +74,57 @@@ static void block_sigs_onoff(int block
      return;
  }
  
+ /*!
+  * Get lock on db lock file
+  *
+  * @args cmd       (r) !=0: lock, 0: unlock
+  * @returns            1 if lock was acquired, 0 if file is already locked, -1 on error
+  */
+ static int get_lock(int cmd)
+ {
+     static int lockfd = -1;
+     struct flock lock;
+     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 ((lockfd = open(LOCKFILENAME, O_RDWR | O_CREAT, 0644)) < 0) {
+             LOG(log_error, logtype_cnid, "get_lock: error opening lockfile: %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) {
+             LOG(log_debug, logtype_cnid, "get_lock: couldn't lock");
+             return 0;
+         } else {
+             LOG(log_error, logtype_cnid, "get_lock: fcntl F_WRLCK lockfile: %s", strerror(errno));
+             return -1;
+         }
+     }
+     LOG(log_debug, logtype_cnid, "get_lock: got lock");
+     return 1;
+ }
  /*
    The dbd_XXX and comm_XXX functions all obey the same protocol for return values:
  
@@@ -119,6 -169,20 +171,20 @@@ static int loop(struct db_param *dbp
          dbp->flush_interval, timebuf);
  
      while (1) {
+         /*
+          * If we haven't got the lock yet, get it now.
+          * Prevents a race with dbd:
+          *   1. no cnid_dbd running
+          *   2. dbd -r starts, gets the lock
+          *   3. cnid_dbd starts, doesn't get lock, doesn't run recovery, all is fine
+          *   4. dbd from (2) finishes, drops lock
+          *   5. anothet dbd but this time with -re is started which
+          *      - succeeds getting the lock
+          *      - runs recovery => this kills (3)
+          */
+         if (!db_locked)
+             if ((db_locked = get_lock(1)) == -1)
+                 return -1;
          timeout = min(time_next_flush, time_last_rqst +dbp->idle_timeout);
          if (timeout > now)
              timeout -= now;
              case CNID_DBD_OP_REBUILD_ADD:
                  ret = dbd_rebuild_add(dbd, &rqst, &rply);
                  break;
 +            case CNID_DBD_OP_SEARCH:
 +                ret = dbd_search(dbd, &rqst, &rply);
 +                break;
              default:
                  LOG(log_error, logtype_cnid, "loop: unknown op %d", rqst.op);
                  ret = -1;
                  break;
              }
 -            
 +
              if ((cret = comm_snd(&rply)) < 0 || ret < 0) {
                  dbif_txn_abort(dbd);
                  return -1;
@@@ -256,34 -317,6 +322,6 @@@ static void switch_to_user(char *dir
      }
  }
  
- /* ------------------------ */
- static int get_lock(void)
- {
-     int lockfd;
-     struct flock lock;
-     if ((lockfd = open(LOCKFILENAME, O_RDWR | O_CREAT, 0644)) < 0) {
-         LOG(log_error, logtype_cnid, "main: error opening lockfile: %s", strerror(errno));
-         exit(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) {
-             LOG(log_error, logtype_cnid, "get_lock: locked");
-             exit(0);
-         } else {
-             LOG(log_error, logtype_cnid, "main: fcntl F_WRLCK lockfile: %s", strerror(errno));
-             exit(1);
-         }
-     }
-     return lockfd;
- }
  
  /* ----------------------- */
  static void set_signal(void)
      }
  }
  
- /* ----------------------- */
- static void free_lock(int lockfd)
- {
-     struct flock lock;
-     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);
- }
  /* ------------------------ */
  int main(int argc, char *argv[])
  {
      struct db_param *dbp;
      int err = 0;
-     int lockfd, ctrlfd, clntfd;
+     int ctrlfd, clntfd;
 -    char *dir, *logconfig;
 +    char *logconfig;
  
      set_processname("cnid_dbd");
  
          exit(1);
      }
  
 -    dir = argv[1];
      ctrlfd = atoi(argv[2]);
      clntfd = atoi(argv[3]);
      logconfig = strdup(argv[4]);
      setuplog(logconfig);
  
 -    switch_to_user(dir);
 +    /* Load .volinfo file */
 +    if (loadvolinfo(argv[1], &volinfo) == -1) {
 +        LOG(log_error, logtype_cnid, "Cant load volinfo for \"%s\"", argv[1]);
 +        exit(EXIT_FAILURE);
 +    }
 +    /* Put "/.AppleDB" at end of volpath, get path from volinfo file */
 +    char dbpath[MAXPATHLEN+1];
 +    if ((strlen(volinfo.v_dbpath) + strlen("/.AppleDB")) > MAXPATHLEN ) {
 +        LOG(log_error, logtype_cnid, "CNID db pathname too long: \"%s\"", volinfo.v_dbpath);
 +        exit(EXIT_FAILURE);
 +    }
 +    strncpy(dbpath, volinfo.v_dbpath, MAXPATHLEN - strlen("/.AppleDB"));
 +    strcat(dbpath, "/.AppleDB");
 +
 +    if (vol_load_charsets(&volinfo) == -1) {
 +        LOG(log_error, logtype_cnid, "Error loading charsets!");
 +        exit(EXIT_FAILURE);
 +    }
 +    LOG(log_debug, logtype_cnid, "db dir: \"%s\"", dbpath);
 +
 +    switch_to_user(dbpath);
  
      /* Before we do anything else, check if there is an instance of cnid_dbd
         running already and silently exit if yes. */
-     lockfd = get_lock();
+     if ((db_locked = get_lock(1)) == -1) {
+         exit(1);
+     }
  
 -    LOG(log_info, logtype_cnid, "Startup, DB dir %s", dir);
 -
      set_signal();
  
      /* SIGINT and SIGTERM are always off, unless we are in pselect */
      block_sigs_onoff(1);
  
 -    if ((dbp = db_param_read(dir, CNID_DBD)) == NULL)
 +    if ((dbp = db_param_read(dbpath)) == NULL)
          exit(1);
      LOG(log_maxdebug, logtype_cnid, "Finished parsing db_param config file");
  
 -    if (NULL == (dbd = dbif_init(".", "cnid2.db")))
 +    if (NULL == (dbd = dbif_init(dbpath, "cnid2.db")))
          exit(2);
  
-     if (dbif_env_open(dbd, dbp, DBOPTIONS) < 0)
+     /* Only recover if we got the lock */
+     if (dbif_env_open(dbd,
+                       dbp,
+                       db_locked ? DBOPTIONS | DB_RECOVER : DBOPTIONS) < 0)
          exit(2); /* FIXME: same exit code as failure for dbif_open() */
      LOG(log_debug, logtype_cnid, "Finished initializing BerkeleyDB environment");
  
      }
      LOG(log_debug, logtype_cnid, "Finished opening BerkeleyDB databases");
  
 -    if (dbd_stamp(dbd) < 0) {
 -        dbif_close(dbd);
 -        exit(5);
 -    }
 -    LOG(log_maxdebug, logtype_cnid, "Finished checking database stamp");
 -
      if (comm_init(dbp, ctrlfd, clntfd) < 0) {
          dbif_close(dbd);
          exit(3);
      if (dbif_close(dbd) < 0)
          err++;
  
 -    if (dbif_prep_upgrade(dir) < 0)
 +    if (dbif_env_remove(dbpath) < 0)
          err++;
  
-     free_lock(lockfd);
+     (void)get_lock(0);
  
      if (err)
          exit(4);