]> arthur.barton.de Git - netatalk.git/blobdiff - etc/afpd/directory.c
Time Machine support: new volume option tm
[netatalk.git] / etc / afpd / directory.c
index 6d3fe1bd47493497833539d9f449014114cc6cc5..0a5f0893fe71f5934556f2fb94188d99e066a502 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: directory.c,v 1.67 2003-04-15 07:18:45 didg Exp $
+ * $Id: directory.c,v 1.71.2.4.2.15.2.14 2009-09-07 11:35:04 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,24 @@ 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 <dirent.h>
+
+#include <grp.h>
+#include <pwd.h>
+#include <sys/param.h>
+#include <errno.h>
+#include <utime.h>
+#include <atalk/adouble.h>
+
+#include <atalk/afp.h>
+#include <atalk/util.h>
+#include <atalk/cnid.h>
+#include <atalk/logger.h>
 
 #include "directory.h"
 #include "desktop.h"
@@ -58,6 +53,7 @@ char *strchr (), *strrchr ();
 #include "filedir.h"
 #include "globals.h"
 #include "unix.h"
+#include "mangle.h"
 
 struct dir     *curdir;
 int             afp_errno;
@@ -125,7 +121,7 @@ u_int32_t   did;
 #ifdef ATACC
 int path_isadir(struct path *o_path)
 {
-    return o_path->dir != NULL;
+    return o_path->d_dir != NULL;
 #if 0
     return o_path->m_name == '\0' || /* we are in a it */
            !o_path->st_valid ||      /* in cache but we can't chdir in it */ 
@@ -156,7 +152,7 @@ struct dir *dir;
         if ( strcmp( dir->d_u_name, name ) == 0 ) {
             break;
         }
-        dir = (dir == curdir->d_child->d_prev) ? NULL : dir->d_next;
+        dir = (dir == cdir->d_child->d_prev) ? NULL : dir->d_next;
     }
     return dir;
 }            
@@ -164,16 +160,18 @@ struct dir *dir;
 /* -----------------------------------------
  * 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;
 {
-#ifdef CNID_DB
     struct dir   *ret;
     char        *upath;
-    u_int32_t   id;
+    cnid_t      id, cnid;
     static char  path[MAXPATHLEN + 1];
     size_t len,  pathlen;
     char         *ptr;
@@ -190,12 +188,12 @@ u_int32_t did;
     utf8 = utf8_encoding();
     maxpath = (utf8)?MAXPATHLEN -7:255;
     id = did;
-    if (NULL == (upath = cnid_resolve(vol->v_db, &id, buffer, buflen)) ) {
+    if (NULL == (upath = cnid_resolve(vol->v_cdb, &id, buffer, buflen)) ) {
         afp_errno = AFPERR_NOOBJ;
         return NULL;
     }
     ptr = path + MAXPATHLEN;
-    if (NULL == ( mpath = utompath(vol, upath, utf8) ) ) {
+    if (NULL == ( mpath = utompath(vol, upath, did, utf8) ) ) {
         afp_errno = AFPERR_NOOBJ;
         return NULL;
     }
@@ -209,9 +207,10 @@ u_int32_t  did;
         if (ret != NULL) {
             break;
         }
-        if ( NULL == (upath = cnid_resolve(vol->v_db, &id, buffer, buflen))
+        cnid = id;
+        if ( NULL == (upath = cnid_resolve(vol->v_cdb, &id, buffer, buflen))
              ||
-             NULL == (mpath = utompath(vol, upath, utf8))
+             NULL == (mpath = utompath(vol, upath, cnid, utf8))
         ) {
             afp_errno = AFPERR_NOOBJ;
             return NULL;
@@ -251,7 +250,7 @@ u_int32_t   did;
     /* cname is not efficient */
     if (cname( vol, ret, &ptr ) == NULL )
         return NULL;
-#endif
+       
     return dirsearch(vol, did);
 }
 
@@ -393,9 +392,7 @@ struct dir *dir;
 /* 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( const struct vol *vol _U_, struct dir  *dir)
 {
 #ifdef REMOVE_NODES
     struct ofork *of, *last;
@@ -407,10 +404,7 @@ struct dir *dir;
 
     /* 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);
+    dirfreename(dir);
     dir->d_m_name = NULL;
     dir->d_u_name = NULL;
 #else /* ! REMOVE_NODES */
@@ -564,13 +558,26 @@ struct vol        *vol;
 struct dir     *dir;
 struct path *path;
 {
-    path->u_name = mtoupath(vol, path->m_name, utf8_encoding() );
-    path->dir = NULL;
+    char *save_m_name;
+
+    if ( path->u_name == NULL) {
+        path->u_name = mtoupath(vol, path->m_name, dir->d_did, (path->m_type==3) );
+    }
+    path->d_dir = 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 ) {
         return( NULL );
     }
@@ -579,11 +586,26 @@ struct path *path;
         return( NULL );
     }
 
-    if (( dir = adddir( vol, dir, path)) == NULL ) {
-        return( NULL );
+    /* FIXME: if this is an AFP3 connection and path->m_type != 3 we might screw dircache here */
+    if ( utf8_encoding() && path->m_type != 3)
+    {
+       save_m_name = path->m_name;
+       path->m_name = NULL;
+        if (( dir = adddir( vol, dir, path)) == NULL ) {
+           free(save_m_name);
+            return( NULL );
+        }
+       path->m_name = save_m_name;
+        LOG(log_debug, logtype_afpd, "AFP3 connection, mismatch in mtype: %u, unix: %s, mac: %s",
+            path->m_type, path->u_name, path->m_name);
+    }
+    else { 
+        if (( dir = adddir( vol, dir, path)) == NULL ) {
+            return( NULL );
+        }
     }
 
-    path->dir = dir;
+    path->d_dir = dir;
     if ( movecwd( vol, dir ) < 0 ) {
         return( NULL );
     }
@@ -715,7 +737,7 @@ static int deletedir(char *dir)
 }
 
 /* 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;
@@ -724,7 +746,6 @@ static int copydir(char *src, char *dst, int noadouble)
     struct utimbuf      ut;
     size_t slen, dlen;
     size_t srem, drem;
-    
     int err;
 
     /* doesn't exist or the path is too long. */
@@ -770,9 +791,9 @@ static int copydir(char *src, char *dst, int noadouble)
             strcpy(dpath + dlen, de->d_name);
 
             if (S_ISDIR(st.st_mode)) {
-                if (AFP_OK != (err = copydir(spath, dpath, noadouble)))
+                if (AFP_OK != (err = copydir(vol, spath, dpath)))
                     goto copydir_done;
-            } else if (AFP_OK != (err = copyfile(spath, dpath, NULL, noadouble))) {
+            } else if (AFP_OK != (err = copyfile(vol, vol, spath, dpath, NULL, NULL))) {
                 goto copydir_done;
 
             } else {
@@ -965,6 +986,7 @@ char        **cpath;
     u_int16_t   len16;
     int         size = 0;
     char        sep;
+    int         toUTF8 = 0;
                
     data = *cpath;
     afp_errno = AFPERR_NOOBJ;
@@ -974,6 +996,10 @@ char       **cpath;
        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) {
@@ -993,14 +1019,13 @@ char     **cpath;
     default:
         afp_errno = AFPERR_PARAM;
         return( NULL );
-    
     }
     *cpath += len + size;
     *path = '\0';
     ret.m_name = path;
     ret.st_errno = 0;
     ret.st_valid = 0;
-    ret.dir = NULL;
+    ret.d_dir = NULL;
     for ( ;; ) {
         if ( len == 0 ) {
             if (movecwd( vol, dir ) < 0 ) {
@@ -1015,9 +1040,13 @@ char     **cpath;
                     if ( movecwd( vol, dir->d_parent ) < 0 ) {
                         return NULL;                   
                    }
+                   /* FIXME should we set, 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.dir = dir;
+                   ret.d_dir = dir;
                    return &ret;
                } else if (afp_errno == AFPERR_NOOBJ) {
                     if ( movecwd( vol, dir->d_parent ) < 0 ) {
@@ -1034,6 +1063,11 @@ char     **cpath;
                        ret.u_name =  path +tp;
                        
                    }
+                   /* FIXME should we set :
+                     ret.st_valid = 1;
+                     ret.st_errno = ENOENT;
+                   */
+                   
                    dir_invalidate(vol, dir);
                    return &ret;
                }
@@ -1042,10 +1076,10 @@ char    **cpath;
             }
             if (*path == '\0') {
                ret.u_name = ".";
-               ret.dir = dir;
+               ret.d_dir = dir;
             }               
             return &ret;
-        }
+        } /* if (len == 0) */
 
         if (*data == sep ) {
             data++;
@@ -1064,6 +1098,10 @@ char     **cpath;
         p = path;
         while ( *data != sep && len > 0 ) {
             *p++ = *data++;
+            if (p > &path[ MAXPATHLEN]) {
+                afp_errno = AFPERR_PARAM;
+                return( NULL );
+            }
             len--;
         }
 
@@ -1076,6 +1114,35 @@ char     **cpath;
         *p = '\0';
 
         if ( p != path ) { /* we got something */
+            ret.u_name = NULL;
+            if (afp_version >= 30) {
+                char *t;
+                cnid_t fileid;
+                
+                if (toUTF8) {
+                    static char        temp[ MAXPATHLEN + 1];
+
+                    /* not an UTF8 name */
+                    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 demagnle_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 ( !extend ) {
                 cdir = dir->d_child;
                 while (cdir) {
@@ -1109,7 +1176,7 @@ char      **cpath;
 
             } else {
                 cdir = extenddir( vol, dir, &ret );
-            }
+            } /* if (!extend) */
 
             if ( cdir == NULL ) {
 
@@ -1121,8 +1188,8 @@ char      **cpath;
                 dir = cdir;    
                 *path = '\0';
             }
-        }
-    }
+        } /* if (p != path) */
+    } /* for (;;) */
 }
 
 /*
@@ -1162,7 +1229,7 @@ struct dir        *dir;
         }
         *--p = '/';
         p -= n;
-        strncpy( p, u, n );
+        memcpy( p, u, n );
     }
     if ( d != curdir ) {
         n = strlen( vol->v_path );
@@ -1172,7 +1239,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) {
@@ -1216,6 +1283,26 @@ 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;
+
+}
+
+/* --------------------- */
+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
@@ -1234,6 +1321,7 @@ int getdirparams(const struct vol *vol,
     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;
     
@@ -1242,13 +1330,21 @@ int getdirparams(const struct vol *vol,
                   (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)) {
+
+        ad_init(&ad, vol->v_adouble, vol->v_ad_options);
+       if ( !ad_metadata( upath, 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 ) {
@@ -1260,8 +1356,7 @@ int getdirparams(const struct vol *vol,
         case DIRPBIT_ATTR :
             if ( isad ) {
                 ad_getattr(&ad, &ashort);
-            } else if (*dir->d_u_name == '.' && strcmp(dir->d_u_name, ".") 
-                        && strcmp(dir->d_u_name, "..")) {
+            } else if (invisible_dots(vol, dir->d_u_name)) {
                 ashort = htons(ATTRBIT_INVISIBLE);
             } else
                 ashort = 0;
@@ -1271,15 +1366,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 :
@@ -1311,12 +1399,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 (*dir->d_u_name  == '.' && strcmp(dir->d_u_name , ".") &&
-                        strcmp(dir->d_u_name , "..")) {
+                /* 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;
@@ -1344,12 +1430,12 @@ int getdirparams(const struct vol *vol,
             ashort = 0;
             /* this needs to handle current directory access rights */
             if (st->st_ctime == dir->ctime) {
-               ashort = dir->offcnt;
+                ashort = (dir->offcnt > 0xffff)?0xffff:dir->offcnt;
             }
             else if ((ret = for_each_dirent(vol, upath, NULL,NULL)) >= 0) {
-                ashort = ret;
-                dir->offcnt = ashort;
+                dir->offcnt = ret;
                 dir->ctime = st->st_ctime;
+                ashort = (dir->offcnt > 0xffff)?0xffff:dir->offcnt;
             }
             ashort = htons( ashort );
             memcpy( data, &ashort, sizeof( ashort ));
@@ -1403,6 +1489,27 @@ 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 );
@@ -1415,12 +1522,12 @@ int getdirparams(const struct vol *vol,
     if ( l_nameoff ) {
         ashort = htons( data - buf );
         memcpy( l_nameoff, &ashort, sizeof( ashort ));
-        data = set_name(vol, 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(vol, 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 );
@@ -1429,11 +1536,25 @@ int getdirparams(const struct vol *vol,
     return( AFP_OK );
 }
 
+/* ----------------------------- */
+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(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+AFPObj  *obj;
+char   *ibuf, *rbuf _U_;
+int    ibuflen _U_, *rbuflen;
 {
     struct vol *vol;
     struct dir *dir;
@@ -1469,9 +1590,11 @@ int              ibuflen, *rbuflen;
         return get_afp_errno(AFPERR_NOOBJ); 
     }
 
-    /* FIXME access error or not a file */
     if ( *path->m_name != '\0' ) {
-        return (path_isadir( path))? afp_errno:AFPERR_BADTYPE ;
+        rc = path_error(path, AFPERR_NOOBJ);
+        /* maybe we are trying to set perms back */
+        if (rc != AFPERR_ACCESS)
+            return rc;
     }
 
     /*
@@ -1497,12 +1620,28 @@ struct path Cur_Path = {
     0,
     "",  /* mac name */
     ".", /* unix name */
+    NULL,
     0,  /* stat is not set */
-    0,  /* */
+    0  /* errno */
 };
 
+/* ------------------ */
+static int set_dir_errors(struct path *path, const char *where, int err)
+{
+    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(const struct vol *vol, 
-                 struct path *path, u_int16_t bitmap, char *buf )
+                 struct path *path, u_int16_t d_bitmap, char *buf )
 {
     struct maccess     ma;
     struct adouble     ad;
@@ -1510,15 +1649,126 @@ int setdirparams(const struct vol *vol,
     struct timeval      tv;
 
     char                *upath;
-    int                        bit = 0, aint, isad = 1;
+    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) {
@@ -1530,8 +1780,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;
         }
@@ -1542,13 +1793,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_get_HF_flags( &ad ) & 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;
@@ -1557,212 +1808,120 @@ 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 );
+                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;
+            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;
-                    goto setdirparam_done;
-                    break;
-                case EROFS :
-                    err = AFPERR_VLOCK;
+                else {
                     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;
+            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;
-            case EROFS :
-                err = AFPERR_VLOCK;
-                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;
@@ -1785,14 +1944,22 @@ setdirparam_done:
     }
 
     if ( isad ) {
+        if (path->st_valid && !path->st_errno) {
+            struct stat *st = &path->st;
+
+            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, ADFLAGS_HF );
         ad_close( &ad, ADFLAGS_HF );
     }
 
-    if (change_parent_mdate && curdir->d_did != DIRDID_ROOT
+    if (change_parent_mdate && dir->d_did != DIRDID_ROOT
             && gettimeofday(&tv, NULL) == 0) {
-       if (!movecwd(vol, curdir->d_parent)) {
+       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 ?*/
@@ -1802,10 +1969,103 @@ setdirparam_done:
     return err;
 }
 
+int afp_syncdir(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj  *obj _U_;
+char    *ibuf, *rbuf _U_;
+int     ibuflen _U_, *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(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
+AFPObj  *obj;
 char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int    ibuflen _U_, *rbuflen;
 {
     struct adouble     ad;
     struct vol         *vol;
@@ -1846,9 +2106,6 @@ int               ibuflen, *rbuflen;
         return AFPERR_EXIST;
 
     upath = s_path->u_name;
-    if (0 != (err = check_name(vol, upath))) {
-       return err;
-    }
 
     if (AFP_OK != (err = netatalk_mkdir( upath))) {
         return err;
@@ -1866,17 +2123,16 @@ int             ibuflen, *rbuflen;
         return( AFPERR_PARAM );
     }
 
-    memset(&ad, 0, sizeof(ad));
+    ad_init(&ad, vol->v_adouble, vol->v_ad_options);
     if (ad_open( ".", vol_noadouble(vol)|ADFLAGS_HF|ADFLAGS_DIR,
                  O_RDWR|O_CREAT, 0666, &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 );
 
@@ -1893,10 +2149,10 @@ createdir_done:
  * newparent curdir
  *
 */
-int renamedir(src, dst, dir, newparent, newname, noadouble)
+int renamedir(vol, src, dst, dir, newparent, newname)
+const struct vol *vol;
 char   *src, *dst, *newname;
 struct dir     *dir, *newparent;
-const int noadouble;
 {
     struct adouble     ad;
     struct dir         *parent;
@@ -1918,7 +2174,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;
             }
@@ -1930,18 +2186,25 @@ const int noadouble;
         }
     }
 
-    memset(&ad, 0, sizeof(ad));
+    if (vol->v_adouble == AD_VERSION2_OSX) {
+        /* We simply move the corresponding ad file as well */
+        char   tempbuf[258]="._";
+        rename(vol->ad_path(src,0),strcat(tempbuf,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_init(&ad, vol->v_adouble, vol->v_ad_options);
+
+    if (!ad_open( dst, ADFLAGS_HF|ADFLAGS_DIR, O_RDWR, 0, &ad)) {
+        ad_setname(&ad, newname);
         ad_flush( &ad, ADFLAGS_HF );
         ad_close( &ad, ADFLAGS_HF );
     }
-    
+
     if (dir->d_m_name == dir->d_u_name)
         dir->d_u_name = NULL;
 
@@ -1982,10 +2245,9 @@ const int noadouble;
 
 #define DOT_APPLEDOUBLE_LEN 13
 /* delete an empty directory */
-int deletecurdir( vol, path, pathlen )
+int deletecurdir( vol, path )
 const struct vol       *vol;
 char *path;
-int pathlen;
 {
     struct dirent *de;
     struct stat st;
@@ -2001,9 +2263,8 @@ int pathlen;
 
     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);
+    if ( ad_metadata( ".", ADFLAGS_DIR, &ad) == 0 ) {
 
         ad_getattr(&ad, &ashort);
         ad_close( &ad, ADFLAGS_HF );
@@ -2012,33 +2273,41 @@ int pathlen;
         }
     }
 
-    /* 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;
-            }
+    if (vol->v_adouble == AD_VERSION2_OSX) {
+       
+        if ((err = netatalk_unlink(vol->ad_path(".",0) )) ) {
+            return err;
+        }
+    }
+    else {
+        /* 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 ((err = netatalk_unlink(path))) {
-                closedir(dp);
-                return err;
+                strcpy(path + DOT_APPLEDOUBLE_LEN, de->d_name);
+                if ((err = netatalk_unlink(path))) {
+                    closedir(dp);
+                    return err;
+                }
             }
+            closedir(dp);
         }
-        closedir(dp);
-    }
 
-    if ( (err = netatalk_rmdir( ".AppleDouble" ))  ) {
-       return err;
+        if ( (err = netatalk_rmdir( ".AppleDouble" ))  ) {
+            return err;
+        }
     }
 
     /* now get rid of dangling symlinks */
@@ -2068,9 +2337,7 @@ int pathlen;
 
     if ( !(err = netatalk_rmdir(fdir->d_u_name))) {
         dirchildremove(curdir, fdir);
-#ifdef CNID_DB
-        cnid_delete(vol->v_db, fdir->d_did);
-#endif /* CNID_DB */
+        cnid_delete(vol->v_cdb, fdir->d_did);
         dir_remove( vol, fdir );
         err = AFP_OK;
     }
@@ -2086,9 +2353,9 @@ delete_done:
 }
 
 int afp_mapid(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
+AFPObj  *obj;
 char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int    ibuflen _U_, *rbuflen;
 {
     struct passwd      *pw;
     struct group       *gr;
@@ -2104,6 +2371,12 @@ int              ibuflen, *rbuflen;
     id = ntohl(id);
     *rbuflen = 0;
 
+    if (sfunc == 3 || sfunc == 4) {
+        if (afp_version < 30) {
+            return( AFPERR_PARAM );
+        }
+        utf8 = 1;
+    }
     if ( id != 0 ) {
         switch ( sfunc ) {
         case 1 :
@@ -2111,30 +2384,22 @@ int             ibuflen, *rbuflen;
             if (( pw = getpwuid( id )) == NULL ) {
                 return( AFPERR_NOITEM );
             }
-            name = pw->pw_name;
+           len = convert_string_allocate( obj->options.unixcharset, ((!utf8)?obj->options.maccharset:CH_UTF8_MAC),
+                                            pw->pw_name, strlen(pw->pw_name), &name);
             break;
 
         case 2 :
         case 4 : /* unicode */
-            if (( gr = (struct group *)getgrgid( id )) == NULL ) {
+            if (NULL == ( gr = (struct group *)getgrgid( id ))) {
                 return( AFPERR_NOITEM );
             }
-            name = gr->gr_name;
+           len = convert_string_allocate( obj->options.unixcharset, (!utf8)?obj->options.maccharset:CH_UTF8_MAC,
+                                            gr->gr_name, strlen(gr->gr_name), &name);
             break;
 
         default :
             return( AFPERR_PARAM );
         }
-        switch ( sfunc ) {
-        case 3:
-        case 4:
-            if (afp_version < 30) {
-                return( AFPERR_PARAM );
-            }
-            utf8 = 1;
-            /* map to unicode */
-            break;            
-        }
         len = strlen( name );
 
     } else {
@@ -2155,13 +2420,15 @@ 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;
+AFPObj  *obj _U_;
 char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int    ibuflen _U_, *rbuflen;
 {
     struct passwd      *pw;
     struct group       *gr;
@@ -2171,9 +2438,13 @@ int              ibuflen, *rbuflen;
 
     ibuf++;
     sfunc = (unsigned char) *ibuf++;
+    *rbuflen = 0;
     switch ( sfunc ) {
     case 1 : 
     case 2 : /* unicode */
+        if (afp_version < 30) {
+            return( AFPERR_PARAM );
+        }
         memcpy(&ulen, ibuf, sizeof(ulen));
         len = ntohs(ulen);
         ibuf += 2;
@@ -2183,7 +2454,6 @@ int               ibuflen, *rbuflen;
         len = (unsigned char) *ibuf++;
         break;
     default :
-        *rbuflen = 0;
         return( AFPERR_PARAM );
     }
 
@@ -2193,8 +2463,7 @@ int               ibuflen, *rbuflen;
         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;
@@ -2202,8 +2471,7 @@ int               ibuflen, *rbuflen;
 
         case 2 : /* unicode */
         case 4 :
-            if (( gr = (struct group *)getgrnam( ibuf )) == NULL ) {
-                *rbuflen = 0;
+            if (NULL == ( gr = (struct group *)getgrnam( ibuf ))) {
                 return( AFPERR_NOITEM );
             }
             id = gr->gr_gid;
@@ -2222,9 +2490,9 @@ int               ibuflen, *rbuflen;
   variable DID support 
 */
 int afp_closedir(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+AFPObj  *obj _U_;
+char   *ibuf _U_, *rbuf _U_;
+int    ibuflen _U_, *rbuflen;
 {
 #if 0
     struct vol   *vol;
@@ -2261,9 +2529,9 @@ int               ibuflen, *rbuflen;
  * there's a pb again with case but move it to cname
 */
 int afp_opendir(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
+AFPObj  *obj _U_;
 char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int    ibuflen  _U_, *rbuflen;
 {
     struct vol         *vol;
     struct dir         *parentdir;
@@ -2293,7 +2561,7 @@ int               ibuflen, *rbuflen;
     }
 
     if ( *path->m_name != '\0' ) {
-        return (path_isadir(path))? afp_errno:AFPERR_BADTYPE ;
+       return path_error(path, AFPERR_NOOBJ);
     }
 
     if ( !path->st_valid && of_stat(path ) < 0 ) {