]> arthur.barton.de Git - netatalk.git/blobdiff - etc/afpd/unix.c
try to make afpd directory access right works the way Macs expect
[netatalk.git] / etc / afpd / unix.c
index 83464411649c04c9d9933c3c94c80d5f64ac5506..7f948bc907a8492f53fc956fca1e2fd1dd999e9a 100644 (file)
@@ -1,25 +1,46 @@
 /*
+ * $Id: unix.c,v 1.36 2002-08-29 18:57:26 didg Exp $
+ *
  * Copyright (c) 1990,1993 Regents of The University of Michigan.
  * All Rights Reserved.  See COPYRIGHT.
  */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif
+#endif /* HAVE_CONFIG_H */
 
 #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 <sys/syslog.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
 #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 "auth.h"
 #include "directory.h"
 #include "volume.h"
  * 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;
+const struct vol       *vol;
+VolSpace    *bfree, *btotal;
+u_int32_t   *bsize;
 {
+    VolSpace maxVolSpace = (~(VolSpace)0);
+
 #ifdef ultrix
     struct fs_data     sfs;
 #else /*ultrix*/
     struct statfs      sfs;
 #endif /*ultrix*/
 
+
     if ( statfs( vol->v_path, &sfs ) < 0 ) {
-       return( AFPERR_PARAM );
+        LOG(log_error, logtype_afpd, "ustatfs_getvolspace unable to stat %s", vol->v_path);
+        return( AFPERR_PARAM );
     }
 
 #ifdef ultrix
-    *bfree = (VolSpace) sfs.fd_req.bfreen * 1024;
+    *bfree = (VolSpace) sfs.fd_req.bfreen;
     *bsize = 1024;
-#else
-    *bfree = (VolSpace) sfs.f_bavail * sfs.f_frsize;
+#else /* !ultrix */
+    *bfree = (VolSpace) sfs.f_bavail;
     *bsize = sfs.f_frsize;
-#endif ultrix
+#endif /* ultrix */
+
+    if ( *bfree > maxVolSpace / *bsize ) {
+        *bfree = maxVolSpace;
+    } else {
+        *bfree *= *bsize;
+    }
 
 #ifdef ultrix
-    *btotal = (VolSpace) 
-      ( sfs.fd_req.btot - ( sfs.fd_req.bfree - sfs.fd_req.bfreen )) * 1024;
-#else ultrix
-    *btotal = (VolSpace) 
-      ( sfs.f_blocks - ( sfs.f_bfree - sfs.f_bavail )) * sfs.f_frsize;
-#endif ultrix
+    *btotal = (VolSpace)
+              ( sfs.fd_req.btot - ( sfs.fd_req.bfree - sfs.fd_req.bfreen ));
+#else /* !ultrix */
+    *btotal = (VolSpace)
+              ( sfs.f_blocks - ( sfs.f_bfree - sfs.f_bavail ));
+#endif /* ultrix */
+
+    /* see similar block above comments */
+    if ( *btotal > maxVolSpace / *bsize ) {
+        *btotal = maxVolSpace;
+    } else {
+        *btotal *= *bsize;
+    }
+
     return( AFP_OK );
 }
 
 static __inline__ int utombits( bits )
-    mode_t     bits;
+mode_t bits;
 {
     int                mbits;
 
     mbits = 0;
 
-    mbits |= ( bits & ( S_IREAD >> 6 )) ? AR_UREAD : 0;
+    mbits |= ( bits & ( S_IREAD >> 6 )) ? (AR_UREAD | AR_USEARCH) : 0;
     mbits |= ( bits & ( S_IWRITE >> 6 )) ? AR_UWRITE : 0;
-    mbits |= ( bits & ( S_IEXEC >> 6) ) ? AR_USEARCH : 0;
+    /* Do we really need this?
+        mbits |= ( bits & ( S_IEXEC >> 6) ) ? AR_USEARCH : 0; */
 
     return( mbits );
 }
 
+/* --------------------------------
+    cf AFP 3.0 page 63
+*/
 void utommode( stat, ma )
-    struct stat                *stat;
-    struct maccess     *ma;
+struct stat            *stat;
+struct maccess *ma;
 {
-    mode_t             mode;
+mode_t mode;
 
     mode = stat->st_mode;
-
     ma->ma_world = utombits( mode );
     mode = mode >> 3;
 
@@ -91,68 +133,100 @@ void utommode( stat, ma )
 
     ma->ma_owner = utombits( mode );
 
-#ifdef ADMIN_GRP
-    if ( uuid == 0 )
-        ma->ma_user = AR_UWRITE | AR_UREAD | AR_USEARCH | AR_UOWN;
-    else
-#endif
-
-    if ( uuid == stat->st_uid ) {
-       ma->ma_user = ma->ma_owner | AR_UOWN;
-    } else if ( gmem( stat->st_gid )) {
-       ma->ma_user = ma->ma_group;
-    } else {
-       ma->ma_user = ma->ma_world;
+    /* ma_user is a union of all permissions */
+    ma->ma_user = 0;
+    if ( (uuid == stat->st_uid) || (uuid == 0)) {
+        ma->ma_user = ma->ma_owner | AR_UOWN;
     }
+    if ( gmem( stat->st_gid )) {
+        ma->ma_user |= ma->ma_group;
+    } 
+    ma->ma_user |= ma->ma_world;
 
     /*
      * There are certain things the mac won't try if you don't have
      * 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 ?
      */
+#if 0
     if ( ma->ma_user & AR_UWRITE ) {
-       ma->ma_user |= AR_UOWN;
+        ma->ma_user |= AR_UOWN;
     }
+#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.
+ */
+void accessmode( path, ma, dir, st )
+char           *path;
+struct maccess *ma;
+struct dir     *dir;
+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)
+            return;
+        st = &sb;
+    }
+    utommode( st, ma );
+    return;
+}
+
 int gmem( gid )
-    const gid_t        gid;
+const gid_t    gid;
 {
     int                i;
 
     for ( i = 0; i < ngroups; i++ ) {
-       if ( groups[ i ] == gid ) {
-           return( 1 );
-       }
+        if ( groups[ i ] == gid ) {
+            return( 1 );
+        }
     }
     return( 0 );
 }
 
 static __inline__ mode_t mtoubits( bits )
-    u_char     bits;
+u_char bits;
 {
     mode_t     mode;
 
     mode = 0;
 
-    mode |= ( bits & AR_UREAD ) ? ( S_IREAD >> 6 ) : 0;
-    mode |= ( bits & AR_UWRITE ) ? ( S_IWRITE >> 6 ) : 0;
-    mode |= ( bits & AR_USEARCH ) ? ( S_IEXEC >> 6 ) : 0;
+    mode |= ( bits & AR_UREAD ) ? ( (S_IREAD | S_IEXEC) >> 6 ) : 0;
+    mode |= ( bits & AR_UWRITE ) ? ( (S_IWRITE | S_IEXEC) >> 6 ) : 0;
+    /* I don't think there's a way to set the SEARCH bit by itself on a Mac
+        mode |= ( bits & AR_USEARCH ) ? ( S_IEXEC >> 6 ) : 0; */
 
     return( mode );
 }
 
+/* ----------------------------------
+   from the finder's share windows (menu--> File--> sharing...)
+   and from AFP 3.0 spec page 63 
+   the mac mode should be save somewhere 
+*/
 mode_t mtoumode( ma )
-    struct maccess     *ma;
+struct maccess *ma;
 {
     mode_t             mode;
 
     mode = 0;
-    mode |= mtoubits( ma->ma_owner );
+    mode |= mtoubits( ma->ma_owner |ma->ma_world);
     mode = mode << 3;
 
-    mode |= mtoubits( ma->ma_group );
+    mode |= mtoubits( ma->ma_group |ma->ma_world);
     mode = mode << 3;
 
     mode |= mtoubits( ma->ma_world );
@@ -160,9 +234,58 @@ mode_t mtoumode( ma )
     return( mode );
 }
 
+int stickydirmode(name, mode, dropbox)
+char * name;
+const mode_t mode;
+const int dropbox;
+{
+    int retval;
+#ifdef DROPKLUDGE
+    int uid;
+#endif /* DROPKLUDGE */
+
+    /* Turn on the sticky bit if this is a drop box, also turn off the setgid bit */
+    retval=0;
+#ifdef DROPKLUDGE
+    if (dropbox) {
+        if (mode & S_IWOTH) {
+            if (mode & S_IROTH);
+            else { /* if S_IWOTH and not S_IROTH */
+                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) );
+                    return(AFPERR_ACCESS);
+                } else {
+#ifdef DEBUG
+                    LOG(log_info, logtype_afpd, "stickydirmode: (debug) chmod \"%s\": %s", name, strerror(retval) );
+#endif /* DEBUG */
+                    seteuid(uid);
+                } /* end getting retval */
+            } /* end if not & S_IROTH */
+        } else { /* end if S_IWOTH and not S_IROTH */
+#endif /* DROPKLUDGE */
+
+            /*
+            *  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;
+            }
+#ifdef DROPKLUDGE
+        } /* end if not mode */
+    } /* end checking for "dropbox" */
+#endif /* DROPKLUDGE */
+    return retval;
+}
 
 int setdeskmode( mode )
-    const mode_t       mode;
+const mode_t   mode;
 {
     char               wd[ MAXPATHLEN + 1];
     struct stat         st;
@@ -171,156 +294,185 @@ int setdeskmode( mode )
     DIR                        *desk, *sub;
 
     if ( getcwd( wd , MAXPATHLEN) == NULL ) {
-       return( -1 );
+        return( -1 );
     }
     if ( chdir( ".AppleDesktop" ) < 0 ) {
-       return( -1 );
+        return( -1 );
     }
     if (( desk = opendir( "." )) == NULL ) {
-       if ( chdir( wd ) < 0 ) {
-           syslog( LOG_ERR, "setdeskmode: chdir %s: %m", wd );
-       }
-       return( -1 );
+        if ( chdir( wd ) < 0 ) {
+            LOG(log_error, logtype_afpd, "setdeskmode: chdir %s: %s", wd, strerror(errno) );
+        }
+        return( -1 );
     }
     for ( deskp = readdir( desk ); deskp != NULL; deskp = readdir( desk )) {
-       if ( strcmp( deskp->d_name, "." ) == 0 ||
-               strcmp( deskp->d_name, ".." ) == 0 || strlen( deskp->d_name ) > 2 ) {
-           continue;
-       }
-       strcpy( modbuf, deskp->d_name );
-       strcat( modbuf, "/" );
-       m = strchr( modbuf, '\0' );
-       if (( sub = opendir( deskp->d_name )) == NULL ) {
-           continue;
-       }
-       for ( subp = readdir( sub ); subp != NULL; subp = readdir( sub )) {
-           if ( strcmp( subp->d_name, "." ) == 0 ||
-                   strcmp( subp->d_name, ".." ) == 0 ) {
-               continue;
-           }
-           *m = '\0';
-           strcat( modbuf, subp->d_name );
-           /* XXX: need to preserve special modes */
-           if (stat(modbuf, &st) < 0) {
-               syslog( LOG_DEBUG, "setdeskmode: stat %s: %m", modbuf );
-               continue;
-           }       
-
-           if (S_ISDIR(st.st_mode)) {
-             if ( chmod( modbuf,  DIRBITS | mode ) < 0 ) {
-               syslog( LOG_DEBUG, "setdeskmode: chmod %s: %m", modbuf );
-             }
-           } else if ( chmod( modbuf,  mode ) < 0 ) {
-               syslog( LOG_DEBUG, "setdeskmode: chmod %s: %m", modbuf );
-           }
-
-       }
-       closedir( sub );
-       /* XXX: need to preserve special modes */
-       if ( chmod( deskp->d_name,  DIRBITS | mode ) < 0 ) {
-           syslog( LOG_DEBUG, "setdeskmode: chmod %s: %m", deskp->d_name );
-       }
+        if ( strcmp( deskp->d_name, "." ) == 0 ||
+                strcmp( deskp->d_name, ".." ) == 0 || strlen( deskp->d_name ) > 2 ) {
+            continue;
+        }
+        strcpy( modbuf, deskp->d_name );
+        strcat( modbuf, "/" );
+        m = strchr( modbuf, '\0' );
+        if (( sub = opendir( deskp->d_name )) == NULL ) {
+            continue;
+        }
+        for ( subp = readdir( sub ); subp != NULL; subp = readdir( sub )) {
+            if ( strcmp( subp->d_name, "." ) == 0 ||
+                    strcmp( subp->d_name, ".." ) == 0 ) {
+                continue;
+            }
+            *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) );
+                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) );
+                }
+            } else if ( chmod( modbuf,  mode & ~default_options.umask ) < 0 && errno != EPERM ) {
+                LOG(log_error, logtype_afpd, "setdeskmode: chmod %s: %s",
+                    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) );
+        }
     }
     closedir( desk );
     if ( chdir( wd ) < 0 ) {
-       syslog( LOG_ERR, "setdeskmode: chdir %s: %m", wd );
-       return -1;
+        LOG(log_error, logtype_afpd, "setdeskmode: chdir %s: %s", wd, strerror(errno) );
+        return -1;
     }
     /* XXX: need to preserve special modes */
-    if ( chmod( ".AppleDesktop",  DIRBITS | mode ) < 0 ) {
-       syslog( LOG_DEBUG, "setdeskmode: chmod .AppleDesktop: %m" );
+    if ( chmod( ".AppleDesktop",  (DIRBITS | mode) & ~default_options.umask ) < 0 && errno != EPERM ) {
+        LOG(log_error, logtype_afpd, "setdeskmode: chmod .AppleDesktop: %s", strerror(errno) );
     }
     return( 0 );
 }
 
-int setdirmode( mode, noadouble )
-    const mode_t mode;
-    const int noadouble;
+int setfilmode(name, mode, st)
+char * name;
+mode_t mode;
+struct stat *st;
+{
+struct stat sb;
+mode_t mask = S_IRUSR |S_IWUSR | S_IRGRP | S_IWGRP |S_IROTH | S_IWOTH;
+
+    if (!st) {
+        if (stat(name, &sb) != 0)
+            return;
+        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;
+}
+
+int setdirmode( mode, noadouble, dropbox )
+const mode_t mode;
+const int noadouble;
+const int dropbox;
 {
     char               buf[ MAXPATHLEN + 1];
     struct stat                st;
     char               *m;
     struct dirent      *dirp;
     DIR                        *dir;
-
+    
     if (( dir = opendir( "." )) == NULL ) {
-       syslog( LOG_ERR, "setdirmode: opendir .: %m" );
-       return( -1 );
+        LOG(log_error, logtype_afpd, "setdirmode: opendir .: %s", strerror(errno) );
+        return( -1 );
     }
 
     for ( dirp = readdir( dir ); dirp != NULL; dirp = readdir( dir )) {
-       if ( *dirp->d_name == '.' ) {
-           continue;
-       }
-       if ( stat( dirp->d_name, &st ) < 0 ) {
-           syslog( LOG_DEBUG, "setdirmode: stat %s: %m", dirp->d_name );
-           continue;
-       }
-
-       if (S_ISREG(st.st_mode)) {
-           /* XXX: need to preserve special modes */
-           if (S_ISDIR(st.st_mode)) {
-             if ( chmod( dirp->d_name, DIRBITS | mode ) < 0 ) {
-               syslog( LOG_DEBUG, "setdirmode: chmod %s: %m", dirp->d_name );
-             }
-           } else if ( chmod( dirp->d_name, mode ) < 0 ) {
-               syslog( LOG_DEBUG, "setdirmode: chmod %s: %m", dirp->d_name );
-           }
-       }
+        if ( *dirp->d_name == '.' ) {
+            continue;
+        }
+        if ( stat( dirp->d_name, &st ) < 0 ) {
+            LOG(log_error, logtype_afpd, "setdirmode: stat %s: %s",
+                dirp->d_name, strerror(errno) );
+            continue;
+        }
+
+        if (S_ISREG(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) );
+                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;
-       syslog( LOG_ERR, "setdirmode: opendir .AppleDouble: %m" );
-       return( -1 );
+            goto setdirmode_noadouble;
+        LOG(log_error, logtype_afpd, "setdirmode: opendir .AppleDouble: %s", strerror(errno) );
+        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 ) {
-           syslog( LOG_DEBUG, "setdirmode: stat %s: %m", buf );
-           continue;
-       }
-
-       if (S_ISDIR(st.st_mode)) {
-         if ( chmod(buf,  DIRBITS | mode) < 0 ) {
-           syslog( LOG_DEBUG, "setdirmode: chmod %s: %m", buf );
-         }
-       } else if ( chmod(buf, mode) < 0 ) {
-           syslog( LOG_DEBUG, "setdirmode: chmod %s: %m", buf );
-       }
-    }
+        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_ISREG(st.st_mode)) {
+           if (setfilmode(dirp->d_name, 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 ( chmod( ".AppleDouble",  DIRBITS | mode ) < 0 ) {
-       syslog( LOG_ERR, "setdirmode: chmod .AppleDouble: %m" );
-       return( -1 );
-    }
+    if ( stickydirmode(".AppleDouble", DIRBITS | mode, dropbox) < 0 )
+        return( -1 );
 
 setdirmode_noadouble:
     /* XXX: need to preserve special modes */
-    if ( chmod( ".",  DIRBITS | mode ) < 0 ) {
-       syslog( LOG_ERR, "setdirmode: chmod .: %m" );
-       return( -1 );
-    }
+    if ( stickydirmode(".", DIRBITS | mode, dropbox) < 0 )
+        return( -1 );
     return( 0 );
 }
 
 int setdeskowner( uid, gid )
-    const uid_t        uid;
-    const gid_t        gid;
+const uid_t    uid;
+const gid_t    gid;
 {
     char               wd[ MAXPATHLEN + 1];
     char               modbuf[12 + 1], *m;
@@ -328,54 +480,57 @@ int setdeskowner( uid, gid )
     DIR                        *desk, *sub;
 
     if ( getcwd( wd, MAXPATHLEN ) == NULL ) {
-       return( -1 );
+        return( -1 );
     }
     if ( chdir( ".AppleDesktop" ) < 0 ) {
-       return( -1 );
+        return( -1 );
     }
     if (( desk = opendir( "." )) == NULL ) {
-       if ( chdir( wd ) < 0 ) {
-           syslog( LOG_ERR, "setdeskowner: chdir %s: %m", wd );
-       }
-       return( -1 );
+        if ( chdir( wd ) < 0 ) {
+            LOG(log_error, logtype_afpd, "setdeskowner: chdir %s: %s", wd, strerror(errno) );
+        }
+        return( -1 );
     }
     for ( deskp = readdir( desk ); deskp != NULL; deskp = readdir( desk )) {
-       if ( strcmp( deskp->d_name, "." ) == 0 ||
-            strcmp( deskp->d_name, ".." ) == 0 || 
-            strlen( deskp->d_name ) > 2 ) {
-           continue;
-       }
-       strcpy( modbuf, deskp->d_name );
-       strcat( modbuf, "/" );
-       m = strchr( modbuf, '\0' );
-       if (( sub = opendir( deskp->d_name )) == NULL ) {
-           continue;
-       }
-       for ( subp = readdir( sub ); subp != NULL; subp = readdir( sub )) {
-           if ( strcmp( subp->d_name, "." ) == 0 ||
-                strcmp( subp->d_name, ".." ) == 0 ) {
-               continue;
-           }
-           *m = '\0';
-           strcat( modbuf, subp->d_name );
-           /* XXX: add special any uid, ignore group bits */
-           if ( chown( modbuf, uid, gid ) < 0 ) {
-               syslog( LOG_DEBUG, "setdeskown: chown %s: %m", modbuf );
-           }
-       }
-       closedir( sub );
-       /* XXX: add special any uid, ignore group bits */
-       if ( chown( deskp->d_name, uid, gid ) < 0 ) {
-           syslog( LOG_DEBUG, "setdeskowner: chown %s: %m", deskp->d_name );
-       }
+        if ( strcmp( deskp->d_name, "." ) == 0 ||
+                strcmp( deskp->d_name, ".." ) == 0 ||
+                strlen( deskp->d_name ) > 2 ) {
+            continue;
+        }
+        strcpy( modbuf, deskp->d_name );
+        strcat( modbuf, "/" );
+        m = strchr( modbuf, '\0' );
+        if (( sub = opendir( deskp->d_name )) == NULL ) {
+            continue;
+        }
+        for ( subp = readdir( sub ); subp != NULL; subp = readdir( sub )) {
+            if ( strcmp( subp->d_name, "." ) == 0 ||
+                    strcmp( subp->d_name, ".." ) == 0 ) {
+                continue;
+            }
+            *m = '\0';
+            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) );
+            }
+        }
+        closedir( sub );
+        /* XXX: add special any uid, ignore group bits */
+        if ( chown( deskp->d_name, uid, gid ) < 0 && errno != EPERM ) {
+            LOG(log_error, logtype_afpd, "setdeskowner: chown %s: %s",
+                deskp->d_name, strerror(errno) );
+        }
     }
     closedir( desk );
     if ( chdir( wd ) < 0 ) {
-       syslog( LOG_ERR, "setdeskowner: chdir %s: %m", wd );
-       return -1;
+        LOG(log_error, logtype_afpd, "setdeskowner: chdir %s: %s", wd, strerror(errno) );
+        return -1;
     }
-    if ( chown( ".AppleDesktop", uid, gid ) < 0 ) {
-       syslog( LOG_ERR, "setdeskowner: chown .AppleDesktop: %m" );
+    if ( chown( ".AppleDesktop", uid, gid ) < 0 && errno != EPERM ) {
+        LOG(log_error, logtype_afpd, "setdeskowner: chown .AppleDesktop: %s",
+            strerror(errno) );
     }
     return( 0 );
 }
@@ -386,9 +541,9 @@ int setdeskowner( uid, gid )
  * 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;
+const uid_t    uid;
+const gid_t    gid;
+const int   noadouble;
 {
     char               buf[ MAXPATHLEN + 1];
     struct stat                st;
@@ -397,42 +552,46 @@ int setdirowner( uid, gid, noadouble )
     DIR                        *dir;
 
     if (( dir = opendir( "." )) == NULL ) {
-       return( -1 );
+        return( -1 );
     }
     for ( dirp = readdir( dir ); dirp != NULL; dirp = readdir( dir )) {
-       if ( *dirp->d_name == '.' ) {
-           continue;
-       };
-       if ( stat( dirp->d_name, &st ) < 0 ) {
-           syslog( LOG_DEBUG, "setdirowner: stat %s: %m", dirp->d_name );
-           continue;
-       }
-       if (( st.st_mode & S_IFMT ) == S_IFREG ) {
-           if ( chown( dirp->d_name, uid, gid ) < 0 ) {
-               syslog( LOG_DEBUG, "setdirowner: chown %s: %m", dirp->d_name );
-           }
-       }
+        if ( *dirp->d_name == '.' ) {
+            continue;
+        };
+        if ( stat( dirp->d_name, &st ) < 0 ) {
+            LOG(log_error, logtype_afpd, "setdirowner: stat %s: %s",
+                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) );
+                /* return ( -1 ); Sometimes this is okay */
+            }
+        }
     }
     closedir( dir );
     if (( dir = opendir( ".AppleDouble" )) == NULL ) {
-      if (noadouble)
-       goto setdirowner_noadouble;
-      return( -1 );
+        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 ) {
-           syslog( LOG_DEBUG, "setdirowner: chown %d/%d %s: %m",
-                   uid, gid, buf );
-       }
+        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 );
 
@@ -440,21 +599,24 @@ int setdirowner( uid, gid, noadouble )
      * We cheat: we know that chown doesn't do anything.
      */
     if ( stat( ".AppleDouble", &st ) < 0 ) {
-       syslog( LOG_ERR, "setdirowner: stat .AppleDouble: %m" );
-       return( -1 );
+        LOG(log_error, logtype_afpd, "setdirowner: stat .AppleDouble: %s", strerror(errno) );
+        return( -1 );
     }
-    if ( gid && gid != st.st_gid && chown( ".AppleDouble", uid, gid ) < 0 ) {
-       syslog( LOG_DEBUG, "setdirowner: chown %d/%d .AppleDouble: %m",
-               uid, gid);
+    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 */
     }
 
 setdirowner_noadouble:
     if ( stat( ".", &st ) < 0 ) {
-       return( -1 );
+        return( -1 );
     }
-    if ( gid && gid != st.st_gid && chown( ".", uid, gid ) < 0 ) {
-        syslog( LOG_DEBUG, "setdirowner: chown %d/%d .: %m",
-               uid, gid);
+    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) );
     }
 
     return( 0 );