]> arthur.barton.de Git - netatalk.git/blobdiff - etc/afpd/unix.c
Symlink patch from Anton Starikov
[netatalk.git] / etc / afpd / unix.c
index 72b634e63898c9465b951ee98ccbd9673298e65a..29ec76a15fada1133f1438268810e8016c6705e9 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: unix.c,v 1.43 2003-03-09 19:55:35 didg Exp $
+ * $Id: unix.c,v 1.59.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 <stdio.h>
 #include <stdlib.h>
-#include <unistd.h>
-#include <errno.h>
-#include <sys/types.h>
-#include <sys/param.h>
-#include <sys/stat.h>
-#include <atalk/logger.h>
-#include <netatalk/endian.h>
-#include <dirent.h>
-#include <limits.h>
-#include <atalk/adouble.h>
-#include <atalk/afp.h>
 
 /* STDC check */
-#if STDC_HEADERS
+#ifdef STDC_HEADERS
 #include <string.h>
 #else /* STDC_HEADERS */
+
 #ifndef HAVE_STRCHR
 #define strchr index
 #define strrchr index
 #endif /* HAVE_STRCHR */
 char *strchr (), *strrchr ();
+
 #ifndef HAVE_MEMCPY
 #define memcpy(d,s,n) bcopy ((s), (d), (n))
 #define memmove(d,s,n) bcopy ((s), (d), (n))
 #endif /* ! HAVE_MEMCPY */
 #endif /* STDC_HEADERS */
 
-#ifdef HAVE_FCNTL_H
-#include <fcntl.h>
-#endif /* HAVE_FCNTL_H */
+#include <errno.h>
+#include <limits.h>
+#include <sys/param.h>
+#include <atalk/logger.h>
+#include <atalk/adouble.h>
+#include <atalk/vfs.h>
+#include <atalk/afp.h>
+#include <atalk/util.h>
+#include <atalk/unix.h>
+
 #include "auth.h"
 #include "directory.h"
 #include "volume.h"
 #include "unix.h"
+#include "fork.h"
+
+#ifdef HAVE_NFSv4_ACLS
+extern void acltoownermode(char *path, struct stat *st,uid_t uid, struct maccess *ma);
+#endif
+
 
 /*
  * Get the free space on a partition.
  */
-int ustatfs_getvolspace( vol, bfree, btotal, bsize )
-const struct vol       *vol;
-VolSpace    *bfree, *btotal;
-u_int32_t   *bsize;
+int ustatfs_getvolspace(const struct vol *vol, VolSpace *bfree, VolSpace *btotal, u_int32_t *bsize)
 {
     VolSpace maxVolSpace = (~(VolSpace)0);
 
@@ -100,17 +101,16 @@ u_int32_t   *bsize;
     return( AFP_OK );
 }
 
-static __inline__ int utombits( bits )
-mode_t bits;
+static int utombits(mode_t bits)
 {
     int                mbits;
 
     mbits = 0;
 
-    mbits |= ( bits & ( S_IREAD >> 6 )) ? (AR_UREAD | AR_USEARCH) : 0;
+    mbits |= ( bits & ( S_IREAD >> 6 ))  ? AR_UREAD  : 0;
     mbits |= ( bits & ( S_IWRITE >> 6 )) ? AR_UWRITE : 0;
-    /* Do we really need this?
-        mbits |= ( bits & ( S_IEXEC >> 6) ) ? AR_USEARCH : 0; */
+    /* Do we really need this? */
+    mbits |= ( bits & ( S_IEXEC >> 6) ) ? AR_USEARCH : 0;
 
     return( mbits );
 }
@@ -118,9 +118,7 @@ mode_t      bits;
 /* --------------------------------
     cf AFP 3.0 page 63
 */
-void utommode( stat, ma )
-struct stat            *stat;
-struct maccess *ma;
+void utommode(struct stat *stat, struct maccess *ma)
 {
 mode_t mode;
 
@@ -151,45 +149,47 @@ mode_t mode;
      * the "owner" bit set, even tho you can do these things on unix wiht
      * only write permission.  What were the things?
      * 
-     * FIXME and so what ?
+     * FIXME 
+     * ditto seems to care if st_uid is 0 ?
+     * was ma->ma_user & AR_UWRITE
+     * but 0 as owner is a can of worms.
      */
-#if 0
-    if ( ma->ma_user & AR_UWRITE ) {
+    if ( !stat->st_uid ) {
         ma->ma_user |= AR_UOWN;
     }
-#endif    
 }
 
-
+#ifdef accessmode
+#undef accessmode
+#endif
 /*
  * Calculate the mode for a directory using a stat() call to
  * estimate permission.
  *
  * Note: the previous method, using access(), does not work correctly
  * over NFS.
- * FIXME what about ACL?
+ *
+ * dir parameter is used by AFS
  */
-void accessmode( path, ma, dir, st )
-char           *path;
-struct maccess *ma;
-struct dir     *dir;
-struct stat     *st;
+void accessmode(char *path, struct maccess *ma, struct dir *dir _U_, struct stat *st)
 
 {
 struct stat     sb;
 
     ma->ma_user = ma->ma_owner = ma->ma_world = ma->ma_group = 0;
     if (!st) {
-        if (stat(path, &sb) != 0)
+        if (lstat(path, &sb) != 0)
             return;
         st = &sb;
     }
     utommode( st, ma );
-    return;
+#ifdef HAVE_NFSv4_ACLS
+    /* 10.5 Finder looks at OS 9 mode, so we must do some mapping */
+    acltoownermode( path, st, uuid, ma);
+#endif
 }
 
-int gmem( gid )
-const gid_t    gid;
+int gmem(const gid_t gid)
 {
     int                i;
 
@@ -201,8 +201,7 @@ const gid_t gid;
     return( 0 );
 }
 
-static __inline__ mode_t mtoubits( bits )
-u_char bits;
+static mode_t mtoubits(u_char bits)
 {
     mode_t     mode;
 
@@ -221,8 +220,7 @@ u_char      bits;
    and from AFP 3.0 spec page 63 
    the mac mode should be save somewhere 
 */
-mode_t mtoumode( ma )
-struct maccess *ma;
+mode_t mtoumode(struct maccess *ma)
 {
     mode_t             mode;
 
@@ -238,57 +236,9 @@ struct maccess     *ma;
     return( mode );
 }
 
-/*
-   a dropbox is a folder where w is set but not r eg:
-   rwx-wx-wx or rwx-wx-- 
-   rwx----wx (is not asked by a Mac with OS >= 8.0 ?)
-*/
-static int stickydirmode(name, mode, dropbox)
-char * name;
-const mode_t mode;
-const int dropbox;
-{
-    int retval = 0;
-
-#ifdef DROPKLUDGE
-    /* Turn on the sticky bit if this is a drop box, also turn off the setgid bit */
-    if (dropbox) {
-        int uid;
-
-        if ( ( (mode & S_IWOTH) && !(mode & S_IROTH)) ||
-             ( (mode & S_IWGRP) && !(mode & S_IRGRP)) )
-        {
-            uid=geteuid();
-            if ( seteuid(0) < 0) {
-                LOG(log_error, logtype_afpd, "stickydirmode: unable to seteuid root: %s", strerror(errno));
-            }
-            if ( (retval=chmod( name, ( (DIRBITS | mode | S_ISVTX) & ~default_options.umask) )) < 0) {
-                LOG(log_error, logtype_afpd, "stickydirmode: chmod \"%s\": %s", name, strerror(errno) );
-            } else {
-#ifdef DEBUG
-                LOG(log_info, logtype_afpd, "stickydirmode: (debug) chmod \"%s\": %s", name, strerror(retval) );
-#endif /* DEBUG */
-            }
-            seteuid(uid);
-            return retval;
-        }
-    }
-#endif /* DROPKLUDGE */
+#define EXEC_MODE (S_IXGRP | S_IXUSR | S_IXOTH)
 
-    /*
-     *  Ignore EPERM errors:  We may be dealing with a directory that is
-     *  group writable, in which case chmod will fail.
-     */
-    if ( (chmod( name, (DIRBITS | mode) & ~default_options.umask ) < 0) && errno != EPERM)  {
-        LOG(log_error, logtype_afpd, "stickydirmode: chmod \"%s\": %s",name, strerror(errno) );
-        retval = -1;
-    }
-
-    return retval;
-}
-
-int setdeskmode( mode )
-const mode_t   mode;
+int setdeskmode(const mode_t mode)
 {
     char               wd[ MAXPATHLEN + 1];
     struct stat         st;
@@ -296,6 +246,10 @@ const mode_t       mode;
     struct dirent      *deskp, *subp;
     DIR                        *desk, *sub;
 
+    if (!dir_rx_set(mode)) {
+        /* want to remove read and search access to owner it will screw the volume */
+        return -1 ;
+    }
     if ( getcwd( wd , MAXPATHLEN) == NULL ) {
         return( -1 );
     }
@@ -327,28 +281,24 @@ const mode_t      mode;
             *m = '\0';
             strcat( modbuf, subp->d_name );
             /* XXX: need to preserve special modes */
-            if (stat(modbuf, &st) < 0) {
-                LOG(log_error, logtype_afpd, "setdeskmode: stat %s: %s",
-                    modbuf, strerror(errno) );
+            if (lstat(modbuf, &st) < 0) {
+                LOG(log_error, logtype_afpd, "setdeskmode: stat %s: %s",fullpathname(modbuf), strerror(errno) );
                 continue;
             }
 
             if (S_ISDIR(st.st_mode)) {
                 if ( chmod( modbuf,  (DIRBITS | mode) & ~default_options.umask ) < 0 && errno != EPERM ) {
-                    LOG(log_error, logtype_afpd, "setdeskmode: chmod %s: %s",
-                        modbuf, strerror(errno) );
+                     LOG(log_error, logtype_afpd, "setdeskmode: chmod %s: %s",fullpathname(modbuf), strerror(errno) );
                 }
-            } else if ( chmod( modbuf,  mode & ~default_options.umask ) < 0 && errno != EPERM ) {
-                LOG(log_error, logtype_afpd, "setdeskmode: chmod %s: %s",
-                    modbuf, strerror(errno) );
+            } else if ( chmod( modbuf,  mode & ~(default_options.umask | EXEC_MODE) ) < 0 && errno != EPERM ) {
+                LOG(log_error, logtype_afpd, "setdeskmode: chmod %s: %s",fullpathname(modbuf), strerror(errno) );
             }
 
         }
         closedir( sub );
         /* XXX: need to preserve special modes */
         if ( chmod( deskp->d_name,  (DIRBITS | mode) & ~default_options.umask ) < 0 && errno != EPERM ) {
-            LOG(log_error, logtype_afpd, "setdeskmode: chmod %s: %s",
-                deskp->d_name, strerror(errno) );
+            LOG(log_error, logtype_afpd, "setdeskmode: chmod %s: %s",fullpathname(deskp->d_name), strerror(errno) );
         }
     }
     closedir( desk );
@@ -358,124 +308,111 @@ const mode_t    mode;
     }
     /* XXX: need to preserve special modes */
     if ( chmod( ".AppleDesktop",  (DIRBITS | mode) & ~default_options.umask ) < 0 && errno != EPERM ) {
-        LOG(log_error, logtype_afpd, "setdeskmode: chmod .AppleDesktop: %s", strerror(errno) );
+        LOG(log_error, logtype_afpd, "setdeskmode: chmod %s: %s", fullpathname(".AppleDesktop"),strerror(errno) );
     }
     return( 0 );
 }
 
-int setfilmode(name, mode, st)
-char * name;
-mode_t mode;
-struct stat *st;
+/* --------------------- */
+int setfilunixmode (const struct vol *vol, struct path* path, mode_t mode)
 {
-struct stat sb;
-mode_t mask = S_IRUSR |S_IWUSR | S_IRGRP | S_IWGRP |S_IROTH | S_IWOTH;
+    if (!path->st_valid) {
+        of_stat(path);
+    }
 
-    if (!st) {
-        if (stat(name, &sb) != 0)
+    if (path->st_errno) {
+        return -1;
+    }
+        
+    mode |= vol->v_fperm;
+
+    if (setfilmode( path->u_name, mode, &path->st, vol->v_umask) < 0)
+        return -1;
+    /* we need to set write perm if read set for resource fork */
+    return vol->vfs->vfs_setfilmode(vol, path->u_name, mode, &path->st);
+}
+
+
+/* --------------------- */
+int setdirunixmode(const struct vol *vol, const char *name, mode_t mode)
+{
+
+    int dropbox = (vol->v_flags & AFPVOL_DROPBOX);
+    mode |= vol->v_dperm;
+
+    if (dir_rx_set(mode)) {
+       /* extending right? dir first then .AppleDouble in rf_setdirmode */
+       if ( stickydirmode(name, DIRBITS | mode, dropbox, vol->v_umask) < 0 )
+               return -1;
+    }
+    if (vol->vfs->vfs_setdirunixmode(vol, name, mode, NULL) < 0 && !vol_noadouble(vol)) {
+        return  -1 ;
+    }
+    if (!dir_rx_set(mode)) {
+       if ( stickydirmode(name, DIRBITS | mode, dropbox, vol->v_umask) < 0 )
             return -1;
-        st = &sb;
     }
-   mode &= mask;       /* keep only rw-rw-rw in mode */
-   mode |= st->st_mode & ~mask; /* keep other bits from previous mode */
-   if ( chmod( name,  mode & ~default_options.umask ) < 0 && errno != EPERM ) {
-       return -1;
-   }
-   return 0;
+    return 0;
 }
 
-int setdirmode( mode, noadouble, dropbox )
-const mode_t mode;
-const int noadouble;
-const int dropbox;
+/* --------------------- */
+int setdirmode(const struct vol *vol, const char *name, mode_t mode)
 {
-    char               buf[ MAXPATHLEN + 1];
     struct stat                st;
-    char               *m;
     struct dirent      *dirp;
     DIR                        *dir;
+    mode_t              hf_mode;
+    int                 osx = vol->v_adouble == AD_VERSION2_OSX;
+    int                 dropbox = (vol->v_flags & AFPVOL_DROPBOX);
     
-    if (( dir = opendir( "." )) == NULL ) {
-        LOG(log_error, logtype_afpd, "setdirmode: opendir .: %s", strerror(errno) );
+    mode |= vol->v_dperm;
+    hf_mode = ad_hf_mode(mode);
+
+    if (dir_rx_set(mode)) {
+       /* extending right? dir first */
+       if ( stickydirmode(name, DIRBITS | mode, dropbox, vol->v_umask) < 0 )
+               return -1;
+    }
+    
+    if (( dir = opendir( name )) == NULL ) {
+        LOG(log_error, logtype_afpd, "setdirmode: opendir: %s", fullpathname(name), strerror(errno) );
         return( -1 );
     }
 
     for ( dirp = readdir( dir ); dirp != NULL; dirp = readdir( dir )) {
-        if ( *dirp->d_name == '.' ) {
+        /* FIXME */
+        if ( *dirp->d_name == '.' && (!osx || dirp->d_name[1] != '_')) {
             continue;
         }
-        if ( stat( dirp->d_name, &st ) < 0 ) {
-            LOG(log_error, logtype_afpd, "setdirmode: stat %s: %s",
-                dirp->d_name, strerror(errno) );
+        if ( lstat( dirp->d_name, &st ) < 0 ) {
+            LOG(log_error, logtype_afpd, "setdirmode: stat %s: %s",dirp->d_name, strerror(errno) );
             continue;
         }
 
         if (!S_ISDIR(st.st_mode)) {
-           if (setfilmode(dirp->d_name, mode, &st) < 0) {
-                LOG(log_error, logtype_afpd, "setdirmode: chmod %s: %s",
-                    dirp->d_name, strerror(errno) );
+           int setmode = (osx && *dirp->d_name == '.')?hf_mode:mode;
+
+           if (setfilmode(dirp->d_name, setmode, &st, vol->v_umask) < 0) {
+                LOG(log_error, logtype_afpd, "setdirmode: chmod %s: %s",dirp->d_name, strerror(errno) );
                 return -1;
            }
         }
-#if 0
-            /* XXX: need to preserve special modes */
-        else if (S_ISDIR(st.st_mode)) {
-                if (stickydirmode(dirp->d_name, DIRBITS | mode, dropbox) < 0)
-                    return (-1);
-            } else if (stickydirmode(dirp->d_name, mode, dropbox) < 0)
-                return (-1);
-        }
-#endif
     }
     closedir( dir );
-
-    /* change perm of .AppleDouble's files
-    */
-    if (( dir = opendir( ".AppleDouble" )) == NULL ) {
-        if (noadouble)
-            goto setdirmode_noadouble;
-        LOG(log_error, logtype_afpd, "setdirmode: opendir .AppleDouble: %s", strerror(errno) );
-        return( -1 );
+    
+    if (vol->vfs->vfs_setdirmode(vol, name, mode, NULL) < 0 && !vol_noadouble(vol)) {
+        return  -1 ;
     }
-    strcpy( buf, ".AppleDouble" );
-    strcat( buf, "/" );
-    m = strchr( buf, '\0' );
-    for ( dirp = readdir( dir ); dirp != NULL; dirp = readdir( dir )) {
-        if ( strcmp( dirp->d_name, "." ) == 0 ||
-                strcmp( dirp->d_name, ".." ) == 0 ) {
-            continue;
-        }
-        *m = '\0';
-        strcat( buf, dirp->d_name );
-
-        if ( stat( buf, &st ) < 0 ) {
-            LOG(log_error, logtype_afpd, "setdirmode: stat %s: %s", buf, strerror(errno) );
-            continue;
-        }
-        if (!S_ISDIR(st.st_mode)) {
-           if (setfilmode(buf, ad_hf_mode(mode), &st) < 0) {
-               /* FIXME what do we do then? */
-           }
-        }
-    } /* end for */
-    closedir( dir );
-
-    /* XXX: use special bits to tag directory permissions */
 
-    /* XXX: need to preserve special modes */
-    if ( stickydirmode(".AppleDouble", DIRBITS | mode, dropbox) < 0 )
-        return( -1 );
-
-setdirmode_noadouble:
-    /* XXX: need to preserve special modes */
-    if ( stickydirmode(".", DIRBITS | mode, dropbox) < 0 )
-        return( -1 );
+    if (!dir_rx_set(mode)) {
+       if ( stickydirmode(name, DIRBITS | mode, dropbox, vol->v_umask) < 0 )
+               return -1;
+    }
     return( 0 );
 }
 
-int setdeskowner( uid, gid )
-const uid_t    uid;
-const gid_t    gid;
+/* ----------------------------- */
+int setdeskowner(const uid_t uid, const gid_t gid)
 {
     char               wd[ MAXPATHLEN + 1];
     char               modbuf[12 + 1], *m;
@@ -515,8 +452,7 @@ const gid_t gid;
             strcat( modbuf, subp->d_name );
             /* XXX: add special any uid, ignore group bits */
             if ( chown( modbuf, uid, gid ) < 0 && errno != EPERM ) {
-                LOG(log_error, logtype_afpd, "setdeskown: chown %s: %s",
-                    modbuf, strerror(errno) );
+                LOG(log_error, logtype_afpd, "setdeskown: chown %s: %s", fullpathname(modbuf), strerror(errno) );
             }
         }
         closedir( sub );
@@ -532,94 +468,82 @@ const gid_t       gid;
         return -1;
     }
     if ( chown( ".AppleDesktop", uid, gid ) < 0 && errno != EPERM ) {
-        LOG(log_error, logtype_afpd, "setdeskowner: chown .AppleDesktop: %s",
-            strerror(errno) );
+        LOG(log_error, logtype_afpd, "setdeskowner: chown %s: %s", fullpathname(".AppleDouble"), strerror(errno) );
     }
     return( 0 );
 }
 
+/* ----------------------------- */
+int setfilowner(const struct vol *vol, const uid_t uid, const gid_t gid, struct path* path)
+{
+
+    if (!path->st_valid) {
+        of_stat(path);
+    }
+
+    if (path->st_errno) {
+        return -1;
+    }
+
+    if ( chown( path->u_name, uid, gid ) < 0 && errno != EPERM ) {
+        LOG(log_debug, logtype_afpd, "setfilowner: chown %d/%d %s: %s",
+            uid, gid, path->u_name, strerror(errno) );
+       return -1;
+    }
+
+    if (vol->vfs->vfs_chown(vol, path->u_name, uid, gid ) < 0 && errno != EPERM) {
+        LOG(log_debug, logtype_afpd, "setfilowner: rf_chown %d/%d %s: %s",
+            uid, gid, path->u_name, strerror(errno) );
+        return -1;
+    }
+
+    return 0;
+}
 
-/* uid/gid == 0 need to be handled as special cases. they really mean
+/* --------------------------------- 
+ * uid/gid == 0 need to be handled as special cases. they really mean
  * that user/group should inherit from other, but that doesn't fit
  * into the unix permission scheme. we can get around this by
  * co-opting some bits. */
-int setdirowner( uid, gid, noadouble )
-const uid_t    uid;
-const gid_t    gid;
-const int   noadouble;
+int setdirowner(const struct vol *vol, const char *name, const uid_t uid, const gid_t gid)
 {
-    char               buf[ MAXPATHLEN + 1];
     struct stat                st;
-    char               *m;
     struct dirent      *dirp;
     DIR                        *dir;
+    int                 osx = vol->v_adouble == AD_VERSION2_OSX;
 
-    if (( dir = opendir( "." )) == NULL ) {
+    if (( dir = opendir( name )) == NULL ) {
         return( -1 );
     }
     for ( dirp = readdir( dir ); dirp != NULL; dirp = readdir( dir )) {
-        if ( *dirp->d_name == '.' ) {
+        if ( *dirp->d_name == '.' && (!osx || dirp->d_name[1] != '_')) {
             continue;
-        };
-        if ( stat( dirp->d_name, &st ) < 0 ) {
+        }
+        if ( lstat( dirp->d_name, &st ) < 0 ) {
             LOG(log_error, logtype_afpd, "setdirowner: stat %s: %s",
-                dirp->d_name, strerror(errno) );
+                fullpathname(dirp->d_name), strerror(errno) );
             continue;
         }
         if (( st.st_mode & S_IFMT ) == S_IFREG ) {
             if ( chown( dirp->d_name, uid, gid ) < 0 && errno != EPERM ) {
                 LOG(log_debug, logtype_afpd, "setdirowner: chown %s: %s",
-                    dirp->d_name, strerror(errno) );
+                    fullpathname(dirp->d_name), strerror(errno) );
                 /* return ( -1 ); Sometimes this is okay */
             }
         }
     }
     closedir( dir );
-    if (( dir = opendir( ".AppleDouble" )) == NULL ) {
-        if (noadouble)
-            goto setdirowner_noadouble;
-        return( -1 );
-    }
-    strcpy( buf, ".AppleDouble" );
-    strcat( buf, "/" );
-    m = strchr( buf, '\0' );
-    for ( dirp = readdir( dir ); dirp != NULL; dirp = readdir( dir )) {
-        if ( strcmp( dirp->d_name, "." ) == 0 ||
-                strcmp( dirp->d_name, ".." ) == 0 ) {
-            continue;
-        }
-        *m = '\0';
-        strcat( buf, dirp->d_name );
-        if ( chown( buf, uid, gid ) < 0 && errno != EPERM ) {
-            LOG(log_debug, logtype_afpd, "setdirowner: chown %d/%d %s: %s",
-                uid, gid, buf, strerror(errno) );
-            /* return ( -1 ); Sometimes this is okay */
-        }
-    }
-    closedir( dir );
 
-    /*
-     * We cheat: we know that chown doesn't do anything.
-     */
-    if ( stat( ".AppleDouble", &st ) < 0 ) {
-        LOG(log_error, logtype_afpd, "setdirowner: stat .AppleDouble: %s", strerror(errno) );
-        return( -1 );
-    }
-    if ( gid && gid != st.st_gid && chown( ".AppleDouble", uid, gid ) < 0 &&
-            errno != EPERM ) {
-        LOG(log_debug, logtype_afpd, "setdirowner: chown %d/%d .AppleDouble: %s",
-            uid, gid, strerror(errno) );
-        /* return ( -1 ); Sometimes this is okay */
+    if (vol->vfs->vfs_setdirowner(vol, name, uid, gid) < 0) {
+        return -1;
     }
-
-setdirowner_noadouble:
-    if ( stat( ".", &st ) < 0 ) {
+    
+    if ( lstat( ".", &st ) < 0 ) {
         return( -1 );
     }
-    if ( gid && gid != st.st_gid && chown( ".", uid, gid ) < 0 &&
-            errno != EPERM ) {
-        LOG(log_debug, logtype_afpd, "setdirowner: chown %d/%d .: %s",
-            uid, gid, strerror(errno) );
+    if ( gid && gid != st.st_gid && chown( ".", uid, gid ) < 0 && errno != EPERM ) {
+        LOG(log_debug, logtype_afpd, "setdirowner: chown %d/%d %s: %s",
+            uid, gid, fullpathname("."), strerror(errno) );
     }
 
     return( 0 );
@@ -637,19 +561,19 @@ static int recursive_chown(const char *path, uid_t uid, gid_t gid) {
     newpath[PATH_MAX] = '\0';
     
     if (chown(path, uid, gid) < 0) {
-        LOG(log_error, logtype_afpd, "cannot chown() file [%s] (uid = %d): %s\n", path, uid, strerror(errno));
+        LOG(log_error, logtype_afpd, "cannot chown() file [%s] (uid = %d): %s", path, uid, strerror(errno));
        return -1;
     }
 
-    if (stat(path, &sbuf) < 0) {
-       LOG(log_error, logtype_afpd, "cannot chown() file [%s] (uid = %d): %s\n", path, uid, strerror(errno));
+    if (lstat(path, &sbuf) < 0) {
+       LOG(log_error, logtype_afpd, "cannot chown() file [%s] (uid = %d): %s", path, uid, strerror(errno));
        return -1;
     }
        
     if (S_ISDIR(sbuf.st_mode)) {
        odir = opendir(path);
        if (odir == NULL) {
-           LOG(log_error, logtype_afpd, "cannot opendir() [%s] (uid = %d): %s\n", path, uid, strerror(errno));
+           LOG(log_error, logtype_afpd, "cannot opendir() [%s] (uid = %d): %s", path, uid, strerror(errno));
            goto recursive_chown_end;
        }
        while (NULL != (entry=readdir(odir)) ) {
@@ -672,43 +596,3 @@ recursive_chown_end:
 }
 #endif
 
-/* This is equivalent of unix rename(). */
-int unix_rename(const char *oldpath, const char *newpath)
-{
-       char pd_name[PATH_MAX+1];
-       int i;
-        struct stat pd_stat;
-        uid_t uid;
-
-       if (rename(oldpath, newpath) < 0)
-               return -1;
-#if 0
-       for (i = 0; i <= PATH_MAX && newpath[i] != '\0'; i++)
-               pd_name[i] = newpath[i];
-       pd_name[i] = '\0';
-
-       while (i > 0 && pd_name[i] != '/') i--;
-       if (pd_name[i] == '/') i++;
-
-        pd_name[i++] = '.'; pd_name[i++] = '\0';
-
-        if (stat(pd_name, &pd_stat) < 0) {
-           LOG(log_error, logtype_afpd, "stat() of parent dir failed: pd_name = %s, uid = %d: %s\n",
-               pd_name, geteuid(), strerror(errno));
-               return 0;
-       }
-
-       /* So we have SGID bit set... */
-        if ((S_ISGID & pd_stat.st_mode)        != 0) {
-            uid = geteuid();
-            if (seteuid(0) < 0)
-               LOG(log_error, logtype_afpd, "seteuid() failed: %s\n", strerror(errno));
-            if (recursive_chown(newpath, uid, pd_stat.st_gid) < 0)
-               LOG(log_error, logtype_afpd, "chown() of parent dir failed: newpath=%s, uid=%d: %s\n",
-                   pd_name, geteuid(), strerror(errno));
-            seteuid(uid);
-       }
-#endif
-       return 0;
-}
-