]> arthur.barton.de Git - netatalk.git/blobdiff - etc/afpd/directory.c
Symlink patch from Anton Starikov
[netatalk.git] / etc / afpd / directory.c
index 32c73ba1d590fd906e8e9e84cce8d4a2f9fd34cd..c50429ee9a6dcec8a0c0a3a58b8aa214c1e71089 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: directory.c,v 1.55 2003-01-08 15:01:33 didg Exp $
+ * $Id: directory.c,v 1.121.2.1 2010-01-02 10:22:32 franklahm Exp $
  *
  * Copyright (c) 1990,1993 Regents of The University of Michigan.
  * All Rights Reserved.  See COPYRIGHT.
 #include "config.h"
 #endif /* HAVE_CONFIG_H */
 
-#include <atalk/logger.h>
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <errno.h>
-#include <sys/time.h>
-#include <sys/param.h>
-#include <netatalk/endian.h>
-#include <atalk/adouble.h>
-#include <atalk/afp.h>
-#include <atalk/util.h>
-#ifdef CNID_DB
-#include <atalk/cnid.h>
-#endif /* CNID_DB */
-#include <utime.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <dirent.h>
-#ifdef HAVE_FCNTL_H
-#include <fcntl.h>
-#endif /* HAVE_FCNTL_H */
-#include <grp.h>
-#include <pwd.h>
-
 /* STDC check */
 #if STDC_HEADERS
 #include <string.h>
@@ -49,6 +26,26 @@ char *strchr (), *strrchr ();
 #define memmove(d,s,n) bcopy ((s), (d), (n))
 #endif /* ! HAVE_MEMCPY */
 #endif /* STDC_HEADERS */
+#ifdef HAVE_STRINGS_H
+#include <strings.h>
+#endif
+#include <stdio.h>
+#include <stdlib.h>
+
+#include <grp.h>
+#include <pwd.h>
+#include <sys/param.h>
+#include <errno.h>
+#include <utime.h>
+
+#include <atalk/adouble.h>
+#include <atalk/vfs.h>
+#include <atalk/afp.h>
+#include <atalk/util.h>
+#include <atalk/cnid.h>
+#include <atalk/logger.h>
+#include <atalk/uuid.h>
+#include <atalk/unix.h>
 
 #include "directory.h"
 #include "desktop.h"
@@ -58,17 +55,23 @@ char *strchr (), *strrchr ();
 #include "filedir.h"
 #include "globals.h"
 #include "unix.h"
+#include "mangle.h"
+#include "hash.h"
 
-struct dir     *curdir;
+#ifdef HAVE_NFSv4_ACLS
+extern void addir_inherit_acl(const struct vol *vol);
+#endif
+
+struct dir  *curdir;
 int             afp_errno;
 
 #define SENTINEL (&sentinel)
 static struct dir sentinel = { SENTINEL, SENTINEL, NULL, DIRTREE_COLOR_BLACK,
-                                 NULL, NULL, NULL, NULL, NULL, 0, 0, 
-                                 0, 0, NULL, NULL};
+                               NULL, NULL, NULL, NULL, NULL, 0, 0,
+                               0, 0, NULL, NULL, 0, NULL};
 static struct dir rootpar  = { SENTINEL, SENTINEL, NULL, 0,
-                                 NULL, NULL, NULL, NULL, NULL, 0, 0, 
-                                 0, 0, NULL, NULL};
+                               NULL, NULL, NULL, NULL, NULL, 0, 0,
+                               0, 0, NULL, NULL, 0, NULL};
 
 /* (from IM: Toolbox Essentials)
  * dirFinderInfo (DInfo) fields:
@@ -87,16 +90,28 @@ static struct dir rootpar  = { SENTINEL, SENTINEL, NULL, 0,
  * frPutAway:   4    home directory ID
  */
 
+static struct dir *
+vol_tree_root(const struct vol *vol, u_int32_t did)
+{
+    struct dir *dir;
+
+    if (vol->v_curdir && vol->v_curdir->d_did == did) {
+        dir = vol->v_curdir;
+    }
+    else {
+        dir = vol->v_root;
+    }
+    return dir;
+}
+
 /*
  * redid did assignment for directories. now we use red-black trees.
  * how exciting.
  */
 struct dir *
-            dirsearch( vol, did )
-            const struct vol   *vol;
-u_int32_t      did;
+dirsearch(const struct vol *vol, u_int32_t did)
 {
-    struct dir *dir;
+    struct dir  *dir;
 
 
     /* check for 0 did */
@@ -111,7 +126,8 @@ u_int32_t   did;
         return( &rootpar );
     }
 
-    dir = vol->v_root;
+    dir = vol_tree_root(vol, did);
+
     afp_errno = AFPERR_NOOBJ;
     while ( dir != SENTINEL ) {
         if (dir->d_did == did)
@@ -121,38 +137,73 @@ u_int32_t did;
     return NULL;
 }
 
+/* ------------------- */
+int get_afp_errno(const int param)
+{
+    if (afp_errno != AFPERR_DID1)
+        return afp_errno;
+    return param;
+}
+
+/* ------------------- */
+struct dir *
+dirsearch_byname( const struct vol *vol, struct dir *cdir, char *name)
+{
+    struct dir *dir = NULL;
+
+    if ((cdir->d_did != DIRDID_ROOT_PARENT) && (cdir->d_child)) {
+        struct dir key;
+        hnode_t *hn;
+
+        key.d_parent = cdir;
+        key.d_u_name = name;
+        key.d_u_name_len = strlen(name);
+        hn = hash_lookup(vol->v_hash, &key);
+        if (hn) {
+            dir = hnode_get(hn);
+        }
+    }
+    return dir;
+}
+
 /* -----------------------------------------
- * if did is not in the cache resolve it with cnid 
- * 
+ * if did is not in the cache resolve it with cnid
+ *
+ * FIXME
+ * OSX call it with bogus id, ie file ID not folder ID,
+ * and we are really bad in this case.
  */
 struct dir *
-            dirlookup( vol, did )
-            const struct vol   *vol;
-u_int32_t      did;
+dirlookup( struct vol *vol, u_int32_t did)
 {
-#ifdef CNID_DB
-    struct dir *ret;
-    char               *upath;
-    u_int32_t  id;
-    static char                path[MAXPATHLEN + 1];
-    int len;
-    int pathlen;
-    char *ptr;
-    static char buffer[12 + MAXPATHLEN + 1];
-    int buflen = 12 + MAXPATHLEN + 1;
-    char *mpath;
-    
+    struct dir   *ret;
+    char     *upath;
+    cnid_t       id, cnid;
+    static char  path[MAXPATHLEN + 1];
+    size_t len,  pathlen;
+    char         *ptr;
+    static char  buffer[12 + MAXPATHLEN + 1];
+    int          buflen = 12 + MAXPATHLEN + 1;
+    char         *mpath;
+    int          utf8;
+    size_t       maxpath;
+
     ret = dirsearch(vol, did);
-    if (ret != NULL)
+    if (ret != NULL || afp_errno == AFPERR_PARAM)
         return ret;
 
+    utf8 = utf8_encoding();
+    maxpath = (utf8)?MAXPATHLEN -7:255;
     id = did;
-    if ((upath = cnid_resolve(vol->v_db, &id, buffer, buflen)) == NULL) {
+    if (NULL == (upath = cnid_resolve(vol->v_cdb, &id, buffer, buflen)) ) {
         afp_errno = AFPERR_NOOBJ;
         return NULL;
     }
     ptr = path + MAXPATHLEN;
-    mpath = utompath(vol, upath);
+    if (NULL == ( mpath = utompath(vol, upath, did, utf8) ) ) {
+        afp_errno = AFPERR_NOOBJ;
+        return NULL;
+    }
     len = strlen(mpath);
     pathlen = len;          /* no 0 in the last part */
     len++;
@@ -163,37 +214,81 @@ u_int32_t did;
         if (ret != NULL) {
             break;
         }
-        if ((upath = cnid_resolve(vol->v_db, &id, buffer, buflen)) == NULL) {
+        cnid = id;
+        if ( NULL == (upath = cnid_resolve(vol->v_cdb, &id, buffer, buflen))
+             ||
+             NULL == (mpath = utompath(vol, upath, cnid, utf8))
+            ) {
             afp_errno = AFPERR_NOOBJ;
             return NULL;
         }
-        mpath = utompath(vol, upath);
+
         len = strlen(mpath) + 1;
         pathlen += len;
-        if (pathlen > 255) {
+        if (pathlen > maxpath) {
             afp_errno = AFPERR_PARAM;
             return NULL;
         }
         strcpy(ptr - len, mpath);
         ptr -= len;
     }
-    /* fill the cache */
-    ptr--;
-    *ptr = (unsigned char)pathlen;
-    ptr--;
-    *ptr = 2;
+
+    /* fill the cache, another place where we know about the path type */
+    if (utf8) {
+        u_int16_t temp16;
+        u_int32_t temp;
+
+        ptr -= 2;
+        temp16 = htons(pathlen);
+        memcpy(ptr, &temp16, sizeof(temp16));
+
+        temp = htonl(kTextEncodingUTF8);
+        ptr -= 4;
+        memcpy(ptr, &temp, sizeof(temp));
+        ptr--;
+        *ptr = 3;
+    }
+    else {
+        ptr--;
+        *ptr = (unsigned char)pathlen;
+        ptr--;
+        *ptr = 2;
+    }
     /* cname is not efficient */
     if (cname( vol, ret, &ptr ) == NULL )
         return NULL;
-#endif
+
     return dirsearch(vol, did);
 }
 
+/* child addition/removal */
+static void dirchildadd(const struct vol *vol, struct dir *a, struct dir *b)
+{
+    if (!a->d_child)
+        a->d_child = b;
+    else {
+        b->d_next = a->d_child;
+        b->d_prev = b->d_next->d_prev;
+        b->d_next->d_prev = b;
+        b->d_prev->d_next = b;
+    }
+    if (!hash_alloc_insert(vol->v_hash, b, b)) {
+        LOG(log_error, logtype_afpd, "dirchildadd: can't hash %s", b->d_u_name);
+    }
+}
+
+static void dirchildremove(struct dir *a,struct dir *b)
+{
+    if (a->d_child == b)
+        a->d_child = (b == b->d_next) ? NULL : b->d_next;
+    b->d_next->d_prev = b->d_prev;
+    b->d_prev->d_next = b->d_next;
+    b->d_next = b->d_prev = b;
+}
+
 /* --------------------------- */
 /* rotate the tree to the left */
-static void dir_leftrotate(vol, dir)
-struct vol *vol;
-struct dir *dir;
+static void dir_leftrotate(struct vol *vol, struct dir *dir)
 {
     struct dir *right = dir->d_right;
 
@@ -222,9 +317,7 @@ struct dir *dir;
 
 
 /* rotate the tree to the right */
-static void dir_rightrotate(vol, dir)
-struct vol *vol;
-struct dir *dir;
+static void dir_rightrotate(struct vol *vol, struct dir *dir)
 {
     struct dir *left = dir->d_left;
 
@@ -252,9 +345,7 @@ struct dir *dir;
 
 #if 0
 /* recolor after a removal */
-static struct dir *dir_rmrecolor(vol, dir)
-            struct vol *vol;
-struct dir *dir;
+static struct dir *dir_rmrecolor(struct vol *vol, struct dir *dir)
 {
     struct dir *leaf;
 
@@ -272,7 +363,7 @@ struct dir *dir;
 
             /* right leaf has black end nodes */
             if ((leaf->d_left->d_color == DIRTREE_COLOR_BLACK) &&
-                    (leaf->d_right->d_color = DIRTREE_COLOR_BLACK)) {
+                (leaf->d_right->d_color = DIRTREE_COLOR_BLACK)) {
                 leaf->d_color = DIRTREE_COLOR_RED; /* recolor leaf as red */
                 dir = dir->d_back; /* ascend */
             } else {
@@ -299,7 +390,7 @@ struct dir *dir;
 
             /* left leaf has black end nodes */
             if ((leaf->d_right->d_color == DIRTREE_COLOR_BLACK) &&
-                    (leaf->d_left->d_color = DIRTREE_COLOR_BLACK)) {
+                (leaf->d_left->d_color = DIRTREE_COLOR_BLACK)) {
                 leaf->d_color = DIRTREE_COLOR_RED; /* recolor leaf as red */
                 dir = dir->d_back; /* ascend */
             } else {
@@ -323,13 +414,25 @@ struct dir *dir;
 }
 #endif /* 0 */
 
+/* --------------------- */
+static void dir_hash_del(const struct vol *vol, struct dir *dir)
+{
+    hnode_t *hn;
+
+    hn = hash_lookup(vol->v_hash, dir);
+    if (!hn) {
+        LOG(log_error, logtype_afpd, "dir_hash_del: %s not hashed", dir->d_u_name);
+    }
+    else {
+        hash_delete(vol->v_hash, hn);
+    }
+}
 
 /* remove the node from the tree. this is just like insertion, but
  * different. actually, it has to worry about a bunch of things that
  * insertion doesn't care about. */
-static void dir_remove( vol, dir )
-struct vol     *vol;
-struct dir     *dir;
+
+static void dir_remove( struct vol *vol, struct dir *dir)
 {
 #ifdef REMOVE_NODES
     struct ofork *of, *last;
@@ -340,13 +443,13 @@ struct dir        *dir;
         return;
 
     /* i'm not sure if it really helps to delete stuff. */
-#ifndef REMOVE_NODES 
-    if (dir->d_u_name != dir->d_m_name) {
-        free(dir->d_u_name);
-    }
-    free(dir->d_m_name);
+    dir_hash_del(vol, dir);
+    vol->v_curdir = NULL;
+#ifndef REMOVE_NODES
+    dirfreename(dir);
     dir->d_m_name = NULL;
     dir->d_u_name = NULL;
+    dir->d_m_name_ucs2 = NULL;
 #else /* ! REMOVE_NODES */
 
     /* go searching for a node with at most one child */
@@ -372,10 +475,10 @@ struct dir        *dir;
     }
 
     /* we want to free node, but we also want to free the data in dir.
-    * currently, that's d_name and the directory traversal bits.
-    * we just copy the necessary bits and then fix up all the
-    * various pointers to the directory. needless to say, there are
-    * a bunch of places that store the directory struct. */
+     * currently, that's d_name and the directory traversal bits.
+     * we just copy the necessary bits and then fix up all the
+     * various pointers to the directory. needless to say, there are
+     * a bunch of places that store the directory struct. */
     if (node != dir) {
         struct dir save, *tmp;
 
@@ -393,7 +496,7 @@ struct dir  *dir;
             rootpar.d_child = vol->v_dir;
         } else {
             /* if we aren't the root directory, we have parents and
-            * siblings to worry about */
+             * siblings to worry about */
             if (dir->d_parent->d_child == node)
                 dir->d_parent->d_child = dir;
             dir->d_next->d_prev = dir;
@@ -422,11 +525,14 @@ struct dir        *dir;
         /* set the node's d_name */
         node->d_m_name = save.d_m_name;
         node->d_u_name = save.d_u_name;
+        node->d_m_name_ucs2 = save.d_m_name_ucs2;
     }
 
     if (node->d_color == DIRTREE_COLOR_BLACK)
         dir_rmrecolor(vol, leaf);
 
+    if (node->d_m_name_ucs2)
+        free(node->d_u_name_ucs2);
     if (node->d_u_name != node->d_m_name) {
         free(node->d_u_name);
     }
@@ -440,32 +546,29 @@ struct dir        *dir;
  *
  * FIXME what about opened forks with refs to it?
  * it's an afp specs violation because you can't delete
- * an opened forks. Now afpd doesn't care about forks opened by other 
+ * an opened forks. Now afpd doesn't care about forks opened by other
  * process. It's fixable within afpd if fnctl_lock, doable with smb and
  * next to impossible for nfs and local filesystem access.
  */
-static void dir_invalidate( vol, dir )
-const struct vol *vol;
-struct dir *dir;
+static void dir_invalidate( struct vol *vol, struct dir *dir)
 {
-       if (curdir == dir) {
-           /* v_root can't be deleted */
-               if (movecwd(vol, vol->v_root) < 0) 
-                       printf("Yuup cant change dir to v_root\n");
-       }
-       /* FIXME */
+    if (curdir == dir) {
+        /* v_root can't be deleted */
+        if (movecwd(vol, vol->v_root) < 0) {
+            LOG(log_error, logtype_afpd, "cname can't chdir to : %s", vol->v_root);
+        }
+    }
+    /* FIXME */
     dirchildremove(dir->d_parent, dir);
-       dir_remove( vol, dir );
+    dir_remove( vol, dir );
 }
 
 /* ------------------------------------ */
-static struct dir *dir_insert(vol, dir)
-            const struct vol *vol;
-struct dir *dir;
+static struct dir *dir_insert(const struct vol *vol, struct dir *dir)
 {
-    struct dir *pdir;
+    struct dir  *pdir;
 
-    pdir = vol->v_root;
+    pdir = vol_tree_root(vol, dir->d_did);
     while (pdir->d_did != dir->d_did ) {
         if ( pdir->d_did > dir->d_did ) {
             if ( pdir->d_left == SENTINEL ) {
@@ -486,32 +589,128 @@ struct dir *dir;
     return pdir;
 }
 
+#define ENUMVETO "./../Network Trash Folder/TheVolumeSettingsFolder/TheFindByContentFolder/:2eDS_Store/Contents/Desktop Folder/Trash/Benutzer/"
+
+int
+caseenumerate(const struct vol *vol, struct path *path, struct dir *dir)
+{
+    DIR               *dp;
+    struct dirent     *de;
+    int               ret;
+    static u_int32_t  did = 0;
+    static char       cname[MAXPATHLEN];
+    static char       lname[MAXPATHLEN];
+    ucs2_t        u2_path[MAXPATHLEN];
+    ucs2_t        u2_dename[MAXPATHLEN];
+    char          *tmp, *savepath;
+
+    if (!(vol->v_flags & AFPVOL_CASEINSEN))
+        return -1;
+
+    if (veto_file(ENUMVETO, path->u_name))
+        return -1;
+
+    savepath = path->u_name;
+
+    /* very simple cache */
+    if ( dir->d_did == did && strcmp(lname, path->u_name) == 0) {
+        path->u_name = cname;
+        path->d_dir = NULL;
+        if (of_stat( path ) == 0 ) {
+            return 0;
+        }
+        /* something changed, we cannot stat ... */
+        did = 0;
+    }
+
+    if (NULL == ( dp = opendir( "." )) ) {
+        LOG(log_debug, logtype_afpd, "caseenumerate: opendir failed: %s", dir->d_u_name);
+        return -1;
+    }
+
+
+    /* LOG(log_debug, logtype_afpd, "caseenumerate: for %s", path->u_name); */
+    if ((size_t) -1 == convert_string(vol->v_volcharset, CH_UCS2, path->u_name, -1, u2_path, sizeof(u2_path)) )
+        LOG(log_debug, logtype_afpd, "caseenumerate: conversion failed for %s", path->u_name);
+
+    /*LOG(log_debug, logtype_afpd, "caseenumerate: dir: %s, path: %s", dir->d_u_name, path->u_name); */
+    ret = -1;
+    for ( de = readdir( dp ); de != NULL; de = readdir( dp )) {
+        if (NULL == check_dirent(vol, de->d_name))
+            continue;
+
+        if ((size_t) -1 == convert_string(vol->v_volcharset, CH_UCS2, de->d_name, -1, u2_dename, sizeof(u2_dename)) )
+            continue;
+
+        if (strcasecmp_w( u2_path, u2_dename) == 0) {
+            tmp = path->u_name;
+            strlcpy(cname, de->d_name, sizeof(cname));
+            path->u_name = cname;
+            path->d_dir = NULL;
+            if (of_stat( path ) == 0 ) {
+                LOG(log_debug, logtype_afpd, "caseenumerate: using dir: %s, path: %s", de->d_name, path->u_name);
+                strlcpy(lname, tmp, sizeof(lname));
+                did = dir->d_did;
+                ret = 0;
+                break;
+            }
+            else
+                path->u_name = tmp;
+        }
+
+    }
+    closedir(dp);
+
+    if (ret) {
+        /* invalidate cache */
+        cname[0] = 0;
+        did = 0;
+        path->u_name = savepath;
+    }
+    /* LOG(log_debug, logtype_afpd, "caseenumerate: path on ret: %s", path->u_name); */
+    return ret;
+}
+
 
 /*
  * attempt to extend the current dir. tree to include path
  * as a side-effect, movecwd to that point and return the new dir
  */
 static struct dir *
-            extenddir( vol, dir, path )
-struct vol     *vol;
-struct dir     *dir;
-struct path *path;
+extenddir(struct vol *vol, struct dir *dir, struct path *path)
 {
-    char       *p;
+    path->d_dir = NULL;
 
-    path->u_name = p = mtoupath(vol, path->m_name );
-    if ( of_stat( path ) != 0 ) {
-        return( NULL );
+    if ( path->u_name == NULL) {
+        afp_errno = AFPERR_PARAM;
+        return NULL;
+    }
+
+    if (check_name(vol, path->u_name)) {
+        /* the name is illegal */
+        LOG(log_info, logtype_afpd, "extenddir: illegal path: '%s'", path->u_name);
+        path->u_name = NULL;
+        afp_errno = AFPERR_PARAM;
+        return NULL;
+    }
+
+    if (of_stat( path ) != 0 ) {
+        if (!(vol->v_flags & AFPVOL_CASEINSEN))
+            return NULL;
+        else if(caseenumerate(vol, path, dir) != 0)
+            return(NULL);
     }
 
     if (!S_ISDIR(path->st.st_mode)) {
         return( NULL );
     }
 
+    /* mac name is always with the right encoding (from cname()) */
     if (( dir = adddir( vol, dir, path)) == NULL ) {
         return( NULL );
     }
 
+    path->d_dir = dir;
     if ( movecwd( vol, dir ) < 0 ) {
         return( NULL );
     }
@@ -519,16 +718,44 @@ struct path *path;
     return( dir );
 }
 
+/* -------------------------
+   appledouble mkdir afp error code.
+*/
+static int netatalk_mkdir(const char *name)
+{
+    if (ad_mkdir(name, DIRBITS | 0777) < 0) {
+        switch ( errno ) {
+        case ENOENT :
+            return( AFPERR_NOOBJ );
+        case EROFS :
+            return( AFPERR_VLOCK );
+        case EPERM:
+        case EACCES :
+            return( AFPERR_ACCESS );
+        case EEXIST :
+            return( AFPERR_EXIST );
+        case ENOSPC :
+        case EDQUOT :
+            return( AFPERR_DFULL );
+        default :
+            return( AFPERR_PARAM );
+        }
+    }
+    return AFP_OK;
+}
+
+/* ------------------- */
 static int deletedir(char *dir)
 {
     char path[MAXPATHLEN + 1];
     DIR *dp;
-    struct dirent      *de;
+    struct dirent   *de;
     struct stat st;
     size_t len;
-    int err;
+    int err = AFP_OK;
+    size_t remain;
 
-    if ((len = strlen(dir)) > sizeof(path))
+    if ((len = strlen(dir)) +2 > sizeof(path))
         return AFPERR_PARAM;
 
     /* already gone */
@@ -538,118 +765,94 @@ static int deletedir(char *dir)
     strcpy(path, dir);
     strcat(path, "/");
     len++;
-    while ((de = readdir(dp))) {
+    remain = sizeof(path) -len -1;
+    while ((de = readdir(dp)) && err == AFP_OK) {
         /* skip this and previous directory */
         if (!strcmp(de->d_name, ".") || !strcmp(de->d_name, ".."))
             continue;
 
-        strncpy(path + len, de->d_name, sizeof(path) - len);
-        if (stat(path, &st) == 0) {
-            if (S_ISDIR(st.st_mode)) {
-                if ((err = deletedir(path)) < 0) {
-                    closedir(dp);
-                    return err;
-                }
-            } else if (unlink(path) < 0) {
-                switch (errno) {
-                case ENOENT :
-                    continue; /* somebody went and deleted it behind our backs. */
-                case EROFS:
-                    err = AFPERR_VLOCK;
-                    break;
-                case EPERM:
-                case EACCES :
-                    err = AFPERR_ACCESS;
-                    break;
-                default :
-                    err = AFPERR_PARAM;
-                }
-                closedir(dp);
-                return err;
-            }
+        if (strlen(de->d_name) > remain) {
+            err = AFPERR_PARAM;
+            break;
+        }
+        strcpy(path + len, de->d_name);
+        if (lstat(path, &st)) {
+            continue;
+        }
+        if (S_ISDIR(st.st_mode)) {
+            err = deletedir(path);
+        } else {
+            err = netatalk_unlink(path);
         }
     }
     closedir(dp);
 
     /* okay. the directory is empty. delete it. note: we already got rid
        of .AppleDouble.  */
-    if (rmdir(dir) < 0) {
-        switch ( errno ) {
-        case ENOENT :
-            break;
-        case ENOTEMPTY : /* should never happen */
-            return( AFPERR_DIRNEMPT );
-        case EPERM:
-        case EACCES :
-            return( AFPERR_ACCESS );
-        case EROFS:
-            return AFPERR_VLOCK;
-        default :
-            return( AFPERR_PARAM );
-        }
+    if (err == AFP_OK) {
+        err = netatalk_rmdir(dir);
     }
-    return AFP_OK;
+    return err;
 }
 
 /* do a recursive copy. */
-static int copydir(char *src, char *dst, int noadouble)
+static int copydir(const struct vol *vol, char *src, char *dst)
 {
     char spath[MAXPATHLEN + 1], dpath[MAXPATHLEN + 1];
     DIR *dp;
-    struct dirent      *de;
+    struct dirent   *de;
     struct stat st;
     struct utimbuf      ut;
     size_t slen, dlen;
+    size_t srem, drem;
     int err;
 
     /* doesn't exist or the path is too long. */
     if (((slen = strlen(src)) > sizeof(spath) - 2) ||
-            ((dlen = strlen(dst)) > sizeof(dpath) - 2) ||
-            ((dp = opendir(src)) == NULL))
+        ((dlen = strlen(dst)) > sizeof(dpath) - 2) ||
+        ((dp = opendir(src)) == NULL))
         return AFPERR_PARAM;
 
     /* try to create the destination directory */
-    if (ad_mkdir(dst, DIRBITS | 0777) < 0) {
+    if (AFP_OK != (err = netatalk_mkdir(dst)) ) {
         closedir(dp);
-        switch ( errno ) {
-        case ENOENT :
-            return( AFPERR_NOOBJ );
-        case EROFS :
-            return( AFPERR_VLOCK );
-        case EPERM:
-        case EACCES :
-            return( AFPERR_ACCESS );
-        case EEXIST :
-            return( AFPERR_EXIST );
-        case ENOSPC :
-        case EDQUOT :
-            return( AFPERR_DFULL );
-        default :
-            return( AFPERR_PARAM );
-        }
+        return err;
     }
 
     /* set things up to copy */
     strcpy(spath, src);
     strcat(spath, "/");
     slen++;
+    srem = sizeof(spath) - slen -1;
+
     strcpy(dpath, dst);
     strcat(dpath, "/");
     dlen++;
+    drem = sizeof(dpath) - dlen -1;
+
     err = AFP_OK;
     while ((de = readdir(dp))) {
         /* skip this and previous directory */
         if (!strcmp(de->d_name, ".") || !strcmp(de->d_name, ".."))
             continue;
 
-        strncpy(spath + slen, de->d_name, sizeof(spath) - slen);
-        if (stat(spath, &st) == 0) {
-            strncpy(dpath + dlen, de->d_name, sizeof(dpath) - dlen);
+        if (strlen(de->d_name) > srem) {
+            err = AFPERR_PARAM;
+            break;
+        }
+        strcpy(spath + slen, de->d_name);
+
+        if (lstat(spath, &st) == 0) {
+            if (strlen(de->d_name) > drem) {
+                err = AFPERR_PARAM;
+                break;
+            }
+            strcpy(dpath + dlen, de->d_name);
 
             if (S_ISDIR(st.st_mode)) {
-                if ((err = copydir(spath, dpath, noadouble)) < 0)
+                if (AFP_OK != (err = copydir(vol, spath, dpath)))
                     goto copydir_done;
-            } else if ((err = copyfile(spath, dpath, NULL, noadouble)) < 0) {
+            } else if (AFP_OK != (err = copyfile(vol, vol, spath, dpath, NULL, NULL))) {
                 goto copydir_done;
 
             } else {
@@ -661,7 +864,7 @@ static int copydir(char *src, char *dst, int noadouble)
     }
 
     /* keep the same time stamp. */
-    if (stat(src, &st) == 0) {
+    if (lstat(src, &st) == 0) {
         ut.actime = ut.modtime = st.st_mtime;
         utime(dst, &ut);
     }
@@ -675,9 +878,7 @@ copydir_done:
 /* --- public functions follow --- */
 
 /* NOTE: we start off with at least one node (the root directory). */
-struct dir *dirinsert( vol, dir )
-            struct vol *vol;
-struct dir     *dir;
+static struct dir *dirinsert(struct vol *vol, struct dir *dir)
 {
     struct dir *node;
 
@@ -688,9 +889,9 @@ struct dir  *dir;
     dir->d_color = DIRTREE_COLOR_RED;
 
     /* parent of this node has to be black. if the parent node
-    * is red, then we have a grandparent. */
+     * is red, then we have a grandparent. */
     while ((dir != vol->v_root) &&
-            (dir->d_back->d_color == DIRTREE_COLOR_RED)) {
+           (dir->d_back->d_color == DIRTREE_COLOR_RED)) {
         /* are we on the left tree? */
         if (dir->d_back == dir->d_back->d_back->d_left) {
             node = dir->d_back->d_back->d_right;  /* get the right node */
@@ -733,10 +934,93 @@ struct dir        *dir;
     return NULL;
 }
 
+/* ---------------------------- */
+struct dir *
+adddir(struct vol *vol, struct dir *dir, struct path *path)
+{
+    struct dir  *cdir, *edir;
+    int     upathlen;
+    char        *name;
+    char        *upath;
+    struct stat *st;
+    int         deleted;
+    cnid_t      id;
+
+    upath = path->u_name;
+    st    = &path->st;
+    upathlen = strlen(upath);
+
+    id = get_id(vol, NULL, st, dir->d_did, upath, upathlen);
+    if (id == 0) {
+        return NULL;
+    }
+    if (!path->m_name && !(path->m_name = utompath(vol, upath, id , utf8_encoding()))) {
+        return NULL;
+    }
+    name  = path->m_name;
+    if ((cdir = dirnew(name, upath)) == NULL) {
+        LOG(log_error, logtype_afpd, "adddir: malloc: %s", strerror(errno) );
+        return NULL;
+    }
+    if ((size_t)-1 == convert_string_allocate((utf8_encoding())?CH_UTF8_MAC:vol->v_maccharset, CH_UCS2, path->m_name, -1, (char **)&cdir->d_m_name_ucs2)) {
+        LOG(log_error, logtype_afpd, "Couldn't set UCS2 name for %s", name);
+        cdir->d_m_name_ucs2 = NULL;
+    }
+
+    cdir->d_did = id;
+
+    if ((edir = dirinsert( vol, cdir ))) {
+        /* it's not possible with LASTDID
+           for CNID:
+           - someone else have moved the directory.
+           - it's a symlink inside the share.
+           - it's an ID reused, the old directory was deleted but not
+           the cnid record and the server've reused the inode for
+           the new dir.
+           for HASH (we should get ride of HASH)
+           - someone else have moved the directory.
+           - it's an ID reused as above
+           - it's a hash duplicate and we are in big trouble
+        */
+        deleted = (edir->d_m_name == NULL);
+        if (!deleted)
+            dir_hash_del(vol, edir);
+        dirfreename(edir);
+        edir->d_m_name = cdir->d_m_name;
+        edir->d_u_name = cdir->d_u_name;
+        edir->d_m_name_ucs2 = cdir->d_m_name_ucs2;
+        free(cdir);
+        cdir = edir;
+        LOG(log_error, logtype_afpd, "adddir: insert %s", edir->d_m_name);
+        if (!cdir->d_parent || (cdir->d_parent == dir && !deleted)) {
+            hash_alloc_insert(vol->v_hash, cdir, cdir);
+            return cdir;
+        }
+        /* the old was not in the same folder */
+        if (!deleted)
+            dirchildremove(cdir->d_parent, cdir);
+    }
+
+    /* parent/child directories */
+    cdir->d_parent = dir;
+    dirchildadd(vol, dir, cdir);
+    return( cdir );
+}
+
+/* --- public functions follow --- */
 /* free everything down. we don't bother to recolor as this is only
  * called to free the entire tree */
-void dirfree( dir )
-struct dir     *dir;
+void dirfreename(struct dir *dir)
+{
+    if (dir->d_u_name != dir->d_m_name) {
+        free(dir->d_u_name);
+    }
+    if (dir->d_m_name_ucs2)
+        free(dir->d_m_name_ucs2);
+    free(dir->d_m_name);
+}
+
+void dirfree(struct dir *dir)
 {
     if (!dir || (dir == SENTINEL))
         return;
@@ -749,17 +1033,14 @@ struct dir       *dir;
     }
 
     if (dir != SENTINEL) {
-        if (dir->d_u_name != dir->d_m_name) {
-            free(dir->d_u_name);
-        }
-        free(dir->d_m_name);
+        dirfreename(dir);
         free( dir );
     }
 }
 
 /* --------------------------------------------
- * most of the time mac name and unix name are the same 
-*/
+ * most of the time mac name and unix name are the same
+ */
 struct dir *dirnew(const char *m_name, const char *u_name)
 {
     struct dir *dir;
@@ -773,7 +1054,7 @@ struct dir *dirnew(const char *m_name, const char *u_name)
         return NULL;
     }
 
-    if (m_name == u_name) {
+    if (m_name == u_name || !strcmp(m_name, u_name)) {
         dir->d_u_name = dir->d_m_name;
     }
     else if ((dir->d_u_name = strdup(u_name)) == NULL) {
@@ -782,102 +1063,253 @@ struct dir *dirnew(const char *m_name, const char *u_name)
         return NULL;
     }
 
+    dir->d_u_name_len = strlen(dir->d_u_name);
+    dir->d_m_name_ucs2 = NULL;
     dir->d_left = dir->d_right = SENTINEL;
     dir->d_next = dir->d_prev = dir;
     return dir;
 }
 
-/* -------------------------------------------------- */
-/* XXX: this needs to be changed to handle path types 
- return
- if it's a filename:
-      in extenddir:
-         compute unix name
-         stat the file
-      return mac name
- if it's a dirname:
-      not in the cache
-          in extenddir
-              compute unix name
-              stat the dir
-      in the cache 
-
-  u_name can be
-  XXXX u_name can be an alias on m_name if the m_name is
-  a valid unix name.
-  
-*/
-struct path *
-cname( vol, dir, cpath )
-const struct vol       *vol;
-struct dir     *dir;
-char   **cpath;
+/* ------------------ */
+static hash_val_t hash_fun_dir(const void *key)
 {
-    struct dir            *cdir;
-    static char                   path[ MAXPATHLEN + 1];
-    static struct path ret;
+    const struct dir *k = key;
+
+    static unsigned long randbox[] = {
+        0x49848f1bU, 0xe6255dbaU, 0x36da5bdcU, 0x47bf94e9U,
+        0x8cbcce22U, 0x559fc06aU, 0xd268f536U, 0xe10af79aU,
+        0xc1af4d69U, 0x1d2917b5U, 0xec4c304dU, 0x9ee5016cU,
+        0x69232f74U, 0xfead7bb3U, 0xe9089ab6U, 0xf012f6aeU,
+    };
+
+    const unsigned char *str = (unsigned char *)(k->d_u_name);
+    hash_val_t acc = k->d_parent->d_did;
+
+    while (*str) {
+        acc ^= randbox[(*str + acc) & 0xf];
+        acc = (acc << 1) | (acc >> 31);
+        acc &= 0xffffffffU;
+        acc ^= randbox[((*str++ >> 4) + acc) & 0xf];
+        acc = (acc << 2) | (acc >> 30);
+        acc &= 0xffffffffU;
+    }
+    return acc;
+}
 
-    char               *data, *p;
-    int                        extend = 0;
-    int                        len;
-    u_int32_t   hint;
-    u_int16_t   len16;
-    int         size = 0;
-    char        sep;
-               
-    data = *cpath;
-    afp_errno = AFPERR_NOOBJ;
-    switch (ret.m_type = *data) { /* path type */
-    case 2:
-       data++;
-       len = (unsigned char) *data++;
-       size = 2;
-       sep = 0;
-       break;
-    case 3:
-       if (afp_version >= 30) {
-           data++;
-           memcpy(&hint, data, sizeof(hint));
-           hint = ntohl(hint);
-           data += sizeof(hint);
-           
-           memcpy(&len16, data, sizeof(len16));
-           len = ntohs(len16);
-           data += 2;
-           size = 7;
-           sep = 0; /* '/';*/
-           break;
-        }
-        /* else it's an error */
-    default:
-        afp_errno = AFPERR_PARAM;
-        return( NULL );
-    
+#undef get16bits
+#if (defined(__GNUC__) && defined(__i386__)) || defined(__WATCOMC__)    \
+    || defined(_MSC_VER) || defined (__BORLANDC__) || defined (__TURBOC__)
+#define get16bits(d) (*((const uint16_t *) (d)))
+#endif
+
+#if !defined (get16bits)
+#define get16bits(d) ((((uint32_t)(((const uint8_t *)(d))[1])) << 8)    \
+                      +(uint32_t)(((const uint8_t *)(d))[0]) )
+#endif
+
+static hash_val_t hash_fun2_dir(const void *key)
+{
+    const struct dir *k = key;
+    const char *data = k->d_u_name;
+    int len = k->d_u_name_len;
+    hash_val_t hash = k->d_parent->d_did, tmp;
+
+    int rem = len & 3;
+    len >>= 2;
+
+    /* Main loop */
+    for (;len > 0; len--) {
+        hash  += get16bits (data);
+        tmp    = (get16bits (data+2) << 11) ^ hash;
+        hash   = (hash << 16) ^ tmp;
+        data  += 2*sizeof (uint16_t);
+        hash  += hash >> 11;
+    }
+
+    /* Handle end cases */
+    switch (rem) {
+    case 3: hash += get16bits (data);
+        hash ^= hash << 16;
+        hash ^= data[sizeof (uint16_t)] << 18;
+        hash += hash >> 11;
+        break;
+    case 2: hash += get16bits (data);
+        hash ^= hash << 11;
+        hash += hash >> 17;
+        break;
+    case 1: hash += *data;
+        hash ^= hash << 10;
+        hash += hash >> 1;
+    }
+
+    /* Force "avalanching" of final 127 bits */
+    hash ^= hash << 3;
+    hash += hash >> 5;
+    hash ^= hash << 4;
+    hash += hash >> 17;
+    hash ^= hash << 25;
+    hash += hash >> 6;
+
+    return hash;
+}
+
+/* ---------------- */
+static int hash_comp_dir(const void *key1, const void *key2)
+{
+    const struct dir *k1 = key1;
+    const struct dir *k2 = key2;
+
+    return !(k1->d_parent->d_did == k2->d_parent->d_did && !strcmp(k1->d_u_name, k2->d_u_name));
+}
+
+/* ---------------- */
+hash_t *
+dirhash(void)
+{
+    return hash_create(HASHCOUNT_T_MAX, hash_comp_dir, hash_fun2_dir);
+}
+
+/* ------------------ */
+static struct path *invalidate (struct vol *vol, struct dir *dir, struct path *ret)
+{
+    /* it's tricky:
+       movecwd failed some of dir path are not there anymore.
+       FIXME Is it true with other errors?
+       so we remove dir from the cache
+    */
+    if (dir->d_did == DIRDID_ROOT_PARENT)
+        return NULL;
+    if (afp_errno == AFPERR_ACCESS) {
+        if ( movecwd( vol, dir->d_parent ) < 0 ) {
+            return NULL;
+        }
+        /* FIXME should we set these?, don't need to call stat() after:
+           ret->st_valid = 1;
+           ret->st_errno = EACCES;
+        */
+        ret->m_name = dir->d_m_name;
+        ret->u_name = dir->d_u_name;
+        ret->d_dir = dir;
+        return ret;
+    } else if (afp_errno == AFPERR_NOOBJ) {
+        if ( movecwd( vol, dir->d_parent ) < 0 ) {
+            return NULL;
+        }
+        strcpy(ret->m_name, dir->d_m_name);
+        if (dir->d_m_name == dir->d_u_name) {
+            ret->u_name = ret->m_name;
+        }
+        else {
+            size_t tp = strlen(ret->m_name)+1;
+
+            ret->u_name =  ret->m_name +tp;
+            strcpy(ret->u_name, dir->d_u_name);
+        }
+        /* FIXME should we set :
+           ret->st_valid = 1;
+           ret->st_errno = ENOENT;
+        */
+        dir_invalidate(vol, dir);
+        return ret;
+    }
+    dir_invalidate(vol, dir);
+    return NULL;
+}
+
+/* -------------------------------------------------- */
+/* cname
+   return
+   if it's a filename:
+   in extenddir:
+   compute unix name
+   stat the file or errno
+   return
+   filename
+   curdir: filename parent directory
+
+   if it's a dirname:
+   not in the cache
+   in extenddir
+   compute unix name
+   stat the dir or errno
+   return
+   if chdir error
+   dirname
+   curdir: dir parent directory
+   sinon
+   dirname: ""
+   curdir: dir
+   in the cache
+   return
+   if chdir error
+   dirname
+   curdir: dir parent directory
+   else
+   dirname: ""
+   curdir: dir
+
+*/
+struct path *
+cname(struct vol *vol, struct dir *dir, char **cpath)
+{
+    struct dir         *cdir, *scdir=NULL;
+    static char        path[ MAXPATHLEN + 1];
+    static struct path ret;
+
+    char        *data, *p;
+    int         extend = 0;
+    int         len;
+    u_int32_t   hint;
+    u_int16_t   len16;
+    int         size = 0;
+    char        sep;
+    int         toUTF8 = 0;
+
+    data = *cpath;
+    afp_errno = AFPERR_NOOBJ;
+    memset(&ret, 0, sizeof(ret));
+    switch (ret.m_type = *data) { /* path type */
+    case 2:
+        data++;
+        len = (unsigned char) *data++;
+        size = 2;
+        sep = 0;
+        if (afp_version >= 30) {
+            ret.m_type = 3;
+            toUTF8 = 1;
+        }
+        break;
+    case 3:
+        if (afp_version >= 30) {
+            data++;
+            memcpy(&hint, data, sizeof(hint));
+            hint = ntohl(hint);
+            data += sizeof(hint);
+
+            memcpy(&len16, data, sizeof(len16));
+            len = ntohs(len16);
+            data += 2;
+            size = 7;
+            sep = 0; /* '/';*/
+            break;
+        }
+        /* else it's an error */
+    default:
+        afp_errno = AFPERR_PARAM;
+        return( NULL );
     }
     *cpath += len + size;
     *path = '\0';
     ret.m_name = path;
-    ret.st_errno = 0;
-    ret.st_valid = 0;
     for ( ;; ) {
         if ( len == 0 ) {
-            if ( !extend && movecwd( vol, dir ) < 0 ) {
-               /* it's tricky:
-                  movecwd failed some of dir path are not there anymore.
-                  FIXME Is it true with other errors?
-                  so we remove dir from the cache 
-               */
-               if (dir->d_did == DIRDID_ROOT_PARENT) 
-                   return NULL;
-               if (afp_errno == AFPERR_ACCESS)
-                   return NULL;                
-
-               dir_invalidate(vol, dir);
-               return NULL;
+            if (movecwd( vol, dir ) < 0 ) {
+                return invalidate(vol, dir, &ret );
             }
             if (*path == '\0') {
-               ret.u_name = ".";
-            }               
+                ret.u_name = ".";
+                ret.d_dir = dir;
+            }
             return &ret;
         }
 
@@ -898,81 +1330,172 @@ char    **cpath;
         p = path;
         while ( *data != sep && len > 0 ) {
             *p++ = *data++;
+            if (p > &path[ MAXPATHLEN]) {
+                afp_errno = AFPERR_PARAM;
+                return( NULL );
+            }
             len--;
         }
 
         /* short cut bits by chopping off a trailing \0. this also
-                  makes the traversal happy w/ filenames at the end of the
-                  cname. */
+           makes the traversal happy w/ filenames at the end of the
+           cname. */
         if (len == 1)
             len--;
 
         *p = '\0';
 
-        if ( p != path ) { /* we got something */
-            if ( !extend ) {
-                cdir = dir->d_child;
+        if ( p == path ) { /* end of the name parameter */
+            continue;
+        }
+        ret.u_name = NULL;
+        if (afp_version >= 30) {
+            char *t;
+            cnid_t fileid;
+
+            if (toUTF8) {
+                static char temp[ MAXPATHLEN + 1];
+
+                if (dir->d_did == DIRDID_ROOT_PARENT) {
+                    /*
+                      With uft8 volume name is utf8-mac, but requested path may be a mangled longname. See #2611981.
+                      So we compare it with the longname from the current volume and if they match
+                      we overwrite the requested path with the utf8 volume name so that the following
+                      strcmp can match.
+                    */
+                    ucs2_to_charset(vol->v_maccharset, vol->v_macname, temp, AFPVOL_MACNAMELEN + 1);
+                    if (strcasecmp( path, temp) == 0)
+                        ucs2_to_charset(CH_UTF8_MAC, vol->v_u8mname, path, AFPVOL_U8MNAMELEN);
+                } else {
+                    /* toUTF8 */
+                    if (mtoUTF8(vol, path, strlen(path), temp, MAXPATHLEN) == (size_t)-1) {
+                        afp_errno = AFPERR_PARAM;
+                        return( NULL );
+                    }
+                    strcpy(path, temp);
+                }
+            }
+            /* check for OS X mangled filename :( */
+
+            t = demangle_osx(vol, path, dir->d_did, &fileid);
+            if (t != path) {
+                ret.u_name = t;
+                /* duplicate work but we can't reuse all convert_char we did in demangle_osx
+                 * flags weren't the same
+                 */
+                if ( (t = utompath(vol, ret.u_name, fileid, utf8_encoding())) ) {
+                    /* at last got our view of mac name */
+                    strcpy(path,t);
+                }
+            }
+        }
+        if (ret.u_name == NULL) {
+            if (!(ret.u_name = mtoupath(vol, ret.m_name, dir->d_did, utf8_encoding()))) {
+                afp_errno = AFPERR_PARAM;
+                return NULL;
+            }
+        }
+        if ( !extend ) {
+            ucs2_t *tmpname;
+            cdir = dir->d_child;
+            scdir = NULL;
+            if ( cdir && (vol->v_flags & AFPVOL_CASEINSEN) &&
+                 (size_t)-1 != convert_string_allocate(((ret.m_type == 3)?CH_UTF8_MAC:vol->v_maccharset),
+                                                       CH_UCS2, path, -1, (char **)&tmpname) )
+            {
                 while (cdir) {
-                    if ( strcmp( cdir->d_m_name, path ) == 0 ) {
+                    if (!cdir->d_m_name_ucs2) {
+                        LOG(log_error, logtype_afpd, "cname: no UCS2 name for %s (did %u)!!!", cdir->d_m_name, ntohl(cdir->d_did) );
+                        /* this shouldn't happen !!!! */
+                        goto noucsfallback;
+                    }
+
+                    if ( strcmp_w( cdir->d_m_name_ucs2, tmpname ) == 0 ) {
                         break;
                     }
-                    cdir = (cdir == dir->d_child->d_prev) ? NULL :
-                           cdir->d_next;
+                    if ( strcasecmp_w( cdir->d_m_name_ucs2, tmpname ) == 0 ) {
+                        scdir = cdir;
+                    }
+                    cdir = (cdir == dir->d_child->d_prev) ? NULL :cdir->d_next;
                 }
-                if ( cdir == NULL ) {
-                    ++extend;
-                    /* if dir == curdir it always succeed,
-                       even if curdir is deleted. 
-                       it's not a pb because it will fail in extenddir
+                free(tmpname);
+            }
+            else {
+            noucsfallback:
+                if (dir->d_did == DIRDID_ROOT_PARENT) {
+                    /*
+                      root parent (did 1) has one child: the volume. Requests for did=1 with some <name>
+                      must check against the volume name.
                     */
-                    if ( movecwd( vol, dir ) < 0 ) {
-                       /* dir is not valid anymore 
-                          we delete dir from the cache and abort.
-                       */
-                       if ( dir->d_did != DIRDID_ROOT_PARENT && 
-                             (afp_errno != AFPERR_ACCESS)) {
-                           dir_invalidate(vol, dir);
-                       }
-                        return NULL;
-                    }
-                    cdir = extenddir( vol, dir, &ret );
+                    if (!strcmp(vol->v_dir->d_m_name, ret.m_name))
+                        cdir = vol->v_dir;
+                    else
+                        cdir = NULL;
                 }
+                else {
+                    cdir = dirsearch_byname(vol, dir, ret.u_name);
+                }
+            }
 
-            } else {
-                cdir = extenddir( vol, dir, &ret );
+            if (cdir == NULL && scdir != NULL) {
+                cdir = scdir;
+                /* LOG(log_debug, logtype_afpd, "cname: using casediff for %s, (%s = %s)", fullpathname(cdir->d_u_name), cdir->d_m_name, path ); */
             }
 
             if ( cdir == NULL ) {
-
-                if ( len > 0 ) {
+                ++extend;
+                /* if dir == curdir it always succeed,
+                   even if curdir is deleted.
+                   it's not a pb because it will fail in extenddir
+                */
+                if ( movecwd( vol, dir ) < 0 ) {
+                    /* dir is not valid anymore
+                       we delete dir from the cache and abort.
+                    */
+                    if ( dir->d_did == DIRDID_ROOT_PARENT) {
+                        afp_errno = AFPERR_NOOBJ;
+                        return NULL;
+                    }
+                    if (afp_errno == AFPERR_ACCESS)
+                        return NULL;
+                    dir_invalidate(vol, dir);
                     return NULL;
                 }
+                cdir = extenddir( vol, dir, &ret );
+            }
 
-            } else {
-                dir = cdir;    
-                *path = '\0';
+        } else {
+            cdir = extenddir( vol, dir, &ret );
+        } /* if (!extend) */
+
+        if ( cdir == NULL ) {
+
+            if ( len > 0 || !ret.u_name ) {
+                return NULL;
             }
+
+        } else {
+            dir = cdir;
+            *path = '\0';
         }
-    }
+    } /* for (;;) */
 }
 
 /*
  * Move curdir to dir, with a possible chdir()
  */
-int movecwd( vol, dir)
-const struct vol       *vol;
-struct dir     *dir;
+int movecwd(struct vol *vol, struct dir *dir)
 {
     char path[MAXPATHLEN + 1];
-    struct dir *d;
-    char       *p, *u;
-    int                n;
+    struct dir  *d;
+    char    *p, *u;
+    int     n;
 
     if ( dir == curdir ) {
         return( 0 );
     }
     if ( dir->d_did == DIRDID_ROOT_PARENT) {
-        afp_errno = AFPERR_PARAM;
+        afp_errno = AFPERR_DID1; /* AFPERR_PARAM;*/
         return( -1 );
     }
 
@@ -981,6 +1504,11 @@ struct dir        *dir;
     *p = '.';
     for ( d = dir; d->d_parent != NULL && d != curdir; d = d->d_parent ) {
         u = d->d_u_name;
+        if (!u) {
+            /* parent directory is deleted */
+            afp_errno = AFPERR_NOOBJ;
+            return -1;
+        }
         n = strlen( u );
         if (p -n -1 < path) {
             afp_errno = AFPERR_PARAM;
@@ -988,7 +1516,7 @@ struct dir *dir;
         }
         *--p = '/';
         p -= n;
-        strncpy( p, u, n );
+        memcpy( p, u, n );
     }
     if ( d != curdir ) {
         n = strlen( vol->v_path );
@@ -998,7 +1526,7 @@ struct dir *dir;
         }
         *--p = '/';
         p -= n;
-        strncpy( p, vol->v_path, n );
+        memcpy( p, vol->v_path, n );
     }
     if ( chdir( p ) < 0 ) {
         switch (errno) {
@@ -1008,11 +1536,11 @@ struct dir      *dir;
             break;
         default:
             afp_errno = AFPERR_NOOBJ;
-        
+
         }
         return( -1 );
     }
-    curdir = dir;
+    vol->v_curdir = curdir = dir;
     return( 0 );
 }
 
@@ -1020,18 +1548,18 @@ struct dir      *dir;
  * We can't use unix file's perm to support Apple's inherited protection modes.
  * If we aren't the file's owner we can't change its perms when moving it and smb
  * nfs,... don't even try.
-*/
-#define AFP_CHECK_ACCESS 
+ */
+#define AFP_CHECK_ACCESS
 
 int check_access(char *path, int mode)
 {
 #ifdef AFP_CHECK_ACCESS
-struct maccess ma;
-char *p;
+    struct maccess ma;
+    char *p;
 
     p = ad_dir(path);
     if (!p)
-       return -1;
+        return -1;
 
     accessmode(p, &ma, curdir, NULL);
     if ((mode & OPENACC_WR) && !(ma.ma_user & AR_UWRITE))
@@ -1042,39 +1570,94 @@ char *p;
     return 0;
 }
 
-/* ------------------------------ 
+/* --------------------- */
+int file_access(struct path *path, int mode)
+{
+    struct maccess ma;
+
+    accessmode(path->u_name, &ma, curdir, &path->st);
+    if ((mode & OPENACC_WR) && !(ma.ma_user & AR_UWRITE))
+        return -1;
+    if ((mode & OPENACC_RD) && !(ma.ma_user & AR_UREAD))
+        return -1;
+    return 0;
+
+}
+
+/* --------------------- */
+void setdiroffcnt(struct dir *dir, struct stat *st,  u_int32_t count)
+{
+    dir->offcnt = count;
+    dir->ctime = st->st_ctime;
+    dir->d_flags &= ~DIRF_CNID;
+}
+
+/* ---------------------
+ * is our cached offspring count valid?
+ */
+
+static int diroffcnt(struct dir *dir, struct stat *st)
+{
+    return st->st_ctime == dir->ctime;
+}
+
+/* ---------------------
+ * is our cached also for reenumerate id?
+ */
+
+int dirreenumerate(struct dir *dir, struct stat *st)
+{
+    return st->st_ctime == dir->ctime && (dir->d_flags & DIRF_CNID);
+}
+
+/* --------------------- */
+static int invisible_dots(const struct vol *vol, const char *name)
+{
+    return vol_inv_dots(vol) && *name  == '.' && strcmp(name, ".") && strcmp(name, "..");
+}
+
+/* ------------------------------
    (".", curdir)
    (name, dir) with curdir:name == dir, from afp_enumerate
 */
 
 int getdirparams(const struct vol *vol,
                  u_int16_t bitmap, struct path *s_path,
-                 struct dir *dir, 
-                 char *buf, int *buflen )
+                 struct dir *dir,
+                 char *buf, size_t *buflen )
 {
-    struct maccess     ma;
-    struct adouble     ad;
-    char               *data, *l_nameoff = NULL, *utf_nameoff = NULL;
-    int                        bit = 0, isad = 0;
+    struct maccess  ma;
+    struct adouble  ad;
+    char        *data, *l_nameoff = NULL, *utf_nameoff = NULL;
+    int         bit = 0, isad = 0;
     u_int32_t           aint;
-    u_int16_t          ashort;
+    u_int16_t       ashort;
     int                 ret;
     u_int32_t           utf8 = 0;
+    cnid_t              pdid;
     struct stat *st = &s_path->st;
     char *upath = s_path->u_name;
-    
+
     if ((bitmap & ((1 << DIRPBIT_ATTR)  |
-                  (1 << DIRPBIT_CDATE) |
-                  (1 << DIRPBIT_MDATE) |
-                  (1 << DIRPBIT_BDATE) |
-                  (1 << DIRPBIT_FINFO)))) {
-        memset(&ad, 0, sizeof(ad));
-       if ( !ad_open( upath, ADFLAGS_HF|ADFLAGS_DIR, O_RDONLY,
-                  DIRBITS | 0777, &ad)) {
+                   (1 << DIRPBIT_CDATE) |
+                   (1 << DIRPBIT_MDATE) |
+                   (1 << DIRPBIT_BDATE) |
+                   (1 << DIRPBIT_FINFO)))) {
+
+        ad_init(&ad, vol->v_adouble, vol->v_ad_options);
+        if ( !ad_metadata( upath, vol_noadouble(vol) | ADFLAGS_DIR, &ad) ) {
             isad = 1;
         }
     }
 
+    if ( dir->d_did == DIRDID_ROOT) {
+        pdid = DIRDID_ROOT_PARENT;
+    } else if (dir->d_did == DIRDID_ROOT_PARENT) {
+        pdid = 0;
+    } else {
+        pdid = dir->d_parent->d_did;
+    }
+
     data = buf;
     while ( bitmap != 0 ) {
         while (( bitmap & 1 ) == 0 ) {
@@ -1086,8 +1669,7 @@ int getdirparams(const struct vol *vol,
         case DIRPBIT_ATTR :
             if ( isad ) {
                 ad_getattr(&ad, &ashort);
-            } else if (*upath == '.' && strcmp(upath, ".") &&
-                       strcmp(upath, "..")) {
+            } else if (invisible_dots(vol, dir->d_u_name)) {
                 ashort = htons(ATTRBIT_INVISIBLE);
             } else
                 ashort = 0;
@@ -1097,15 +1679,8 @@ int getdirparams(const struct vol *vol,
             break;
 
         case DIRPBIT_PDID :
-            if ( dir->d_did == DIRDID_ROOT) {
-                aint = DIRDID_ROOT_PARENT;
-            } else if (dir->d_did == DIRDID_ROOT_PARENT) {
-                aint = 0;
-            } else {
-                aint = dir->d_parent->d_did;
-            }
-            memcpy( data, &aint, sizeof( aint ));
-            data += sizeof( aint );
+            memcpy( data, &pdid, sizeof( pdid ));
+            data += sizeof( pdid );
             break;
 
         case DIRPBIT_CDATE :
@@ -1137,12 +1712,10 @@ int getdirparams(const struct vol *vol,
                 ashort = htons(FINDERINFO_CLOSEDVIEW);
                 memcpy(data + FINDERINFO_FRVIEWOFF, &ashort, sizeof(ashort));
 
-                /* dot files are by default invisible */
-                if (*upath == '.' && strcmp(upath, ".") &&
-                        strcmp(upath, "..")) {
+                /* dot files are by default visible */
+                if (invisible_dots(vol, dir->d_u_name)) {
                     ashort = htons(FINDERINFO_INVISIBLE);
-                    memcpy(data + FINDERINFO_FRFLAGOFF,
-                           &ashort, sizeof(ashort));
+                    memcpy(data + FINDERINFO_FRFLAGOFF, &ashort, sizeof(ashort));
                 }
             }
             data += 32;
@@ -1169,13 +1742,12 @@ int getdirparams(const struct vol *vol,
         case DIRPBIT_OFFCNT :
             ashort = 0;
             /* this needs to handle current directory access rights */
-            if (st->st_ctime == dir->ctime) {
-               ashort = dir->offcnt;
+            if (diroffcnt(dir, st)) {
+                ashort = (dir->offcnt > 0xffff)?0xffff:dir->offcnt;
             }
             else if ((ret = for_each_dirent(vol, upath, NULL,NULL)) >= 0) {
-                ashort = ret;
-                dir->offcnt = ashort;
-                dir->ctime = st->st_ctime;
+                setdiroffcnt(dir, st,  ret);
+                ashort = (dir->offcnt > 0xffff)?0xffff:dir->offcnt;
             }
             ashort = htons( ashort );
             memcpy( data, &ashort, sizeof( ashort ));
@@ -1206,7 +1778,7 @@ int getdirparams(const struct vol *vol,
             /* Client has requested the ProDOS information block.
                Just pass back the same basic block for all
                directories. <shirsch@ibm.net> */
-        case DIRPBIT_PDINFO :                    
+        case DIRPBIT_PDINFO :
             if (afp_version >= 30) { /* UTF8 name */
                 utf8 = kTextEncodingUTF8;
                 if (dir->d_m_name) /* root of parent can have a null name */
@@ -1229,9 +1801,30 @@ int getdirparams(const struct vol *vol,
             }
             break;
 
+        case DIRPBIT_UNIXPR :
+            aint = htonl(st->st_uid);
+            memcpy( data, &aint, sizeof( aint ));
+            data += sizeof( aint );
+            aint = htonl(st->st_gid);
+            memcpy( data, &aint, sizeof( aint ));
+            data += sizeof( aint );
+
+            aint = st->st_mode;
+            aint = htonl ( aint & ~S_ISGID );  /* Remove SGID, OSX doesn't like it ... */
+            memcpy( data, &aint, sizeof( aint ));
+            data += sizeof( aint );
+
+            accessmode( upath, &ma, dir , st);
+
+            *data++ = ma.ma_user;
+            *data++ = ma.ma_world;
+            *data++ = ma.ma_group;
+            *data++ = ma.ma_owner;
+            break;
+
         default :
             if ( isad ) {
-                ad_close( &ad, ADFLAGS_HF );
+                ad_close_metadata( &ad );
             }
             return( AFPERR_BITMAP );
         }
@@ -1241,39 +1834,50 @@ int getdirparams(const struct vol *vol,
     if ( l_nameoff ) {
         ashort = htons( data - buf );
         memcpy( l_nameoff, &ashort, sizeof( ashort ));
-        data = set_name(data, dir->d_m_name, 0);
+        data = set_name(vol, data, pdid, dir->d_m_name, dir->d_did, 0);
     }
     if ( utf_nameoff ) {
         ashort = htons( data - buf );
         memcpy( utf_nameoff, &ashort, sizeof( ashort ));
-        data = set_name(data, dir->d_m_name, utf8);
+        data = set_name(vol, data, pdid, dir->d_m_name, dir->d_did, utf8);
     }
     if ( isad ) {
-        ad_close( &ad, ADFLAGS_HF );
+        ad_close_metadata( &ad );
     }
     *buflen = data - buf;
     return( AFP_OK );
 }
 
 /* ----------------------------- */
-int afp_setdirparams(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int path_error(struct path *path, int error)
+{
+/* - a dir with access error
+ * - no error it's a file
+ * - file not found
+ */
+    if (path_isadir(path))
+        return afp_errno;
+    if (path->st_valid && path->st_errno)
+        return error;
+    return AFPERR_BADTYPE ;
+}
+
+/* ----------------------------- */
+int afp_setdirparams(AFPObj *obj, char *ibuf, size_t ibuflen _U_, char *rbuf _U_, size_t *rbuflen)
 {
-    struct vol *vol;
-    struct dir *dir;
+    struct vol  *vol;
+    struct dir  *dir;
     struct path *path;
-    u_int16_t  vid, bitmap;
+    u_int16_t   vid, bitmap;
     u_int32_t   did;
-    int                rc;
+    int     rc;
 
     *rbuflen = 0;
     ibuf += 2;
     memcpy( &vid, ibuf, sizeof( vid ));
     ibuf += sizeof( vid );
 
-    if (( vol = getvolbyvid( vid )) == NULL ) {
+    if (NULL == ( vol = getvolbyvid( vid )) ) {
         return( AFPERR_PARAM );
     }
 
@@ -1283,7 +1887,7 @@ int               ibuflen, *rbuflen;
     memcpy( &did, ibuf, sizeof( did ));
     ibuf += sizeof( int );
 
-    if (( dir = dirlookup( vol, did )) == NULL ) {
+    if (NULL == ( dir = dirlookup( vol, did )) ) {
         return afp_errno;
     }
 
@@ -1291,12 +1895,15 @@ int             ibuflen, *rbuflen;
     bitmap = ntohs( bitmap );
     ibuf += sizeof( bitmap );
 
-    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-        return afp_errno;
+    if (NULL == ( path = cname( vol, dir, &ibuf )) ) {
+        return get_afp_errno(AFPERR_NOOBJ);
     }
 
     if ( *path->m_name != '\0' ) {
-        return( AFPERR_BADTYPE ); /* not a directory */
+        rc = path_error(path, AFPERR_NOOBJ);
+        /* maybe we are trying to set perms back */
+        if (rc != AFPERR_ACCESS)
+            return rc;
     }
 
     /*
@@ -1306,47 +1913,173 @@ int            ibuflen, *rbuflen;
         ibuf++;
     }
 
-    if (( rc = setdirparams(vol, path, bitmap, ibuf )) == AFP_OK ) {
+    if (AFP_OK == ( rc = setdirparams(vol, path, bitmap, ibuf )) ) {
         setvoltime(obj, vol );
     }
     return( rc );
 }
 
 /*
- * cf AFP3.0.pdf page 244 for change_mdate and change_parent_mdate logic  
+ * cf AFP3.0.pdf page 244 for change_mdate and change_parent_mdate logic
  *
  * assume path == '\0' eg. it's a directory in canonical form
-*/
+ */
 
 struct path Cur_Path = {
     0,
     "",  /* mac name */
     ".", /* unix name */
-    0,  /* stat is not set */
-    0,  /* */
+    0,   /* id */
+    NULL,/* struct dir */
+    0,   /* stat is not set */
 };
 
-int setdirparams(const struct vol *vol, 
-                 struct path *path, u_int16_t bitmap, char *buf )
+/* ------------------ */
+static int set_dir_errors(struct path *path, const char *where, int err)
 {
-    struct maccess     ma;
-    struct adouble     ad;
+    switch ( err ) {
+    case EPERM :
+    case EACCES :
+        return AFPERR_ACCESS;
+    case EROFS :
+        return AFPERR_VLOCK;
+    }
+    LOG(log_error, logtype_afpd, "setdirparam(%s): %s: %s", fullpathname(path->u_name), where, strerror(err) );
+    return AFPERR_PARAM;
+}
+
+/* ------------------ */
+int setdirparams(struct vol *vol,
+                 struct path *path, u_int16_t d_bitmap, char *buf )
+{
+    struct maccess  ma;
+    struct adouble  ad;
     struct utimbuf      ut;
     struct timeval      tv;
 
     char                *upath;
-    int                        bit = 0, aint, isad = 1;
-    u_int16_t          ashort, bshort;
+    struct dir          *dir;
+    int         bit, isad = 1;
+    int                 cdate, bdate;
+    int                 owner, group;
+    u_int16_t       ashort, bshort;
     int                 err = AFP_OK;
     int                 change_mdate = 0;
     int                 change_parent_mdate = 0;
     int                 newdate = 0;
+    u_int16_t           bitmap = d_bitmap;
+    u_char              finder_buf[32];
+    u_int32_t       upriv;
+    mode_t              mpriv = 0;
+    u_int16_t           upriv_bit = 0;
 
+    bit = 0;
     upath = path->u_name;
-    memset(&ad, 0, sizeof(ad));
+    dir   = path->d_dir;
+    while ( bitmap != 0 ) {
+        while (( bitmap & 1 ) == 0 ) {
+            bitmap = bitmap>>1;
+            bit++;
+        }
+
+        switch( bit ) {
+        case DIRPBIT_ATTR :
+            change_mdate = 1;
+            memcpy( &ashort, buf, sizeof( ashort ));
+            buf += sizeof( ashort );
+            break;
+        case DIRPBIT_CDATE :
+            change_mdate = 1;
+            memcpy(&cdate, buf, sizeof(cdate));
+            buf += sizeof( cdate );
+            break;
+        case DIRPBIT_MDATE :
+            memcpy(&newdate, buf, sizeof(newdate));
+            buf += sizeof( newdate );
+            break;
+        case DIRPBIT_BDATE :
+            change_mdate = 1;
+            memcpy(&bdate, buf, sizeof(bdate));
+            buf += sizeof( bdate );
+            break;
+        case DIRPBIT_FINFO :
+            change_mdate = 1;
+            memcpy( finder_buf, buf, 32 );
+            buf += 32;
+            break;
+        case DIRPBIT_UID :  /* What kind of loser mounts as root? */
+            change_parent_mdate = 1;
+            memcpy( &owner, buf, sizeof(owner));
+            buf += sizeof( owner );
+            break;
+        case DIRPBIT_GID :
+            change_parent_mdate = 1;
+            memcpy( &group, buf, sizeof( group ));
+            buf += sizeof( group );
+            break;
+        case DIRPBIT_ACCESS :
+            change_mdate = 1;
+            change_parent_mdate = 1;
+            ma.ma_user = *buf++;
+            ma.ma_world = *buf++;
+            ma.ma_group = *buf++;
+            ma.ma_owner = *buf++;
+            mpriv = mtoumode( &ma ) | vol->v_dperm;
+            if (dir_rx_set(mpriv) && setdirmode( vol, upath, mpriv) < 0 ) {
+                err = set_dir_errors(path, "setdirmode", errno);
+                bitmap = 0;
+            }
+            break;
+            /* Ignore what the client thinks we should do to the
+               ProDOS information block.  Skip over the data and
+               report nothing amiss. <shirsch@ibm.net> */
+        case DIRPBIT_PDINFO :
+            if (afp_version < 30) {
+                buf += 6;
+            }
+            else {
+                err = AFPERR_BITMAP;
+                bitmap = 0;
+            }
+            break;
+        case DIRPBIT_UNIXPR :
+            if (vol_unix_priv(vol)) {
+                memcpy( &owner, buf, sizeof(owner)); /* FIXME need to change owner too? */
+                buf += sizeof( owner );
+                memcpy( &group, buf, sizeof( group ));
+                buf += sizeof( group );
+
+                change_mdate = 1;
+                change_parent_mdate = 1;
+                memcpy( &upriv, buf, sizeof( upriv ));
+                buf += sizeof( upriv );
+                upriv = ntohl (upriv) | vol->v_dperm;
+                if (dir_rx_set(upriv)) {
+                    /* maybe we are trying to set perms back */
+                    if ( setdirunixmode(vol, upath, upriv) < 0 ) {
+                        bitmap = 0;
+                        err = set_dir_errors(path, "setdirunixmode", errno);
+                    }
+                }
+                else {
+                    /* do it later */
+                    upriv_bit = 1;
+                }
+                break;
+            }
+            /* fall through */
+        default :
+            err = AFPERR_BITMAP;
+            bitmap = 0;
+            break;
+        }
+
+        bitmap = bitmap>>1;
+        bit++;
+    }
+    ad_init(&ad, vol->v_adouble, vol->v_ad_options);
 
-    if (ad_open( upath, vol_noadouble(vol)|ADFLAGS_HF|ADFLAGS_DIR,
-                 O_RDWR|O_CREAT, 0666, &ad) < 0) {
+    if (ad_open_metadata( upath, vol_noadouble(vol)|ADFLAGS_DIR, O_CREAT, &ad) < 0) {
         /*
          * Check to see what we're trying to set.  If it's anything
          * but ACCESS, UID, or GID, give an error.  If it's any of those
@@ -1355,8 +2088,9 @@ int setdirparams(const struct vol *vol,
          * note: we also don't need to worry about mdate. also, be quiet
          *       if we're using the noadouble option.
          */
-        if (!vol_noadouble(vol) && (bitmap &
-                                    ~((1<<DIRPBIT_ACCESS)|(1<<DIRPBIT_UID)|(1<<DIRPBIT_GID)|
+        if (!vol_noadouble(vol) && (d_bitmap &
+                                    ~((1<<DIRPBIT_ACCESS)|(1<<DIRPBIT_UNIXPR)|
+                                      (1<<DIRPBIT_UID)|(1<<DIRPBIT_GID)|
                                       (1<<DIRPBIT_MDATE)|(1<<DIRPBIT_PDINFO)))) {
             return AFPERR_ACCESS;
         }
@@ -1367,13 +2101,13 @@ int setdirparams(const struct vol *vol,
          * Check to see if a create was necessary. If it was, we'll want
          * to set our name, etc.
          */
-        if ( ad_getoflags( &ad, ADFLAGS_HF ) & O_CREAT ) {
-            ad_setentrylen( &ad, ADEID_NAME, strlen( curdir->d_m_name ));
-            memcpy( ad_entry( &ad, ADEID_NAME ), curdir->d_m_name,
-                    ad_getentrylen( &ad, ADEID_NAME ));
+        if ( (ad_get_HF_flags( &ad ) & O_CREAT)) {
+            ad_setname(&ad, curdir->d_m_name);
         }
     }
 
+    bit = 0;
+    bitmap = d_bitmap;
     while ( bitmap != 0 ) {
         while (( bitmap & 1 ) == 0 ) {
             bitmap = bitmap>>1;
@@ -1382,212 +2116,124 @@ int setdirparams(const struct vol *vol,
 
         switch( bit ) {
         case DIRPBIT_ATTR :
-            change_mdate = 1;
             if (isad) {
-                memcpy( &ashort, buf, sizeof( ashort ));
                 ad_getattr(&ad, &bshort);
+                if ((bshort & htons(ATTRBIT_INVISIBLE)) !=
+                    (ashort & htons(ATTRBIT_INVISIBLE) & htons(ATTRBIT_SETCLR)) )
+                    change_parent_mdate = 1;
                 if ( ntohs( ashort ) & ATTRBIT_SETCLR ) {
                     bshort |= htons( ntohs( ashort ) & ~ATTRBIT_SETCLR );
                 } else {
                     bshort &= ~ashort;
                 }
                 ad_setattr(&ad, bshort);
-                if ((ashort & htons(ATTRBIT_INVISIBLE)))
-                  change_parent_mdate = 1;
             }
-            buf += sizeof( ashort );
             break;
-
         case DIRPBIT_CDATE :
-            change_mdate = 1;
             if (isad) {
-                memcpy(&aint, buf, sizeof(aint));
-                ad_setdate(&ad, AD_DATE_CREATE, aint);
+                ad_setdate(&ad, AD_DATE_CREATE, cdate);
             }
-            buf += sizeof( aint );
             break;
-
         case DIRPBIT_MDATE :
-            memcpy(&newdate, buf, sizeof(newdate));
-            buf += sizeof( newdate );
             break;
-
         case DIRPBIT_BDATE :
-            change_mdate = 1;
             if (isad) {
-                memcpy(&aint, buf, sizeof(aint));
-                ad_setdate(&ad, AD_DATE_BACKUP, aint);
+                ad_setdate(&ad, AD_DATE_BACKUP, bdate);
             }
-            buf += sizeof( aint );
             break;
-
         case DIRPBIT_FINFO :
-            change_mdate = 1;
-            /*
-             * Alright, we admit it, this is *really* sick!
-             * The 4 bytes that we don't copy, when we're dealing
-             * with the root of a volume, are the directory's
-             * location information. This eliminates that annoying
-             * behavior one sees when mounting above another mount
-             * point.
-             */
             if (isad) {
-                if (  curdir->d_did == DIRDID_ROOT ) {
-                    memcpy( ad_entry( &ad, ADEID_FINDERI ), buf, 10 );
-                    memcpy( ad_entry( &ad, ADEID_FINDERI ) + 14, buf + 14, 18 );
+                /* Fixes #2802236 */
+                u_int16_t *fflags = (u_int16_t *)(finder_buf + FINDERINFO_FRFLAGOFF);
+                *fflags &= htons(~FINDERINFO_ISHARED);
+                /* #2802236 end */
+                if (  dir->d_did == DIRDID_ROOT ) {
+                    /*
+                     * Alright, we admit it, this is *really* sick!
+                     * The 4 bytes that we don't copy, when we're dealing
+                     * with the root of a volume, are the directory's
+                     * location information. This eliminates that annoying
+                     * behavior one sees when mounting above another mount
+                     * point.
+                     */
+                    memcpy( ad_entry( &ad, ADEID_FINDERI ), finder_buf, 10 );
+                    memcpy( ad_entry( &ad, ADEID_FINDERI ) + 14, finder_buf + 14, 18 );
                 } else {
-                    memcpy( ad_entry( &ad, ADEID_FINDERI ), buf, 32 );
+                    memcpy( ad_entry( &ad, ADEID_FINDERI ), finder_buf, 32 );
                 }
             }
-            buf += 32;
             break;
-
-        case DIRPBIT_UID :     /* What kind of loser mounts as root? */
-            change_parent_mdate = 1;
-            memcpy( &aint, buf, sizeof(aint));
-            buf += sizeof( aint );
-            if ( (curdir->d_did == DIRDID_ROOT) &&
-                    (setdeskowner( ntohl(aint), -1 ) < 0)) {
-                switch ( errno ) {
-                case EPERM :
-                case EACCES :
-                    err = AFPERR_ACCESS;
-                    goto setdirparam_done;
-                    break;
-                case EROFS :
-                    err = AFPERR_VLOCK;
-                    goto setdirparam_done;
-                    break;
-                default :
-                    LOG(log_error, logtype_afpd, "setdirparam: setdeskowner: %s",
-                        strerror(errno) );
-                    if (!isad) {
-                        err = AFPERR_PARAM;
-                        goto setdirparam_done;
-                    }
-                    break;
+        case DIRPBIT_UID :  /* What kind of loser mounts as root? */
+            if ( (dir->d_did == DIRDID_ROOT) &&
+                 (setdeskowner( ntohl(owner), -1 ) < 0)) {
+                err = set_dir_errors(path, "setdeskowner", errno);
+                if (isad && err == AFPERR_PARAM) {
+                    err = AFP_OK; /* ???*/
                 }
-            }
-            if ( setdirowner( ntohl(aint), -1, vol_noadouble(vol) ) < 0 ) {
-                switch ( errno ) {
-                case EPERM :
-                case EACCES :
-                    err = AFPERR_ACCESS;
+                else {
                     goto setdirparam_done;
-                    break;
-                case EROFS :
-                    err = AFPERR_VLOCK;
-                    goto setdirparam_done;
-                    break;
-                default :
-                    LOG(log_error, logtype_afpd, "setdirparam: setdirowner: %s",
-                        strerror(errno) );
-                    break;
                 }
             }
+            if ( setdirowner(vol, upath, ntohl(owner), -1 ) < 0 ) {
+                err = set_dir_errors(path, "setdirowner", errno);
+                goto setdirparam_done;
+            }
             break;
         case DIRPBIT_GID :
-            change_parent_mdate = 1;
-            memcpy( &aint, buf, sizeof( aint ));
-            buf += sizeof( aint );
-            if (curdir->d_did == DIRDID_ROOT)
-                setdeskowner( -1, ntohl(aint) );
-
-#if 0       /* don't error if we can't set the desktop owner. */
-            switch ( errno ) {
-            case EPERM :
-            case EACCES :
-                err = AFPERR_ACCESS;
-                goto setdirparam_done;
-                break;
-            case EROFS :
-                err = AFPERR_VLOCK;
+            if (dir->d_did == DIRDID_ROOT)
+                setdeskowner( -1, ntohl(group) );
+            if ( setdirowner(vol, upath, -1, ntohl(group) ) < 0 ) {
+                err = set_dir_errors(path, "setdirowner", errno);
                 goto setdirparam_done;
-                break;
-            default :
-                LOG(log_error, logtype_afpd, "setdirparam: setdeskowner: %m" );
-                if (!isad) {
-                    err = AFPERR_PARAM;
-                    goto setdirparam_done;
-                }
-                break;
             }
-#endif /* 0 */
-
-            if ( setdirowner( -1, ntohl(aint), vol_noadouble(vol) ) < 0 ) {
-                switch ( errno ) {
-                case EPERM :
-                case EACCES :
+            break;
+        case DIRPBIT_ACCESS :
+            if (dir->d_did == DIRDID_ROOT) {
+                setdeskmode(mpriv);
+                if (!dir_rx_set(mpriv)) {
+                    /* we can't remove read and search for owner on volume root */
                     err = AFPERR_ACCESS;
                     goto setdirparam_done;
-                    break;
-                case EROFS :
-                    err = AFPERR_VLOCK;
-                    goto setdirparam_done;
-                    break;
-                default :
-                    LOG(log_error, logtype_afpd, "setdirparam: setdirowner: %s",
-                        strerror(errno) );
-                    break;
                 }
             }
-            break;
 
-        case DIRPBIT_ACCESS :
-            change_mdate = 1;
-            change_parent_mdate = 1;
-            ma.ma_user = *buf++;
-            ma.ma_world = *buf++;
-            ma.ma_group = *buf++;
-            ma.ma_owner = *buf++;
-
-            if (curdir->d_did == DIRDID_ROOT)
-                setdeskmode(mtoumode( &ma ));
-#if 0 /* don't error if we can't set the desktop mode */
-            switch ( errno ) {
-            case EPERM :
-            case EACCES :
-                err = AFPERR_ACCESS;
+            if (!dir_rx_set(mpriv) && setdirmode( vol, upath, mpriv) < 0 ) {
+                err = set_dir_errors(path, "setdirmode", errno);
                 goto setdirparam_done;
-            case EROFS :
-                err = AFPERR_VLOCK;
-                goto setdirparam_done;
-            default :
-                LOG(log_error, logtype_afpd, "setdirparam: setdeskmode: %s",
-                    strerror(errno) );
-                break;
-                err = AFPERR_PARAM;
+            }
+            break;
+        case DIRPBIT_PDINFO :
+            if (afp_version >= 30) {
+                err = AFPERR_BITMAP;
                 goto setdirparam_done;
             }
-#endif /* 0 */
-
-            if ( setdirmode( mtoumode( &ma ), vol_noadouble(vol),
-                         (vol->v_flags & AFPVOL_DROPBOX)) < 0 ) {
-                switch ( errno ) {
-                case EPERM :
-                case EACCES :
-                    err = AFPERR_ACCESS;
-                    goto setdirparam_done;
-                case EROFS :
-                    err = AFPERR_VLOCK;
+            break;
+        case DIRPBIT_UNIXPR :
+            if (vol_unix_priv(vol)) {
+                if (dir->d_did == DIRDID_ROOT) {
+                    if (!dir_rx_set(upriv)) {
+                        /* we can't remove read and search for owner on volume root */
+                        err = AFPERR_ACCESS;
+                        goto setdirparam_done;
+                    }
+                    setdeskowner( -1, ntohl(group) );
+                    setdeskmode( upriv );
+                }
+                if ( setdirowner(vol, upath, -1, ntohl(group) ) < 0 ) {
+                    err = set_dir_errors(path, "setdirowner", errno);
                     goto setdirparam_done;
-                default :
-                    LOG(log_error, logtype_afpd, "setdirparam: setdirmode: %s",
-                        strerror(errno) );
-                    err = AFPERR_PARAM;
+                }
+
+                if ( upriv_bit && setdirunixmode(vol, upath, upriv) < 0 ) {
+                    err = set_dir_errors(path, "setdirunixmode", errno);
                     goto setdirparam_done;
                 }
             }
-            break;
-
-        /* Ignore what the client thinks we should do to the
-           ProDOS information block.  Skip over the data and
-           report nothing amiss. <shirsch@ibm.net> */
-        case DIRPBIT_PDINFO :
-            if (afp_version < 30) {
-                buf += 6;
-                break;
+            else {
+                err = AFPERR_BITMAP;
+                goto setdirparam_done;
             }
+            break;
         default :
             err = AFPERR_BITMAP;
             goto setdirparam_done;
@@ -1600,52 +2246,148 @@ int setdirparams(const struct vol *vol,
 
 setdirparam_done:
     if (change_mdate && newdate == 0 && gettimeofday(&tv, NULL) == 0) {
-       newdate = AD_DATE_FROM_UNIX(tv.tv_sec);
+        newdate = AD_DATE_FROM_UNIX(tv.tv_sec);
     }
     if (newdate) {
-       if (isad)
-          ad_setdate(&ad, AD_DATE_MODIFY, newdate);
-       ut.actime = ut.modtime = AD_DATE_TO_UNIX(newdate);
-       utime(upath, &ut);
+        if (isad)
+            ad_setdate(&ad, AD_DATE_MODIFY, newdate);
+        ut.actime = ut.modtime = AD_DATE_TO_UNIX(newdate);
+        utime(upath, &ut);
     }
 
     if ( isad ) {
-        ad_flush( &ad, ADFLAGS_HF );
-        ad_close( &ad, ADFLAGS_HF );
-    }
+        if (path->st_valid && !path->st_errno) {
+            struct stat *st = &path->st;
 
-    if (change_parent_mdate && curdir->d_did != DIRDID_ROOT
-            && gettimeofday(&tv, NULL) == 0) {
-       if (!movecwd(vol, curdir->d_parent)) {
-           newdate = AD_DATE_FROM_UNIX(tv.tv_sec);
-           bitmap = 1<<DIRPBIT_MDATE;
-           setdirparams(vol, &Cur_Path, bitmap, (char *)&newdate);
-           /* should we reset curdir ?*/
-       }
+            if (dir && dir->d_parent) {
+                ad_setid(&ad, st->st_dev, st->st_ino,  dir->d_did, dir->d_parent->d_did, vol->v_stamp);
+            }
+        }
+        ad_flush( &ad);
+        ad_close_metadata( &ad);
+    }
+
+    if (change_parent_mdate && dir->d_did != DIRDID_ROOT
+        && gettimeofday(&tv, NULL) == 0) {
+        if (!movecwd(vol, dir->d_parent)) {
+            newdate = AD_DATE_FROM_UNIX(tv.tv_sec);
+            /* be careful with bitmap because now dir is null */
+            bitmap = 1<<DIRPBIT_MDATE;
+            setdirparams(vol, &Cur_Path, bitmap, (char *)&newdate);
+            /* should we reset curdir ?*/
+        }
     }
 
     return err;
 }
 
-int afp_createdir(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int afp_syncdir(AFPObj *obj _U_, char *ibuf, size_t ibuflen _U_, char *rbuf _U_, size_t *rbuflen)
+{
+#ifdef HAVE_DIRFD
+    DIR                  *dp;
+#endif
+    int                  dfd;
+    struct vol           *vol;
+    struct dir           *dir;
+    u_int32_t            did;
+    u_int16_t            vid;
+
+    *rbuflen = 0;
+    ibuf += 2;
+
+    memcpy( &vid, ibuf, sizeof( vid ));
+    ibuf += sizeof( vid );
+    if (NULL == (vol = getvolbyvid( vid )) ) {
+        return( AFPERR_PARAM );
+    }
+
+    memcpy( &did, ibuf, sizeof( did ));
+    ibuf += sizeof( did );
+
+    /*
+     * Here's the deal:
+     * if it's CNID 2 our only choice to meet the specs is call sync.
+     * For any other CNID just sync that dir. To my knowledge the
+     * intended use of FPSyncDir is to sync the volume so all we're
+     * ever going to see here is probably CNID 2. Anyway, we' prepared.
+     */
+
+    if ( ntohl(did) == 2 ) {
+        sync();
+    } else {
+        if (NULL == ( dir = dirlookup( vol, did )) ) {
+            return afp_errno; /* was AFPERR_NOOBJ */
+        }
+
+        if (movecwd( vol, dir ) < 0 )
+            return ( AFPERR_NOOBJ );
+
+        /*
+         * Assuming only OSens that have dirfd also may require fsyncing directories
+         * in order to flush metadata e.g. Linux.
+         */
+
+#ifdef HAVE_DIRFD
+        if (NULL == ( dp = opendir( "." )) ) {
+            switch( errno ) {
+            case ENOENT :
+                return( AFPERR_NOOBJ );
+            case EACCES :
+                return( AFPERR_ACCESS );
+            default :
+                return( AFPERR_PARAM );
+            }
+        }
+
+        LOG(log_debug, logtype_afpd, "afp_syncdir: dir: '%s'", dir->d_u_name);
+
+        dfd = dirfd( dp );
+        if ( fsync ( dfd ) < 0 )
+            LOG(log_error, logtype_afpd, "afp_syncdir(%s):  %s",
+                dir->d_u_name, strerror(errno) );
+        closedir(dp); /* closes dfd too */
+#endif
+
+        if ( -1 == (dfd = open(vol->ad_path(".", ADFLAGS_DIR), O_RDWR))) {
+            switch( errno ) {
+            case ENOENT:
+                return( AFPERR_NOOBJ );
+            case EACCES:
+                return( AFPERR_ACCESS );
+            default:
+                return( AFPERR_PARAM );
+            }
+        }
+
+        LOG(log_debug, logtype_afpd, "afp_syncdir: ad-file: '%s'",
+            vol->ad_path(".", ADFLAGS_DIR) );
+
+        if ( fsync(dfd) < 0 )
+            LOG(log_error, logtype_afpd, "afp_syncdir(%s): %s",
+                vol->ad_path(dir->d_u_name, ADFLAGS_DIR), strerror(errno) );
+        close(dfd);
+    }
+
+    return ( AFP_OK );
+}
+
+int afp_createdir(AFPObj *obj, char *ibuf, size_t ibuflen _U_, char *rbuf, size_t *rbuflen)
 {
-    struct adouble     ad;
-    struct vol         *vol;
-    struct dir         *dir;
-    char               *upath;
+    struct adouble  ad;
+    struct vol      *vol;
+    struct dir      *dir;
+    char        *upath;
     struct path         *s_path;
-    u_int32_t          did;
-    u_int16_t          vid;
+    u_int32_t       did;
+    u_int16_t       vid;
+    int                 err;
 
     *rbuflen = 0;
     ibuf += 2;
 
     memcpy( &vid, ibuf, sizeof( vid ));
     ibuf += sizeof( vid );
-    if (( vol = getvolbyvid( vid )) == NULL ) {
+    if (NULL == ( vol = getvolbyvid( vid )) ) {
         return( AFPERR_PARAM );
     }
 
@@ -1654,41 +2396,25 @@ int             ibuflen, *rbuflen;
 
     memcpy( &did, ibuf, sizeof( did ));
     ibuf += sizeof( did );
-    if (( dir = dirlookup( vol, did )) == NULL ) {
-        return( AFPERR_NOOBJ );
+    if (NULL == ( dir = dirlookup( vol, did )) ) {
+        return afp_errno; /* was AFPERR_NOOBJ */
     }
+    /* for concurrent access we need to be sure we are not in the
+     * folder we want to create...
+     */
+    movecwd(vol, dir);
 
-    if (( s_path = cname( vol, dir, &ibuf )) == NULL ) {
-        return afp_errno;
+    if (NULL == ( s_path = cname( vol, dir, &ibuf )) ) {
+        return get_afp_errno(AFPERR_PARAM);
     }
-    /* FIXME check done elswhere? cname was able to move curdir to it! */
+    /* cname was able to move curdir to it! */
     if (*s_path->m_name == '\0')
         return AFPERR_EXIST;
 
     upath = s_path->u_name;
-    {
-    int ret;
-        if (0 != (ret = check_name(vol, upath))) {
-            return  ret;
-        }
-    }
 
-    if ( ad_mkdir( upath, DIRBITS | 0777 ) < 0 ) {
-        switch ( errno ) {
-        case ENOENT :
-            return( AFPERR_NOOBJ );
-        case EROFS :
-            return( AFPERR_VLOCK );
-        case EACCES :
-            return( AFPERR_ACCESS );
-        case EEXIST :
-            return( AFPERR_EXIST );
-        case ENOSPC :
-        case EDQUOT :
-            return( AFPERR_DFULL );
-        default :
-            return( AFPERR_PARAM );
-        }
+    if (AFP_OK != (err = netatalk_mkdir( upath))) {
+        return err;
     }
 
     if (of_stat(s_path) < 0) {
@@ -1703,21 +2429,24 @@ int             ibuflen, *rbuflen;
         return( AFPERR_PARAM );
     }
 
-    memset(&ad, 0, sizeof(ad));
-    if (ad_open( ".", vol_noadouble(vol)|ADFLAGS_HF|ADFLAGS_DIR,
-                 O_RDWR|O_CREAT, 0666, &ad ) < 0)  {
+    ad_init(&ad, vol->v_adouble, vol->v_ad_options);
+    if (ad_open_metadata( ".", vol_noadouble(vol)|ADFLAGS_DIR, O_CREAT, &ad ) < 0)  {
         if (vol_noadouble(vol))
             goto createdir_done;
         return( AFPERR_ACCESS );
     }
+    ad_setname(&ad, s_path->m_name);
+    ad_setid( &ad, s_path->st.st_dev, s_path->st.st_ino, dir->d_did, did, vol->v_stamp);
 
-    ad_setentrylen( &ad, ADEID_NAME, strlen( s_path->m_name ));
-    memcpy( ad_entry( &ad, ADEID_NAME ), s_path->m_name,
-            ad_getentrylen( &ad, ADEID_NAME ));
-    ad_flush( &ad, ADFLAGS_HF );
-    ad_close( &ad, ADFLAGS_HF );
+    ad_flush( &ad);
+    ad_close_metadata( &ad);
 
 createdir_done:
+#ifdef HAVE_NFSv4_ACLS
+    /* FIXME: are we really inside the created dir? */
+    addir_inherit_acl(vol);
+#endif
+
     memcpy( rbuf, &dir->d_did, sizeof( u_int32_t ));
     *rbuflen = sizeof( u_int32_t );
     setvoltime(obj, vol );
@@ -1729,17 +2458,17 @@ createdir_done:
  * newname   new mac name
  * newparent curdir
  *
-*/
-int renamedir(src, dst, dir, newparent, newname, noadouble)
-char   *src, *dst, *newname;
-struct dir     *dir, *newparent;
-const int noadouble;
+ */
+int renamedir(const struct vol *vol, char *src, char *dst,
+              struct dir *dir,
+              struct dir *newparent,
+              char *newname)
 {
-    struct adouble     ad;
-    struct dir         *parent;
+    struct adouble  ad;
+    struct dir      *parent;
     char                *buf;
-    int                        len, err;
-        
+    int         len, err;
+
     /* existence check moved to afp_moveandrename */
     if ( unix_rename( src, dst ) < 0 ) {
         switch ( errno ) {
@@ -1755,7 +2484,7 @@ const int noadouble;
         case EXDEV:
             /* this needs to copy and delete. bleah. that means we have
              * to deal with entire directory hierarchies. */
-            if ((err = copydir(src, dst, noadouble)) < 0) {
+            if ((err = copydir(vol, src, dst)) < 0) {
                 deletedir(dst);
                 return err;
             }
@@ -1767,18 +2496,22 @@ const int noadouble;
         }
     }
 
-    memset(&ad, 0, sizeof(ad));
+    vol->vfs->vfs_renamedir(vol, src, dst);
+
     len = strlen( newname );
     /* rename() succeeded so we need to update our tree even if we can't open
-     * .Parent
-    */
-    if ( !ad_open( dst, ADFLAGS_HF|ADFLAGS_DIR, O_RDWR, 0, &ad)) {
-        ad_setentrylen( &ad, ADEID_NAME, len );
-        memcpy( ad_entry( &ad, ADEID_NAME ), newname, len );
-        ad_flush( &ad, ADFLAGS_HF );
-        ad_close( &ad, ADFLAGS_HF );
+     * metadata
+     */
+
+    ad_init(&ad, vol->v_adouble, vol->v_ad_options);
+
+    if (!ad_open_metadata( dst, ADFLAGS_DIR, 0, &ad)) {
+        ad_setname(&ad, newname);
+        ad_flush( &ad);
+        ad_close_metadata( &ad);
     }
-    
+
+    dir_hash_del(vol, dir);
     if (dir->d_m_name == dir->d_u_name)
         dir->d_u_name = NULL;
 
@@ -1791,8 +2524,8 @@ const int noadouble;
     strcpy( dir->d_m_name, newname );
 
     if (newname == dst) {
-       free(dir->d_u_name);
-       dir->d_u_name = dir->d_m_name;
+        free(dir->d_u_name);
+        dir->d_u_name = dir->d_m_name;
     }
     else {
         if ((buf = (char *) realloc( dir->d_u_name, strlen(dst) + 1 )) == NULL ) {
@@ -1803,47 +2536,48 @@ const int noadouble;
         strcpy( dir->d_u_name, dst );
     }
 
+    if (dir->d_m_name_ucs2)
+        free(dir->d_m_name_ucs2);
+
+    dir->d_m_name_ucs2 = NULL;
+    if ((size_t)-1 == convert_string_allocate((utf8_encoding())?CH_UTF8_MAC:vol->v_maccharset, CH_UCS2, dir->d_m_name, -1, (char**)&dir->d_m_name_ucs2))
+        dir->d_m_name_ucs2 = NULL;
+
     if (( parent = dir->d_parent ) == NULL ) {
         return( AFP_OK );
     }
     if ( parent == newparent ) {
+        hash_alloc_insert(vol->v_hash, dir, dir);
         return( AFP_OK );
     }
 
     /* detach from old parent and add to new one. */
     dirchildremove(parent, dir);
     dir->d_parent = newparent;
-    dirchildadd(newparent, dir);
+    dirchildadd(vol, newparent, dir);
     return( AFP_OK );
 }
 
-#define DOT_APPLEDOUBLE_LEN 13
 /* delete an empty directory */
-int deletecurdir( vol, path, pathlen )
-const struct vol       *vol;
-char *path;
-int pathlen;
+int deletecurdir(struct vol *vol)
 {
     struct dirent *de;
     struct stat st;
-    struct dir *fdir;
+    struct dir  *fdir;
     DIR *dp;
-    struct adouble     ad;
-    u_int16_t          ashort;
+    struct adouble  ad;
+    u_int16_t       ashort;
+    int err;
 
     if ( curdir->d_parent == NULL ) {
         return( AFPERR_ACCESS );
     }
 
-    if ( curdir->d_child != NULL ) {
-        return( AFPERR_DIRNEMPT );
-    }
-
     fdir = curdir;
 
-    memset(&ad, 0, sizeof(ad));
-    if ( ad_open( ".", ADFLAGS_HF|ADFLAGS_DIR, O_RDONLY,
-                  DIRBITS | 0777, &ad) == 0 ) {
+    ad_init(&ad, vol->v_adouble, vol->v_ad_options);
+    /* we never want to create a resource fork here, we are going to delete it */
+    if ( ad_metadata( ".", ADFLAGS_NOADOUBLE | ADFLAGS_DIR, &ad) == 0 ) {
 
         ad_getattr(&ad, &ashort);
         ad_close( &ad, ADFLAGS_HF );
@@ -1851,56 +2585,9 @@ int pathlen;
             return  AFPERR_OLOCK;
         }
     }
-
-    /* delete stray .AppleDouble files. this happens to get .Parent files
-       as well. */
-    if ((dp = opendir(".AppleDouble"))) {
-        strcpy(path, ".AppleDouble/");
-        while ((de = readdir(dp))) {
-            /* skip this and previous directory */
-            if (!strcmp(de->d_name, ".") || !strcmp(de->d_name, ".."))
-                continue;
-
-            /* bail if the file exists in the current directory.
-             * note: this will not fail with dangling symlinks */
-            if (stat(de->d_name, &st) == 0) {
-                closedir(dp);
-                return AFPERR_DIRNEMPT;
-            }
-
-            strcpy(path + DOT_APPLEDOUBLE_LEN, de->d_name);
-            if (unlink(path) < 0) {
-                closedir(dp);
-                switch (errno) {
-                case EPERM:
-                case EACCES :
-                    return( AFPERR_ACCESS );
-                case EROFS:
-                    return AFPERR_VLOCK;
-                case ENOENT :
-                    continue;
-                default :
-                    return( AFPERR_PARAM );
-                }
-            }
-        }
-        closedir(dp);
-    }
-
-    if ( rmdir( ".AppleDouble" ) < 0 ) {
-        switch ( errno ) {
-        case ENOENT :
-            break;
-        case ENOTEMPTY :
-            return( AFPERR_DIRNEMPT );
-        case EROFS:
-            return AFPERR_VLOCK;
-        case EPERM:
-        case EACCES :
-            return( AFPERR_ACCESS );
-        default :
-            return( AFPERR_PARAM );
-        }
+    err = vol->vfs->vfs_deletecurdir(vol);
+    if (err) {
+        return err;
     }
 
     /* now get rid of dangling symlinks */
@@ -1912,111 +2599,133 @@ int pathlen;
 
             /* bail if it's not a symlink */
             if ((lstat(de->d_name, &st) == 0) && !S_ISLNK(st.st_mode)) {
+                closedir(dp);
                 return AFPERR_DIRNEMPT;
             }
 
-            if (unlink(de->d_name) < 0) {
-                switch (errno) {
-                case EPERM:
-                case EACCES :
-                    return( AFPERR_ACCESS );
-                case EROFS:
-                    return AFPERR_VLOCK;
-                case ENOENT :
-                    continue;
-                default :
-                    return( AFPERR_PARAM );
-                }
+            if ((err = netatalk_unlink(de->d_name))) {
+                closedir(dp);
+                return err;
             }
         }
-        closedir(dp);
     }
 
     if ( movecwd( vol, curdir->d_parent ) < 0 ) {
-        return afp_errno;
+        err = afp_errno;
+        goto delete_done;
     }
 
-    if ( rmdir(fdir->d_u_name) < 0 ) {
-        switch ( errno ) {
-        case ENOENT :
-            return( AFPERR_NOOBJ );
-        case ENOTEMPTY :
-            return( AFPERR_DIRNEMPT );
-        case EPERM:
-        case EACCES :
-            return( AFPERR_ACCESS );
-        case EROFS:
-            return AFPERR_VLOCK;
-        default :
-            return( AFPERR_PARAM );
-        }
+    if ( !(err = netatalk_rmdir(fdir->d_u_name))) {
+        dirchildremove(curdir, fdir);
+        cnid_delete(vol->v_cdb, fdir->d_did);
+        dir_remove( vol, fdir );
+        err = AFP_OK;
     }
-
-    dirchildremove(curdir, fdir);
-#ifdef CNID_DB
-    cnid_delete(vol->v_db, fdir->d_did);
-#endif /* CNID_DB */
-    dir_remove( vol, fdir );
-
-    return( AFP_OK );
+delete_done:
+    if (dp) {
+        /* inode is used as key for cnid.
+         * Close the descriptor only after cnid_delete
+         * has been called.
+         */
+        closedir(dp);
+    }
+    return err;
 }
 
-int afp_mapid(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int afp_mapid(AFPObj *obj, char *ibuf, size_t ibuflen _U_, char *rbuf, size_t *rbuflen)
 {
-    struct passwd      *pw;
-    struct group       *gr;
-    char               *name;
+    struct passwd   *pw;
+    struct group    *gr;
+    char        *name;
     u_int32_t           id;
-    int                        len, sfunc;
+    int         len, sfunc;
     int         utf8 = 0;
-    
+
     ibuf++;
     sfunc = (unsigned char) *ibuf++;
-    memcpy( &id, ibuf, sizeof( id ));
-
-    id = ntohl(id);
     *rbuflen = 0;
 
-    if ( id != 0 ) {
-        switch ( sfunc ) {
-        case 1 :
-        case 3 :/* unicode */
+
+    if (sfunc >= 3 && sfunc <= 6) {
+        if (afp_version < 30) {
+            return( AFPERR_PARAM );
+        }
+        utf8 = 1;
+    }
+
+    switch ( sfunc ) {
+    case 1 :
+    case 3 :/* unicode */
+        memcpy( &id, ibuf, sizeof( id ));
+        id = ntohl(id);
+        if ( id != 0 ) {
             if (( pw = getpwuid( id )) == NULL ) {
                 return( AFPERR_NOITEM );
             }
-            name = pw->pw_name;
-            break;
-
-        case 2 :
-        case 4 : /* unicode */
-            if (( gr = (struct group *)getgrgid( id )) == NULL ) {
+            len = convert_string_allocate( obj->options.unixcharset, ((!utf8)?obj->options.maccharset:CH_UTF8_MAC),
+                                           pw->pw_name, -1, &name);
+        } else {
+            len = 0;
+            name = NULL;
+        }
+        break;
+    case 2 :
+    case 4 : /* unicode */
+        memcpy( &id, ibuf, sizeof( id ));
+        id = ntohl(id);
+        if ( id != 0 ) {
+            if (NULL == ( gr = (struct group *)getgrgid( id ))) {
                 return( AFPERR_NOITEM );
             }
-            name = gr->gr_name;
-            break;
-
-        default :
-            return( AFPERR_PARAM );
+            len = convert_string_allocate( obj->options.unixcharset, (!utf8)?obj->options.maccharset:CH_UTF8_MAC,
+                                           gr->gr_name, -1, &name);
+        } else {
+            len = 0;
+            name = NULL;
         }
-        switch ( sfunc ) {
-        case 3:
-        case 4:
-            if (afp_version < 30) {
-                return( AFPERR_PARAM );
-            }
-            utf8 = 1;
-            /* map to unicode */
-            break;            
+        break;
+#ifdef HAVE_NFSv4_ACLS
+    case 5 : /* UUID -> username */
+    case 6 : /* UUID -> groupname */
+        if ((afp_version < 32) || !(obj->options.flags & OPTION_UUID ))
+            return AFPERR_PARAM;
+        LOG(log_debug, logtype_afpd, "afp_mapid: valid UUID request");
+        uuidtype_t type;
+        len = getnamefromuuid( ibuf, &name, &type);
+        if (len != 0)       /* its a error code, not len */
+            return AFPERR_NOITEM;
+        if (type == UUID_USER) {
+            if (( pw = getpwnam( name )) == NULL )
+                return( AFPERR_NOITEM );
+            LOG(log_debug, logtype_afpd, "afp_mapid: name:%s -> uid:%d", name, pw->pw_uid);
+            id = htonl(UUID_USER);
+            memcpy( rbuf, &id, sizeof( id ));
+            id = htonl( pw->pw_uid);
+            rbuf += sizeof( id );
+            memcpy( rbuf, &id, sizeof( id ));
+            rbuf += sizeof( id );
+            *rbuflen = 2 * sizeof( id );
+        } else {        /* type == UUID_GROUP */
+            if (( gr = getgrnam( name )) == NULL )
+                return( AFPERR_NOITEM );
+            LOG(log_debug, logtype_afpd, "afp_mapid: group:%s -> gid:%d", name, gr->gr_gid);
+            id = htonl(UUID_GROUP);
+            memcpy( rbuf, &id, sizeof( id ));
+            rbuf += sizeof( id );
+            id = htonl( gr->gr_gid);
+            memcpy( rbuf, &id, sizeof( id ));
+            rbuf += sizeof( id );
+            *rbuflen = 2 * sizeof( id );
         }
+        break;
+#endif
+    default :
+        return( AFPERR_PARAM );
+    }
+
+    if (name)
         len = strlen( name );
 
-    } else {
-        len = 0;
-        name = NULL;
-    }
     if (utf8) {
         u_int16_t tp = htons(len);
         memcpy(rbuf, &tp, sizeof(tp));
@@ -2031,76 +2740,104 @@ int            ibuflen, *rbuflen;
         memcpy( rbuf, name, len );
     }
     *rbuflen += len;
+    if (name)
+        free(name);
     return( AFP_OK );
 }
 
-int afp_mapname(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int afp_mapname(AFPObj *obj _U_, char *ibuf, size_t ibuflen _U_, char *rbuf, size_t *rbuflen)
 {
-    struct passwd      *pw;
-    struct group       *gr;
+    struct passwd   *pw;
+    struct group    *gr;
     int             len, sfunc;
     u_int32_t       id;
     u_int16_t       ulen;
 
     ibuf++;
     sfunc = (unsigned char) *ibuf++;
+    *rbuflen = 0;
+    LOG(log_debug, logtype_afpd, "afp_mapname: sfunc: %d, afp_version: %d", sfunc, afp_version);
     switch ( sfunc ) {
-    case 1 : 
+    case 1 :
     case 2 : /* unicode */
+        if (afp_version < 30) {
+            return( AFPERR_PARAM );
+        }
         memcpy(&ulen, ibuf, sizeof(ulen));
         len = ntohs(ulen);
         ibuf += 2;
+        LOG(log_debug, logtype_afpd, "afp_mapname: alive");
         break;
     case 3 :
     case 4 :
         len = (unsigned char) *ibuf++;
         break;
+#ifdef HAVE_NFSv4_ACLS
+    case 5 : /* username -> UUID  */
+    case 6 : /* groupname -> UUID */
+        if ((afp_version < 32) || !(obj->options.flags & OPTION_UUID ))
+            return AFPERR_PARAM;
+        memcpy(&ulen, ibuf, sizeof(ulen));
+        len = ntohs(ulen);
+        ibuf += 2;
+        break;
+#endif
     default :
-        *rbuflen = 0;
         return( AFPERR_PARAM );
     }
 
     ibuf[ len ] = '\0';
 
-    if ( len != 0 ) {
+    if ( len == 0 )
+        return AFPERR_PARAM;
+    else {
         switch ( sfunc ) {
         case 1 : /* unicode */
         case 3 :
-            if (( pw = (struct passwd *)getpwnam( ibuf )) == NULL ) {
-                *rbuflen = 0;
+            if (NULL == ( pw = (struct passwd *)getpwnam( ibuf )) ) {
                 return( AFPERR_NOITEM );
             }
             id = pw->pw_uid;
+            id = htonl(id);
+            memcpy( rbuf, &id, sizeof( id ));
+            *rbuflen = sizeof( id );
             break;
 
         case 2 : /* unicode */
         case 4 :
-            if (( gr = (struct group *)getgrnam( ibuf )) == NULL ) {
-                *rbuflen = 0;
+            LOG(log_debug, logtype_afpd, "afp_mapname: gettgrnam for name: %s",ibuf);
+            if (NULL == ( gr = (struct group *)getgrnam( ibuf ))) {
                 return( AFPERR_NOITEM );
             }
             id = gr->gr_gid;
+            LOG(log_debug, logtype_afpd, "afp_mapname: gettgrnam for name: %s -> id: %d",ibuf, id);
+            id = htonl(id);
+            memcpy( rbuf, &id, sizeof( id ));
+            *rbuflen = sizeof( id );
+            break;
+#ifdef HAVE_NFSv4_ACLS
+        case 5 :        /* username -> UUID */
+            LOG(log_debug, logtype_afpd, "afp_mapname: name: %s",ibuf);
+            if (0 != getuuidfromname(ibuf, UUID_USER, rbuf))
+                return AFPERR_NOITEM;
+            *rbuflen = UUID_BINSIZE;
             break;
+        case 6 :        /* groupname -> UUID */
+            LOG(log_debug, logtype_afpd, "afp_mapname: name: %s",ibuf);
+            if (0 != getuuidfromname(ibuf, UUID_GROUP, rbuf))
+                return AFPERR_NOITEM;
+            *rbuflen = UUID_BINSIZE;
+            break;
+#endif
         }
-    } else {
-        id = 0;
     }
-    id = htonl(id);
-    memcpy( rbuf, &id, sizeof( id ));
-    *rbuflen = sizeof( id );
     return( AFP_OK );
 }
 
 /* ------------------------------------
-  variable DID support 
+   variable DID support
 */
-int afp_closedir(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int afp_closedir(AFPObj *obj _U_, char *ibuf _U_, size_t ibuflen _U_, char *rbuf _U_, size_t *rbuflen)
 {
 #if 0
     struct vol   *vol;
@@ -2133,19 +2870,16 @@ int             ibuflen, *rbuflen;
     return AFP_OK;
 }
 
-/* did creation gets done automatically 
+/* did creation gets done automatically
  * there's a pb again with case but move it to cname
-*/
-int afp_opendir(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+ */
+int afp_opendir(AFPObj *obj _U_, char *ibuf, size_t ibuflen  _U_, char *rbuf, size_t *rbuflen)
 {
-    struct vol         *vol;
-    struct dir         *parentdir;
-    struct path                *path;
-    u_int32_t          did;
-    u_int16_t          vid;
+    struct vol      *vol;
+    struct dir      *parentdir;
+    struct path     *path;
+    u_int32_t       did;
+    u_int16_t       vid;
 
     *rbuflen = 0;
     ibuf += 2;
@@ -2153,23 +2887,23 @@ int             ibuflen, *rbuflen;
     memcpy(&vid, ibuf, sizeof(vid));
     ibuf += sizeof( vid );
 
-    if (( vol = getvolbyvid( vid )) == NULL ) {
+    if (NULL == ( vol = getvolbyvid( vid )) ) {
         return( AFPERR_PARAM );
     }
 
     memcpy(&did, ibuf, sizeof(did));
     ibuf += sizeof(did);
 
-    if (( parentdir = dirlookup( vol, did )) == NULL ) {
+    if (NULL == ( parentdir = dirlookup( vol, did )) ) {
         return afp_errno;
     }
 
-    if (( path = cname( vol, parentdir, &ibuf )) == NULL ) {
-        return afp_errno;
+    if (NULL == ( path = cname( vol, parentdir, &ibuf )) ) {
+        return get_afp_errno(AFPERR_PARAM);
     }
 
     if ( *path->m_name != '\0' ) {
-        return( AFPERR_BADTYPE ); /* not a directory */
+        return path_error(path, AFPERR_NOOBJ);
     }
 
     if ( !path->st_valid && of_stat(path ) < 0 ) {