]> arthur.barton.de Git - netatalk.git/blobdiff - etc/afpd/file.c
Fix a bogon compile error.
[netatalk.git] / etc / afpd / file.c
index 826577bb5eb0707c33c60be8cc664046eff7bbd6..b21a99226d6578a4ffad724e8294b6b3446bb06a 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: file.c,v 1.36 2002-01-17 16:19:06 jmarcus Exp $
+ * $Id: file.c,v 1.37 2002-01-20 01:32:45 jmarcus Exp $
  *
  * Copyright (c) 1990,1993 Regents of The University of Michigan.
  * All Rights Reserved.  See COPYRIGHT.
@@ -89,11 +89,11 @@ char *strchr (), *strrchr ();
  */
 
 const u_char ufinderi[] = {
-    'T', 'E', 'X', 'T', 'U', 'N', 'I', 'X',
-    0, 0, 0, 0, 0, 0, 0, 0,
-    0, 0, 0, 0, 0, 0, 0, 0,
-    0, 0, 0, 0, 0, 0, 0, 0
-};
+                              'T', 'E', 'X', 'T', 'U', 'N', 'I', 'X',
+                              0, 0, 0, 0, 0, 0, 0, 0,
+                              0, 0, 0, 0, 0, 0, 0, 0,
+                              0, 0, 0, 0, 0, 0, 0, 0
+                          };
 
 int getfilparams(struct vol *vol,
                  u_int16_t bitmap,
@@ -236,1509 +236,1510 @@ int getfilparams(struct vol *vol,
                 case CNID_ERR_MAX:
                     return(AFPERR_MISC);
                 }
+            }
 #endif /* CNID_DB */
 
-                if (aint == 0) {
-                    /*
-                     * What a fucking mess.  First thing:  DID and FNUMs are
-                     * in the same space for purposes of enumerate (and several
-                     * other wierd places).  While we consider this Apple's bug,
-                     * this is the work-around:  In order to maintain constant and
-                     * unique DIDs and FNUMs, we monotonically generate the DIDs
-                     * during the session, and derive the FNUMs from the filesystem.
-                     * Since the DIDs are small, we insure that the FNUMs are fairly
-                     * large by setting thier high bits to the device number.
-                     *
-                     * AFS already does something very similar to this for the
-                     * inode number, so we don't repeat the procedure.
-                     *
-                     * new algorithm:
-                     * due to complaints over did's being non-persistent,
-                     * here's the current hack to provide semi-persistent
-                     * did's:
-                     *      1) we reserve the first bit for file ids.
-                     *      2) the next 7 bits are for the device.
-                     *      3) the remaining 24 bits are for the inode.
-                     *
-                     * both the inode and device information are actually hashes
-                     * that are then truncated to the requisite bit length.
-                     *
-                     * it should be okay to use lstat to deal with symlinks.
-                     */
+            if (aint == 0) {
+                /*
+                 * What a fucking mess.  First thing:  DID and FNUMs are
+                 * in the same space for purposes of enumerate (and several
+                 * other wierd places).  While we consider this Apple's bug,
+                 * this is the work-around:  In order to maintain constant and
+                 * unique DIDs and FNUMs, we monotonically generate the DIDs
+                 * during the session, and derive the FNUMs from the filesystem.
+                 * Since the DIDs are small, we insure that the FNUMs are fairly
+                 * large by setting thier high bits to the device number.
+                 *
+                 * AFS already does something very similar to this for the
+                 * inode number, so we don't repeat the procedure.
+                 *
+                 * new algorithm:
+                 * due to complaints over did's being non-persistent,
+                 * here's the current hack to provide semi-persistent
+                 * did's:
+                 *      1) we reserve the first bit for file ids.
+                 *      2) the next 7 bits are for the device.
+                 *      3) the remaining 24 bits are for the inode.
+                 *
+                 * both the inode and device information are actually hashes
+                 * that are then truncated to the requisite bit length.
+                 *
+                 * it should be okay to use lstat to deal with symlinks.
+                 */
 #ifdef USE_LASTDID
-                    aint = htonl(( st->st_dev << 16 ) | (st->st_ino & 0x0000ffff));
+                aint = htonl(( st->st_dev << 16 ) | (st->st_ino & 0x0000ffff));
 #else /* USE_LASTDID */
-                    lstp = lstat(upath, &lst) < 0 ? st : &lst;
+                lstp = lstat(upath, &lst) < 0 ? st : &lst;
 #ifdef DID_MTAB
-                    aint = htonl( afpd_st_cnid ( lstp ) );
+                aint = htonl( afpd_st_cnid ( lstp ) );
 #else /* DID_MTAB */
-                    aint = htonl(CNID(lstp, 1));
+                aint = htonl(CNID(lstp, 1));
 #endif /* DID_MTAB */
 #endif /* USE_LASTDID */
-                }
+            }
 
-                memcpy(data, &aint, sizeof( aint ));
-                data += sizeof( aint );
-                break;
+            memcpy(data, &aint, sizeof( aint ));
+            data += sizeof( aint );
+            break;
 
-            case FILPBIT_DFLEN :
-                aint = htonl( st->st_size );
-                memcpy(data, &aint, sizeof( aint ));
-                data += sizeof( aint );
-                break;
+        case FILPBIT_DFLEN :
+            aint = htonl( st->st_size );
+            memcpy(data, &aint, sizeof( aint ));
+            data += sizeof( aint );
+            break;
 
-            case FILPBIT_RFLEN :
-                if ( isad ) {
-                    aint = htonl( ad_getentrylen( adp, ADEID_RFORK ));
-                } else {
-                    aint = 0;
-                }
-                memcpy(data, &aint, sizeof( aint ));
-                data += sizeof( aint );
-                break;
+        case FILPBIT_RFLEN :
+            if ( isad ) {
+                aint = htonl( ad_getentrylen( adp, ADEID_RFORK ));
+            } else {
+                aint = 0;
+            }
+            memcpy(data, &aint, sizeof( aint ));
+            data += sizeof( aint );
+            break;
 
-                /* Current client needs ProDOS info block for this file.
-                   Use simple heuristic and let the Mac "type" string tell
-                   us what the PD file code should be.  Everything gets a
-                   subtype of 0x0000 unless the original value was hashed
-                   to "pXYZ" when we created it.  See IA, Ver 2.
-                   <shirsch@ibm.net> */
-            case FILPBIT_PDINFO :
-                if ( isad ) {
-                    memcpy(fdType, ad_entry( adp, ADEID_FINDERI ), 4 );
-
-                    if ( memcmp( fdType, "TEXT", 4 ) == 0 ) {
-                        achar = '\x04';
-                        ashort = 0x0000;
-                    }
-                    else if ( memcmp( fdType, "PSYS", 4 ) == 0 ) {
-                        achar = '\xff';
-                        ashort = 0x0000;
-                    }
-                    else if ( memcmp( fdType, "PS16", 4 ) == 0 ) {
-                        achar = '\xb3';
-                        ashort = 0x0000;
-                    }
-                    else if ( memcmp( fdType, "BINA", 4 ) == 0 ) {
-                        achar = '\x00';
-                        ashort = 0x0000;
-                    }
-                    else if ( fdType[0] == 'p' ) {
-                        achar = fdType[1];
-                        ashort = (fdType[2] * 256) + fdType[3];
-                    }
-                    else {
-                        achar = '\x00';
-                        ashort = 0x0000;
-                    }
+            /* Current client needs ProDOS info block for this file.
+               Use simple heuristic and let the Mac "type" string tell
+               us what the PD file code should be.  Everything gets a
+               subtype of 0x0000 unless the original value was hashed
+               to "pXYZ" when we created it.  See IA, Ver 2.
+               <shirsch@ibm.net> */
+        case FILPBIT_PDINFO :
+            if ( isad ) {
+                memcpy(fdType, ad_entry( adp, ADEID_FINDERI ), 4 );
+
+                if ( memcmp( fdType, "TEXT", 4 ) == 0 ) {
+                    achar = '\x04';
+                    ashort = 0x0000;
+                }
+                else if ( memcmp( fdType, "PSYS", 4 ) == 0 ) {
+                    achar = '\xff';
+                    ashort = 0x0000;
+                }
+                else if ( memcmp( fdType, "PS16", 4 ) == 0 ) {
+                    achar = '\xb3';
+                    ashort = 0x0000;
+                }
+                else if ( memcmp( fdType, "BINA", 4 ) == 0 ) {
+                    achar = '\x00';
+                    ashort = 0x0000;
+                }
+                else if ( fdType[0] == 'p' ) {
+                    achar = fdType[1];
+                    ashort = (fdType[2] * 256) + fdType[3];
                 }
                 else {
                     achar = '\x00';
                     ashort = 0x0000;
                 }
+            }
+            else {
+                achar = '\x00';
+                ashort = 0x0000;
+            }
 
-                *data++ = achar;
-                *data++ = 0;
-                memcpy(data, &ashort, sizeof( ashort ));
-                data += sizeof( ashort );
-                memset(data, 0, sizeof( ashort ));
-                data += sizeof( ashort );
-                break;
+            *data++ = achar;
+            *data++ = 0;
+            memcpy(data, &ashort, sizeof( ashort ));
+            data += sizeof( ashort );
+            memset(data, 0, sizeof( ashort ));
+            data += sizeof( ashort );
+            break;
 
-            default :
-                if ( isad ) {
-                    ad_close( adp, ADFLAGS_HF );
-                }
-                return( AFPERR_BITMAP );
+        default :
+            if ( isad ) {
+                ad_close( adp, ADFLAGS_HF );
             }
-            bitmap = bitmap>>1;
-            bit++;
+            return( AFPERR_BITMAP );
         }
-        if ( nameoff ) {
-            ashort = htons( data - buf );
-            memcpy(nameoff, &ashort, sizeof( ashort ));
-            if ((aint = strlen( path )) > MACFILELEN)
-                aint = MACFILELEN;
-            *data++ = aint;
-            memcpy(data, path, aint );
-            data += aint;
-        }
-        if ( isad ) {
-            ad_close( adp, ADFLAGS_HF );
-        }
-        *buflen = data - buf;
+        bitmap = bitmap>>1;
+        bit++;
+    }
+    if ( nameoff ) {
+        ashort = htons( data - buf );
+        memcpy(nameoff, &ashort, sizeof( ashort ));
+        if ((aint = strlen( path )) > MACFILELEN)
+            aint = MACFILELEN;
+        *data++ = aint;
+        memcpy(data, path, aint );
+        data += aint;
+    }
+    if ( isad ) {
+        ad_close( adp, ADFLAGS_HF );
+    }
+    *buflen = data - buf;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end getfilparams:");
+    LOG(log_info, logtype_default, "end getfilparams:");
 #endif /* DEBUG */
 
-        return( AFP_OK );
-    }
-
-    int afp_createfile(obj, ibuf, ibuflen, rbuf, rbuflen )
-    AFPObj      *obj;
-    char       *ibuf, *rbuf;
-    int                ibuflen, *rbuflen;
-    {
-        struct stat         st;
-        struct adouble ad, *adp;
-        struct vol             *vol;
-        struct dir             *dir;
-        struct ofork        *of;
-        char           *path, *upath;
-        int                    creatf, did, openf, retvalue = AFP_OK;
-        u_int16_t              vid;
+    return( AFP_OK );
+}
+
+int afp_createfile(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj      *obj;
+char   *ibuf, *rbuf;
+int            ibuflen, *rbuflen;
+{
+    struct stat         st;
+    struct adouble     ad, *adp;
+    struct vol         *vol;
+    struct dir         *dir;
+    struct ofork        *of;
+    char               *path, *upath;
+    int                        creatf, did, openf, retvalue = AFP_OK;
+    u_int16_t          vid;
 #ifdef FORCE_UIDGID
-        uidgidset              *uidgid;
+    uidgidset          *uidgid;
 #endif /* FORCE_UIDGID */
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin afp_createfile:");
+    LOG(log_info, logtype_default, "begin afp_createfile:");
 #endif /* DEBUG */
 
-        *rbuflen = 0;
-        ibuf++;
-        creatf = (unsigned char) *ibuf++;
+    *rbuflen = 0;
+    ibuf++;
+    creatf = (unsigned char) *ibuf++;
 
-        memcpy(&vid, ibuf, sizeof( vid ));
-        ibuf += sizeof( vid );
+    memcpy(&vid, ibuf, sizeof( vid ));
+    ibuf += sizeof( vid );
 
-        if (( vol = getvolbyvid( vid )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( vol = getvolbyvid( vid )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if (vol->v_flags & AFPVOL_RO)
-            return AFPERR_VLOCK;
+    if (vol->v_flags & AFPVOL_RO)
+        return AFPERR_VLOCK;
 
-        memcpy(&did, ibuf, sizeof( did));
-        ibuf += sizeof( did );
+    memcpy(&did, ibuf, sizeof( did));
+    ibuf += sizeof( did );
 
-        if (( dir = dirsearch( vol, did )) == NULL ) {
-            return( AFPERR_NOOBJ );
-        }
+    if (( dir = dirsearch( vol, did )) == NULL ) {
+        return( AFPERR_NOOBJ );
+    }
 
-        if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-            return( AFPERR_NOOBJ );
-        }
+    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
+        return( AFPERR_NOOBJ );
+    }
 
-        if (!wincheck(vol, path))
-            return AFPERR_PARAM;
+    if (!wincheck(vol, path))
+        return AFPERR_PARAM;
 
-        upath = mtoupath(vol, path);
+    upath = mtoupath(vol, path);
 
-        if ((vol->v_flags & AFPVOL_NOHEX) && strchr(upath, '/'))
-            return AFPERR_PARAM;
+    if ((vol->v_flags & AFPVOL_NOHEX) && strchr(upath, '/'))
+        return AFPERR_PARAM;
 
-        if (!validupath(vol, upath))
-            return AFPERR_EXIST;
+    if (!validupath(vol, upath))
+        return AFPERR_EXIST;
 
-        /* check for vetoed filenames */
-        if (veto_file(vol->v_veto, upath))
-            return AFPERR_EXIST;
+    /* check for vetoed filenames */
+    if (veto_file(vol->v_veto, upath))
+        return AFPERR_EXIST;
 
-        if ((of = of_findname(vol, curdir, path))) {
-            adp = of->of_ad;
-        } else {
-            memset(&ad, 0, sizeof(ad));
-            adp = &ad;
-        }
-        if ( creatf) {
-            /* on a hard create, fail if file exists and is open */
-            if ((stat(upath, &st) == 0) && of)
-                return AFPERR_BUSY;
-            openf = O_RDWR|O_CREAT|O_TRUNC;
-        } else {
-            openf = O_RDWR|O_CREAT|O_EXCL;
-        }
+    if ((of = of_findname(vol, curdir, path))) {
+        adp = of->of_ad;
+    } else {
+        memset(&ad, 0, sizeof(ad));
+        adp = &ad;
+    }
+    if ( creatf) {
+        /* on a hard create, fail if file exists and is open */
+        if ((stat(upath, &st) == 0) && of)
+            return AFPERR_BUSY;
+        openf = O_RDWR|O_CREAT|O_TRUNC;
+    } else {
+        openf = O_RDWR|O_CREAT|O_EXCL;
+    }
 
 #ifdef FORCE_UIDGID
 
-        /* preserve current euid, egid */
-        save_uidgid ( uidgid );
+    /* preserve current euid, egid */
+    save_uidgid ( uidgid );
 
-        /* perform all switching of users */
-        set_uidgid ( vol );
+    /* perform all switching of users */
+    set_uidgid ( vol );
 
 #endif /* FORCE_UIDGID */
 
-        if ( ad_open( upath, vol_noadouble(vol)|ADFLAGS_DF|ADFLAGS_HF,
-                      openf, 0666, adp) < 0 ) {
-            switch ( errno ) {
-            case EEXIST :
+    if ( ad_open( upath, vol_noadouble(vol)|ADFLAGS_DF|ADFLAGS_HF,
+                  openf, 0666, adp) < 0 ) {
+        switch ( errno ) {
+        case EEXIST :
 #ifdef FORCE_UIDGID
-                /* bring everything back to old euid, egid */
-                restore_uidgid ( uidgid );
+            /* bring everything back to old euid, egid */
+            restore_uidgid ( uidgid );
 #endif /* FORCE_UIDGID */
-                return( AFPERR_EXIST );
-            case EACCES :
+            return( AFPERR_EXIST );
+        case EACCES :
 #ifdef FORCE_UIDGID
-                /* bring everything back to old euid, egid */
-                restore_uidgid ( uidgid );
+            /* bring everything back to old euid, egid */
+            restore_uidgid ( uidgid );
 #endif /* FORCE_UIDGID */
-                return( AFPERR_ACCESS );
-            case ENOENT:
-                /* on noadouble volumes, just creating the data fork is ok */
-                if (vol_noadouble(vol) && (stat(upath, &st) == 0))
-                    goto createfile_done;
-                /* fallthrough */
-            default :
+            return( AFPERR_ACCESS );
+        case ENOENT:
+            /* on noadouble volumes, just creating the data fork is ok */
+            if (vol_noadouble(vol) && (stat(upath, &st) == 0))
+                goto createfile_done;
+            /* fallthrough */
+        default :
 #ifdef FORCE_UIDGID
-                /* bring everything back to old euid, egid */
-                restore_uidgid ( uidgid );
+            /* bring everything back to old euid, egid */
+            restore_uidgid ( uidgid );
 #endif /* FORCE_UIDGID */
-                return( AFPERR_PARAM );
-            }
+            return( AFPERR_PARAM );
         }
+    }
 
-        ad_setentrylen( adp, ADEID_NAME, strlen( path ));
-        memcpy(ad_entry( adp, ADEID_NAME ), path,
-               ad_getentrylen( adp, ADEID_NAME ));
-        ad_flush( adp, ADFLAGS_DF|ADFLAGS_HF );
-        ad_close( adp, ADFLAGS_DF|ADFLAGS_HF );
+    ad_setentrylen( adp, ADEID_NAME, strlen( path ));
+    memcpy(ad_entry( adp, ADEID_NAME ), path,
+           ad_getentrylen( adp, ADEID_NAME ));
+    ad_flush( adp, ADFLAGS_DF|ADFLAGS_HF );
+    ad_close( adp, ADFLAGS_DF|ADFLAGS_HF );
 
 createfile_done:
 
 #ifdef DROPKLUDGE
-        if (vol->v_flags & AFPVOL_DROPBOX) {
-            retvalue = matchfile2dirperms(upath, vol, did);
-        }
+    if (vol->v_flags & AFPVOL_DROPBOX) {
+        retvalue = matchfile2dirperms(upath, vol, did);
+    }
 #endif /* DROPKLUDGE */
 
-        setvoltime(obj, vol );
+    setvoltime(obj, vol );
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end afp_createfile");
+    LOG(log_info, logtype_default, "end afp_createfile");
 #endif /* DEBUG */
 
 #ifdef FORCE_UIDGID
-        /* bring everything back to old euid, egid */
-        restore_uidgid ( uidgid );
+    /* bring everything back to old euid, egid */
+    restore_uidgid ( uidgid );
 #endif /* FORCE_UIDGID */
 
-        return (retvalue);
-    }
+    return (retvalue);
+}
 
-    int afp_setfilparams(obj, ibuf, ibuflen, rbuf, rbuflen )
-    AFPObj      *obj;
-    char       *ibuf, *rbuf;
-    int                ibuflen, *rbuflen;
-    {
-        struct vol     *vol;
-        struct dir     *dir;
-        char   *path;
-        int            did, rc;
-        u_int16_t      vid, bitmap;
+int afp_setfilparams(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj      *obj;
+char   *ibuf, *rbuf;
+int            ibuflen, *rbuflen;
+{
+    struct vol *vol;
+    struct dir *dir;
+    char       *path;
+    int                did, rc;
+    u_int16_t  vid, bitmap;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin afp_setfilparams:");
+    LOG(log_info, logtype_default, "begin afp_setfilparams:");
 #endif /* DEBUG */
 
-        *rbuflen = 0;
-        ibuf += 2;
+    *rbuflen = 0;
+    ibuf += 2;
 
-        memcpy(&vid, ibuf, sizeof( vid ));
-        ibuf += sizeof( vid );
-        if (( vol = getvolbyvid( vid )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    memcpy(&vid, ibuf, sizeof( vid ));
+    ibuf += sizeof( vid );
+    if (( vol = getvolbyvid( vid )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if (vol->v_flags & AFPVOL_RO)
-            return AFPERR_VLOCK;
+    if (vol->v_flags & AFPVOL_RO)
+        return AFPERR_VLOCK;
 
-        memcpy(&did, ibuf, sizeof( did ));
-        ibuf += sizeof( did );
-        if (( dir = dirsearch( vol, did )) == NULL ) {
-            return( AFPERR_NOOBJ );
-        }
+    memcpy(&did, ibuf, sizeof( did ));
+    ibuf += sizeof( did );
+    if (( dir = dirsearch( vol, did )) == NULL ) {
+        return( AFPERR_NOOBJ );
+    }
 
-        memcpy(&bitmap, ibuf, sizeof( bitmap ));
-        bitmap = ntohs( bitmap );
-        ibuf += sizeof( bitmap );
+    memcpy(&bitmap, ibuf, sizeof( bitmap ));
+    bitmap = ntohs( bitmap );
+    ibuf += sizeof( bitmap );
 
-        if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-            return( AFPERR_NOOBJ );
-        }
+    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
+        return( AFPERR_NOOBJ );
+    }
 
-        if ((u_long)ibuf & 1 ) {
-            ibuf++;
-        }
+    if ((u_long)ibuf & 1 ) {
+        ibuf++;
+    }
 
-        if (( rc = setfilparams(vol, path, bitmap, ibuf )) == AFP_OK ) {
-            setvoltime(obj, vol );
-        }
+    if (( rc = setfilparams(vol, path, bitmap, ibuf )) == AFP_OK ) {
+        setvoltime(obj, vol );
+    }
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end afp_setfilparams:");
+    LOG(log_info, logtype_default, "end afp_setfilparams:");
 #endif /* DEBUG */
 
-        return( rc );
-    }
+    return( rc );
+}
 
 
-    int setfilparams(struct vol *vol,
-                     char *path, u_int16_t bitmap, char *buf )
-    {
-        struct adouble ad, *adp;
-        struct ofork        *of;
-        struct extmap  *em;
-        int                    bit = 0, isad = 1, err = AFP_OK;
-        char                *upath;
-        u_char              achar, *fdType, xyy[4];
-        u_int16_t              ashort, bshort;
-        u_int32_t              aint;
-        struct utimbuf ut;
+int setfilparams(struct vol *vol,
+                 char *path, u_int16_t bitmap, char *buf )
+{
+    struct adouble     ad, *adp;
+    struct ofork        *of;
+    struct extmap      *em;
+    int                        bit = 0, isad = 1, err = AFP_OK;
+    char                *upath;
+    u_char              achar, *fdType, xyy[4];
+    u_int16_t          ashort, bshort;
+    u_int32_t          aint;
+    struct utimbuf     ut;
 
 #ifdef FORCE_UIDGID
-        uidgidset              *uidgid;
+    uidgidset          *uidgid;
 
-        uidgid = malloc(sizeof(uidgidset));
+    uidgid = malloc(sizeof(uidgidset));
 #endif /* FORCE_UIDGID */
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin setfilparams:");
+    LOG(log_info, logtype_default, "begin setfilparams:");
 #endif /* DEBUG */
 
-        upath = mtoupath(vol, path);
-        if ((of = of_findname(vol, curdir, path))) {
-            adp = of->of_ad;
-        } else {
-            memset(&ad, 0, sizeof(ad));
-            adp = &ad;
-        }
+    upath = mtoupath(vol, path);
+    if ((of = of_findname(vol, curdir, path))) {
+        adp = of->of_ad;
+    } else {
+        memset(&ad, 0, sizeof(ad));
+        adp = &ad;
+    }
 
 #ifdef FORCE_UIDGID
-        save_uidgid ( uidgid );
-        set_uidgid ( vol );
+    save_uidgid ( uidgid );
+    set_uidgid ( vol );
 #endif /* FORCE_UIDGID */
 
-        if (ad_open( upath, vol_noadouble(vol) | ADFLAGS_HF,
-                     O_RDWR|O_CREAT, 0666, adp) < 0) {
-            /* for some things, we don't need an adouble header */
-            if (bitmap & ~(1<<FILPBIT_MDATE)) {
+    if (ad_open( upath, vol_noadouble(vol) | ADFLAGS_HF,
+                 O_RDWR|O_CREAT, 0666, adp) < 0) {
+        /* for some things, we don't need an adouble header */
+        if (bitmap & ~(1<<FILPBIT_MDATE)) {
 #ifdef FORCE_UIDGID
-                restore_uidgid ( uidgid );
+            restore_uidgid ( uidgid );
 #endif /* FORCE_UIDGID */
-                return vol_noadouble(vol) ? AFP_OK : AFPERR_ACCESS;
-            }
-            isad = 0;
-        } else if ((ad_getoflags( adp, ADFLAGS_HF ) & O_CREAT) ) {
-            ad_setentrylen( adp, ADEID_NAME, strlen( path ));
-            memcpy(ad_entry( adp, ADEID_NAME ), path,
-                   ad_getentrylen( adp, ADEID_NAME ));
+            return vol_noadouble(vol) ? AFP_OK : AFPERR_ACCESS;
+        }
+        isad = 0;
+    } else if ((ad_getoflags( adp, ADFLAGS_HF ) & O_CREAT) ) {
+        ad_setentrylen( adp, ADEID_NAME, strlen( path ));
+        memcpy(ad_entry( adp, ADEID_NAME ), path,
+               ad_getentrylen( adp, ADEID_NAME ));
+    }
+
+    while ( bitmap != 0 ) {
+        while (( bitmap & 1 ) == 0 ) {
+            bitmap = bitmap>>1;
+            bit++;
         }
 
-        while ( bitmap != 0 ) {
-            while (( bitmap & 1 ) == 0 ) {
-                bitmap = bitmap>>1;
-                bit++;
+        switch(  bit ) {
+        case FILPBIT_ATTR :
+            memcpy(&ashort, buf, sizeof( ashort ));
+            ad_getattr(adp, &bshort);
+            if ( ntohs( ashort ) & ATTRBIT_SETCLR ) {
+                bshort |= htons( ntohs( ashort ) & ~ATTRBIT_SETCLR );
+            } else {
+                bshort &= ~ashort;
             }
+            ad_setattr(adp, bshort);
+            buf += sizeof( ashort );
+            break;
 
-            switch(  bit ) {
-            case FILPBIT_ATTR :
-                memcpy(&ashort, buf, sizeof( ashort ));
-                ad_getattr(adp, &bshort);
-                if ( ntohs( ashort ) & ATTRBIT_SETCLR ) {
-                    bshort |= htons( ntohs( ashort ) & ~ATTRBIT_SETCLR );
-                } else {
-                    bshort &= ~ashort;
-                }
-                ad_setattr(adp, bshort);
-                buf += sizeof( ashort );
-                break;
+        case FILPBIT_CDATE :
+            memcpy(&aint, buf, sizeof(aint));
+            ad_setdate(adp, AD_DATE_CREATE, aint);
+            buf += sizeof( aint );
+            break;
 
-            case FILPBIT_CDATE :
-                memcpy(&aint, buf, sizeof(aint));
-                ad_setdate(adp, AD_DATE_CREATE, aint);
-                buf += sizeof( aint );
-                break;
+        case FILPBIT_MDATE :
+            memcpy(&aint, buf, sizeof( aint ));
+            if (isad)
+                ad_setdate(adp, AD_DATE_MODIFY, aint);
+            ut.actime = ut.modtime = AD_DATE_TO_UNIX(aint);
+            utime(upath, &ut);
+            buf += sizeof( aint );
+            break;
 
-            case FILPBIT_MDATE :
-                memcpy(&aint, buf, sizeof( aint ));
-                if (isad)
-                    ad_setdate(adp, AD_DATE_MODIFY, aint);
-                ut.actime = ut.modtime = AD_DATE_TO_UNIX(aint);
-                utime(upath, &ut);
-                buf += sizeof( aint );
-                break;
+        case FILPBIT_BDATE :
+            memcpy(&aint, buf, sizeof(aint));
+            ad_setdate(adp, AD_DATE_BACKUP, aint);
+            buf += sizeof( aint );
+            break;
 
-            case FILPBIT_BDATE :
-                memcpy(&aint, buf, sizeof(aint));
-                ad_setdate(adp, AD_DATE_BACKUP, aint);
-                buf += sizeof( aint );
-                break;
+        case FILPBIT_FINFO :
+            if ((memcmp( ad_entry( adp, ADEID_FINDERI ), ufinderi, 8 ) == 0)
+                    && (em = getextmap( path )) &&
+                    (memcmp(buf, em->em_type, sizeof( em->em_type )) == 0) &&
+                    (memcmp(buf + 4, em->em_creator,
+                            sizeof( em->em_creator )) == 0)) {
+                memcpy(buf, ufinderi, 8 );
+            }
+            memcpy(ad_entry( adp, ADEID_FINDERI ), buf, 32 );
+            buf += 32;
+            break;
 
-            case FILPBIT_FINFO :
-                if ((memcmp( ad_entry( adp, ADEID_FINDERI ), ufinderi, 8 ) == 0)
-                        && (em = getextmap( path )) &&
-                        (memcmp(buf, em->em_type, sizeof( em->em_type )) == 0) &&
-                        (memcmp(buf + 4, em->em_creator,
-                                sizeof( em->em_creator )) == 0)) {
-                    memcpy(buf, ufinderi, 8 );
-                }
-                memcpy(ad_entry( adp, ADEID_FINDERI ), buf, 32 );
-                buf += 32;
+            /* Client needs to set the ProDOS file info for this file.
+               Use defined strings for the simple cases, and convert
+               all else into pXYY per Inside Appletalk.  Always set
+               the creator as "pdos". <shirsch@ibm.net> */
+        case FILPBIT_PDINFO :
+            achar = *buf;
+            buf += 2;
+            memcpy(&ashort, buf, sizeof( ashort ));
+            ashort = ntohs( ashort );
+            buf += 2;
+
+            switch ( (unsigned int) achar )
+            {
+            case 0x04 :
+                fdType = ( u_char *) "TEXT";
                 break;
 
-                /* Client needs to set the ProDOS file info for this file.
-                   Use defined strings for the simple cases, and convert
-                   all else into pXYY per Inside Appletalk.  Always set
-                   the creator as "pdos". <shirsch@ibm.net> */
-            case FILPBIT_PDINFO :
-                achar = *buf;
-                buf += 2;
-                memcpy(&ashort, buf, sizeof( ashort ));
-                ashort = ntohs( ashort );
-                buf += 2;
-
-                switch ( (unsigned int) achar )
-                {
-                case 0x04 :
-                    fdType = ( u_char *) "TEXT";
-                    break;
-
-                case 0xff :
-                    fdType = ( u_char *) "PSYS";
-                    break;
-
-                case 0xb3 :
-                    fdType = ( u_char *) "PS16";
-                    break;
-
-                case 0x00 :
-                    fdType = ( u_char *) "BINA";
-                    break;
-
-                default :
-                    xyy[0] = ( u_char ) 'p';
-                    xyy[1] = achar;
-                    xyy[2] = ( u_char ) ( ashort >> 8 ) & 0xff;
-                    xyy[3] = ( u_char ) ashort & 0xff;
-                    fdType = xyy;
-                    break;
-                }
+            case 0xff :
+                fdType = ( u_char *) "PSYS";
+                break;
 
-                memcpy(ad_entry( adp, ADEID_FINDERI ), fdType, 4 );
-                memcpy(ad_entry( adp, ADEID_FINDERI ) + 4, "pdos", 4 );
+            case 0xb3 :
+                fdType = ( u_char *) "PS16";
                 break;
 
+            case 0x00 :
+                fdType = ( u_char *) "BINA";
+                break;
 
             default :
-                err = AFPERR_BITMAP;
-                goto setfilparam_done;
+                xyy[0] = ( u_char ) 'p';
+                xyy[1] = achar;
+                xyy[2] = ( u_char ) ( ashort >> 8 ) & 0xff;
+                xyy[3] = ( u_char ) ashort & 0xff;
+                fdType = xyy;
+                break;
             }
 
-            bitmap = bitmap>>1;
-            bit++;
+            memcpy(ad_entry( adp, ADEID_FINDERI ), fdType, 4 );
+            memcpy(ad_entry( adp, ADEID_FINDERI ) + 4, "pdos", 4 );
+            break;
+
+
+        default :
+            err = AFPERR_BITMAP;
+            goto setfilparam_done;
         }
 
+        bitmap = bitmap>>1;
+        bit++;
+    }
+
 setfilparam_done:
-        if (isad) {
-            ad_flush( adp, ADFLAGS_HF );
-            ad_close( adp, ADFLAGS_HF );
+    if (isad) {
+        ad_flush( adp, ADFLAGS_HF );
+        ad_close( adp, ADFLAGS_HF );
 
 #ifdef FORCE_UIDGID
-            restore_uidgid ( uidgid );
+        restore_uidgid ( uidgid );
 #endif /* FORCE_UIDGID */
 
-        }
+    }
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end setfilparams:");
+    LOG(log_info, logtype_default, "end setfilparams:");
 #endif /* DEBUG */
 
-        return err;
-    }
+    return err;
+}
+
+/*
+ * renamefile and copyfile take the old and new unix pathnames
+ * and the new mac name.
+ * NOTE: if we have to copy a file instead of renaming it, locks
+ *       will break.
+ */
+int renamefile(src, dst, newname, noadouble )
+char   *src, *dst, *newname;
+const int         noadouble;
+{
+    struct adouble     ad;
+    char               adsrc[ MAXPATHLEN + 1];
+    int                        len, rc;
 
     /*
-     * renamefile and copyfile take the old and new unix pathnames
-     * and the new mac name.
-     * NOTE: if we have to copy a file instead of renaming it, locks
-     *       will break.
+     * Note that this is only checking the existance of the data file,
+     * not the header file.  The thinking is that if the data file doesn't
+     * exist, but the header file does, the right thing to do is remove
+     * the data file silently.
      */
-    int renamefile(src, dst, newname, noadouble )
-    char       *src, *dst, *newname;
-    const int         noadouble;
-    {
-        struct adouble ad;
-        char           adsrc[ MAXPATHLEN + 1];
-        int                    len, rc;
-
-        /*
-         * Note that this is only checking the existance of the data file,
-         * not the header file.  The thinking is that if the data file doesn't
-         * exist, but the header file does, the right thing to do is remove
-         * the data file silently.
-         */
 
-        /* existence check moved to afp_moveandrename */
+    /* existence check moved to afp_moveandrename */
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin renamefile:");
+    LOG(log_info, logtype_default, "begin renamefile:");
 #endif /* DEBUG */
 
-        if ( rename( src, dst ) < 0 ) {
-            switch ( errno ) {
-            case ENOENT :
-                return( AFPERR_NOOBJ );
-            case EPERM:
-            case EACCES :
-                return( AFPERR_ACCESS );
-            case EROFS:
-                return AFPERR_VLOCK;
-            case EXDEV :                       /* Cross device move -- try copy */
-                if (( rc = copyfile(src, dst, newname, noadouble )) != AFP_OK ) {
-                    deletefile( dst );
-                    return( rc );
-                }
-                return deletefile( src );
-            default :
-                return( AFPERR_PARAM );
+    if ( rename( src, dst ) < 0 ) {
+        switch ( errno ) {
+        case ENOENT :
+            return( AFPERR_NOOBJ );
+        case EPERM:
+        case EACCES :
+            return( AFPERR_ACCESS );
+        case EROFS:
+            return AFPERR_VLOCK;
+        case EXDEV :                   /* Cross device move -- try copy */
+            if (( rc = copyfile(src, dst, newname, noadouble )) != AFP_OK ) {
+                deletefile( dst );
+                return( rc );
             }
+            return deletefile( src );
+        default :
+            return( AFPERR_PARAM );
         }
+    }
 
-        strcpy( adsrc, ad_path( src, 0 ));
-        rc = 0;
+    strcpy( adsrc, ad_path( src, 0 ));
+    rc = 0;
 rename_retry:
-        if (rename( adsrc, ad_path( dst, 0 )) < 0 ) {
-            struct stat st;
+    if (rename( adsrc, ad_path( dst, 0 )) < 0 ) {
+        struct stat st;
 
-            switch ( errno ) {
-            case ENOENT :
-                /* check for a source appledouble header. if it exists, make
-                 * a dest appledouble directory and do the rename again. */
-                memset(&ad, 0, sizeof(ad));
-                if (rc || stat(adsrc, &st) ||
-                        (ad_open(dst, ADFLAGS_HF, O_RDWR | O_CREAT, 0666, &ad) < 0))
-                    return AFP_OK;
-                rc++;
-                ad_close(&ad, ADFLAGS_HF);
-                goto rename_retry;
-            case EPERM:
-            case EACCES :
-                return( AFPERR_ACCESS );
-            case EROFS:
-                return AFPERR_VLOCK;
-            default :
-                return( AFPERR_PARAM );
-            }
+        switch ( errno ) {
+        case ENOENT :
+            /* check for a source appledouble header. if it exists, make
+             * a dest appledouble directory and do the rename again. */
+            memset(&ad, 0, sizeof(ad));
+            if (rc || stat(adsrc, &st) ||
+                    (ad_open(dst, ADFLAGS_HF, O_RDWR | O_CREAT, 0666, &ad) < 0))
+                return AFP_OK;
+            rc++;
+            ad_close(&ad, ADFLAGS_HF);
+            goto rename_retry;
+        case EPERM:
+        case EACCES :
+            return( AFPERR_ACCESS );
+        case EROFS:
+            return AFPERR_VLOCK;
+        default :
+            return( AFPERR_PARAM );
         }
+    }
 
-        memset(&ad, 0, sizeof(ad));
-        if ( ad_open( dst, ADFLAGS_HF, O_RDWR, 0666, &ad) < 0 ) {
-            switch ( errno ) {
-            case ENOENT :
-                return( AFPERR_NOOBJ );
-            case EACCES :
-                return( AFPERR_ACCESS );
-            case EROFS:
-                return AFPERR_VLOCK;
-            default :
-                return( AFPERR_PARAM );
-            }
+    memset(&ad, 0, sizeof(ad));
+    if ( ad_open( dst, ADFLAGS_HF, O_RDWR, 0666, &ad) < 0 ) {
+        switch ( errno ) {
+        case ENOENT :
+            return( AFPERR_NOOBJ );
+        case EACCES :
+            return( AFPERR_ACCESS );
+        case EROFS:
+            return AFPERR_VLOCK;
+        default :
+            return( AFPERR_PARAM );
         }
+    }
 
-        len = strlen( newname );
-        ad_setentrylen( &ad, ADEID_NAME, len );
-        memcpy(ad_entry( &ad, ADEID_NAME ), newname, len );
-        ad_flush( &ad, ADFLAGS_HF );
-        ad_close( &ad, ADFLAGS_HF );
+    len = strlen( newname );
+    ad_setentrylen( &ad, ADEID_NAME, len );
+    memcpy(ad_entry( &ad, ADEID_NAME ), newname, len );
+    ad_flush( &ad, ADFLAGS_HF );
+    ad_close( &ad, ADFLAGS_HF );
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end renamefile:");
+    LOG(log_info, logtype_default, "end renamefile:");
 #endif /* DEBUG */
 
-        return( AFP_OK );
-    }
+    return( AFP_OK );
+}
 
-    int afp_copyfile(obj, ibuf, ibuflen, rbuf, rbuflen )
-    AFPObj      *obj;
-    char       *ibuf, *rbuf;
-    int                ibuflen, *rbuflen;
-    {
-        struct vol     *vol;
-        struct dir     *dir;
-        char   *newname, *path, *p;
-        u_int32_t      sdid, ddid;
-        int            plen, err, retvalue = AFP_OK;
-        u_int16_t      svid, dvid;
+int afp_copyfile(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj      *obj;
+char   *ibuf, *rbuf;
+int            ibuflen, *rbuflen;
+{
+    struct vol *vol;
+    struct dir *dir;
+    char       *newname, *path, *p;
+    u_int32_t  sdid, ddid;
+    int                plen, err, retvalue = AFP_OK;
+    u_int16_t  svid, dvid;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin afp_copyfile:");
+    LOG(log_info, logtype_default, "begin afp_copyfile:");
 #endif /* DEBUG */
 
-        *rbuflen = 0;
-        ibuf += 2;
+    *rbuflen = 0;
+    ibuf += 2;
 
-        memcpy(&svid, ibuf, sizeof( svid ));
-        ibuf += sizeof( svid );
-        if (( vol = getvolbyvid( svid )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    memcpy(&svid, ibuf, sizeof( svid ));
+    ibuf += sizeof( svid );
+    if (( vol = getvolbyvid( svid )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        memcpy(&sdid, ibuf, sizeof( sdid ));
-        ibuf += sizeof( sdid );
-        if (( dir = dirsearch( vol, sdid )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    memcpy(&sdid, ibuf, sizeof( sdid ));
+    ibuf += sizeof( sdid );
+    if (( dir = dirsearch( vol, sdid )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        memcpy(&dvid, ibuf, sizeof( dvid ));
-        ibuf += sizeof( dvid );
-        memcpy(&ddid, ibuf, sizeof( ddid ));
-        ibuf += sizeof( ddid );
+    memcpy(&dvid, ibuf, sizeof( dvid ));
+    ibuf += sizeof( dvid );
+    memcpy(&ddid, ibuf, sizeof( ddid ));
+    ibuf += sizeof( ddid );
 
-        if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-            return( AFPERR_NOOBJ );
-        }
-        if ( *path == '\0' ) {
-            return( AFPERR_BADTYPE );
-        }
+    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
+        return( AFPERR_NOOBJ );
+    }
+    if ( *path == '\0' ) {
+        return( AFPERR_BADTYPE );
+    }
 
-        /* don't allow copies when the file is open.
-         * XXX: the spec only calls for read/deny write access.
-         *      however, copyfile doesn't have any of that info,
-         *      and locks need to stay coherent. as a result,
-         *      we just balk if the file is opened already. */
-        if (of_findname(vol, curdir, path))
-            return AFPERR_DENYCONF;
+    /* don't allow copies when the file is open.
+     * XXX: the spec only calls for read/deny write access.
+     *      however, copyfile doesn't have any of that info,
+     *      and locks need to stay coherent. as a result,
+     *      we just balk if the file is opened already. */
+    if (of_findname(vol, curdir, path))
+        return AFPERR_DENYCONF;
 
-        newname = obj->newtmp;
-        strcpy( newname, path );
+    newname = obj->newtmp;
+    strcpy( newname, path );
 
-        p = ctoupath( vol, curdir, newname );
+    p = ctoupath( vol, curdir, newname );
 
-        if (( vol = getvolbyvid( dvid )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( vol = getvolbyvid( dvid )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if (vol->v_flags & AFPVOL_RO)
-            return AFPERR_VLOCK;
+    if (vol->v_flags & AFPVOL_RO)
+        return AFPERR_VLOCK;
 
-        if (( dir = dirsearch( vol, ddid )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( dir = dirsearch( vol, ddid )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-            return( AFPERR_NOOBJ );
-        }
-        if ( *path != '\0' ) {
-            return( AFPERR_BADTYPE );
-        }
+    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
+        return( AFPERR_NOOBJ );
+    }
+    if ( *path != '\0' ) {
+        return( AFPERR_BADTYPE );
+    }
 
-        /* one of the handful of places that knows about the path type */
-        if ( *ibuf++ != 2 ) {
-            return( AFPERR_PARAM );
-        }
-        if (( plen = (unsigned char)*ibuf++ ) != 0 ) {
-            strncpy( newname, ibuf, plen );
-            newname[ plen ] = '\0';
-        }
+    /* one of the handful of places that knows about the path type */
+    if ( *ibuf++ != 2 ) {
+        return( AFPERR_PARAM );
+    }
+    if (( plen = (unsigned char)*ibuf++ ) != 0 ) {
+        strncpy( newname, ibuf, plen );
+        newname[ plen ] = '\0';
+    }
 
-        if ( (err = copyfile(p, mtoupath(vol, newname ), newname,
-                             vol_noadouble(vol))) < 0 ) {
-            return err;
-        }
+    if ( (err = copyfile(p, mtoupath(vol, newname ), newname,
+                         vol_noadouble(vol))) < 0 ) {
+        return err;
+    }
 
-        setvoltime(obj, vol );
+    setvoltime(obj, vol );
 
 #ifdef DROPKLUDGE
-        if (vol->v_flags & AFPVOL_DROPBOX) {
-            retvalue=matchfile2dirperms(newname, vol, sdid);
-        }
+    if (vol->v_flags & AFPVOL_DROPBOX) {
+        retvalue=matchfile2dirperms(newname, vol, sdid);
+    }
 #endif /* DROPKLUDGE */
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end afp_copyfile:");
+    LOG(log_info, logtype_default, "end afp_copyfile:");
 #endif /* DEBUG */
 
-        return( retvalue );
-    }
+    return( retvalue );
+}
 
 
-    static __inline__ int copy_all(const int dfd, const void *buf,
-                                   size_t buflen)
-    {
-        ssize_t cc;
+static __inline__ int copy_all(const int dfd, const void *buf,
+                               size_t buflen)
+{
+    ssize_t cc;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin copy_all:");
+    LOG(log_info, logtype_default, "begin copy_all:");
 #endif /* DEBUG */
 
-        while (buflen > 0) {
-            if ((cc = write(dfd, buf, buflen)) < 0) {
-                switch (errno) {
-                case EINTR:
-                    continue;
-                case EDQUOT:
-                case EFBIG:
-                case ENOSPC:
-                    return AFPERR_DFULL;
-                case EROFS:
-                    return AFPERR_VLOCK;
-                default:
-                    return AFPERR_PARAM;
-                }
+    while (buflen > 0) {
+        if ((cc = write(dfd, buf, buflen)) < 0) {
+            switch (errno) {
+            case EINTR:
+                continue;
+            case EDQUOT:
+            case EFBIG:
+            case ENOSPC:
+                return AFPERR_DFULL;
+            case EROFS:
+                return AFPERR_VLOCK;
+            default:
+                return AFPERR_PARAM;
             }
-            buflen -= cc;
         }
+        buflen -= cc;
+    }
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end copy_all:");
+    LOG(log_info, logtype_default, "end copy_all:");
 #endif /* DEBUG */
 
-        return AFP_OK;
-    }
+    return AFP_OK;
+}
 
-    /* XXX: this needs to use ad_open and ad_lock. so, we need to
    * pass in vol and path */
-    int copyfile(src, dst, newname, noadouble )
-    char       *src, *dst, *newname;
-    const int   noadouble;
-    {
-        struct adouble ad;
-        struct stat         st;
-        char           filebuf[8192];
-        int                    sfd, dfd, len, err = AFP_OK;
-        ssize_t             cc;
+/* XXX: this needs to use ad_open and ad_lock. so, we need to
+ * pass in vol and path */
+int copyfile(src, dst, newname, noadouble )
+char   *src, *dst, *newname;
+const int   noadouble;
+{
+    struct adouble     ad;
+    struct stat         st;
+    char               filebuf[8192];
+    int                        sfd, dfd, len, err = AFP_OK;
+    ssize_t             cc;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin copyfile:");
+    LOG(log_info, logtype_default, "begin copyfile:");
 #endif /* DEBUG */
 
-        if (newname) {
-            if ((sfd = open( ad_path( src, ADFLAGS_HF ), O_RDONLY, 0 )) < 0 ) {
+    if (newname) {
+        if ((sfd = open( ad_path( src, ADFLAGS_HF ), O_RDONLY, 0 )) < 0 ) {
+            switch ( errno ) {
+            case ENOENT :
+                break; /* just copy the data fork */
+            case EACCES :
+                return( AFPERR_ACCESS );
+            default :
+                return( AFPERR_PARAM );
+            }
+        } else {
+            if (( dfd = open( ad_path( dst, ADFLAGS_HF ), O_WRONLY|O_CREAT,
+                              ad_mode( ad_path( dst, ADFLAGS_HF ), 0666 ))) < 0 ) {
+                close( sfd );
                 switch ( errno ) {
                 case ENOENT :
-                    break; /* just copy the data fork */
+                    return( AFPERR_NOOBJ );
                 case EACCES :
                     return( AFPERR_ACCESS );
+                case EROFS:
+                    return AFPERR_VLOCK;
                 default :
                     return( AFPERR_PARAM );
                 }
-            } else {
-                if (( dfd = open( ad_path( dst, ADFLAGS_HF ), O_WRONLY|O_CREAT,
-                                  ad_mode( ad_path( dst, ADFLAGS_HF ), 0666 ))) < 0 ) {
-                    close( sfd );
-                    switch ( errno ) {
-                    case ENOENT :
-                        return( AFPERR_NOOBJ );
-                    case EACCES :
-                        return( AFPERR_ACCESS );
-                    case EROFS:
-                        return AFPERR_VLOCK;
-                    default :
-                        return( AFPERR_PARAM );
-                    }
-                }
+            }
 
-                /* copy the file */
+            /* copy the file */
 #ifdef SENDFILE_FLAVOR_LINUX
-                if (fstat(sfd, &st) == 0) {
-                    if ((cc = sendfile(dfd, sfd, NULL, st.st_size)) < 0) {
-                        switch (errno) {
-                        case EDQUOT:
-                        case EFBIG:
-                        case ENOSPC:
-                            err = AFPERR_DFULL;
-                            break;
-                        case EROFS:
-                            err = AFPERR_VLOCK;
-                            break;
-                        default:
-                            err = AFPERR_PARAM;
-                        }
+            if (fstat(sfd, &st) == 0) {
+                if ((cc = sendfile(dfd, sfd, NULL, st.st_size)) < 0) {
+                    switch (errno) {
+                    case EDQUOT:
+                    case EFBIG:
+                    case ENOSPC:
+                        err = AFPERR_DFULL;
+                        break;
+                    case EROFS:
+                        err = AFPERR_VLOCK;
+                        break;
+                    default:
+                        err = AFPERR_PARAM;
                     }
-                    goto copyheader_done;
                 }
+                goto copyheader_done;
+            }
 #endif /* SENDFILE_FLAVOR_LINUX */
-                while (1) {
-                    if ((cc = read(sfd, filebuf, sizeof(filebuf))) < 0) {
-                        if (errno == EINTR)
-                            continue;
-                        err = AFPERR_PARAM;
-                        break;
-                    }
-
-                    if (!cc || ((err = copy_all(dfd, filebuf, cc)) < 0))
-                        break;
+            while (1) {
+                if ((cc = read(sfd, filebuf, sizeof(filebuf))) < 0) {
+                    if (errno == EINTR)
+                        continue;
+                    err = AFPERR_PARAM;
+                    break;
                 }
 
+                if (!cc || ((err = copy_all(dfd, filebuf, cc)) < 0))
+                    break;
+            }
+
 copyheader_done:
-                close(sfd);
-                close(dfd);
-                if (err < 0) {
-                    unlink(ad_path(dst, ADFLAGS_HF));
-                    return err;
-                }
+            close(sfd);
+            close(dfd);
+            if (err < 0) {
+                unlink(ad_path(dst, ADFLAGS_HF));
+                return err;
             }
         }
+    }
 
-        /* data fork copying */
-        if (( sfd = open( src, O_RDONLY, 0 )) < 0 ) {
-            switch ( errno ) {
-            case ENOENT :
-                return( AFPERR_NOOBJ );
-            case EACCES :
-                return( AFPERR_ACCESS );
-            default :
-                return( AFPERR_PARAM );
-            }
+    /* data fork copying */
+    if (( sfd = open( src, O_RDONLY, 0 )) < 0 ) {
+        switch ( errno ) {
+        case ENOENT :
+            return( AFPERR_NOOBJ );
+        case EACCES :
+            return( AFPERR_ACCESS );
+        default :
+            return( AFPERR_PARAM );
         }
+    }
 
-        if (( dfd = open( dst, O_WRONLY|O_CREAT, ad_mode( dst, 0666 ))) < 0 ) {
-            close( sfd );
-            switch ( errno ) {
-            case ENOENT :
-                return( AFPERR_NOOBJ );
-            case EACCES :
-                return( AFPERR_ACCESS );
-            case EROFS:
-                return AFPERR_VLOCK;
-            default :
-                return( AFPERR_PARAM );
-            }
+    if (( dfd = open( dst, O_WRONLY|O_CREAT, ad_mode( dst, 0666 ))) < 0 ) {
+        close( sfd );
+        switch ( errno ) {
+        case ENOENT :
+            return( AFPERR_NOOBJ );
+        case EACCES :
+            return( AFPERR_ACCESS );
+        case EROFS:
+            return AFPERR_VLOCK;
+        default :
+            return( AFPERR_PARAM );
         }
+    }
 
 #ifdef SENDFILE_FLAVOR_LINUX
-        if (fstat(sfd, &st) == 0) {
-            if ((cc = sendfile(dfd, sfd, NULL, st.st_size)) < 0) {
-                switch (errno) {
-                case EDQUOT:
-                case EFBIG:
-                case ENOSPC:
-                    err = AFPERR_DFULL;
-                    break;
-                default:
-                    err = AFPERR_PARAM;
-                }
+    if (fstat(sfd, &st) == 0) {
+        if ((cc = sendfile(dfd, sfd, NULL, st.st_size)) < 0) {
+            switch (errno) {
+            case EDQUOT:
+            case EFBIG:
+            case ENOSPC:
+                err = AFPERR_DFULL;
+                break;
+            default:
+                err = AFPERR_PARAM;
             }
-            goto copydata_done;
         }
+        goto copydata_done;
+    }
 #endif /* SENDFILE_FLAVOR_LINUX */
 
-        while (1) {
-            if ((cc = read( sfd, filebuf, sizeof( filebuf ))) < 0) {
-                if (errno == EINTR)
-                    continue;
+    while (1) {
+        if ((cc = read( sfd, filebuf, sizeof( filebuf ))) < 0) {
+            if (errno == EINTR)
+                continue;
 
-                err = AFPERR_PARAM;
-                break;
-            }
+            err = AFPERR_PARAM;
+            break;
+        }
 
-            if (!cc || ((err = copy_all(dfd, filebuf, cc)) < 0)) {
-                break;
-            }
+        if (!cc || ((err = copy_all(dfd, filebuf, cc)) < 0)) {
+            break;
         }
+    }
 
 copydata_done:
-        close(sfd);
-        close(dfd);
-        if (err < 0) {
-            unlink(ad_path(dst, ADFLAGS_HF));
-            unlink(dst);
-            return err;
-        }
+    close(sfd);
+    close(dfd);
+    if (err < 0) {
+        unlink(ad_path(dst, ADFLAGS_HF));
+        unlink(dst);
+        return err;
+    }
 
-        if (newname) {
-            memset(&ad, 0, sizeof(ad));
-            if ( ad_open( dst, noadouble | ADFLAGS_HF, O_RDWR|O_CREAT,
-                          0666, &ad) < 0 ) {
-                switch ( errno ) {
-                case ENOENT :
-                    return noadouble ? AFP_OK : AFPERR_NOOBJ;
-                case EACCES :
-                    return( AFPERR_ACCESS );
-                case EROFS:
-                    return AFPERR_VLOCK;
-                default :
-                    return( AFPERR_PARAM );
-                }
+    if (newname) {
+        memset(&ad, 0, sizeof(ad));
+        if ( ad_open( dst, noadouble | ADFLAGS_HF, O_RDWR|O_CREAT,
+                      0666, &ad) < 0 ) {
+            switch ( errno ) {
+            case ENOENT :
+                return noadouble ? AFP_OK : AFPERR_NOOBJ;
+            case EACCES :
+                return( AFPERR_ACCESS );
+            case EROFS:
+                return AFPERR_VLOCK;
+            default :
+                return( AFPERR_PARAM );
             }
-
-            len = strlen( newname );
-            ad_setentrylen( &ad, ADEID_NAME, len );
-            memcpy(ad_entry( &ad, ADEID_NAME ), newname, len );
-            ad_flush( &ad, ADFLAGS_HF );
-            ad_close( &ad, ADFLAGS_HF );
         }
 
+        len = strlen( newname );
+        ad_setentrylen( &ad, ADEID_NAME, len );
+        memcpy(ad_entry( &ad, ADEID_NAME ), newname, len );
+        ad_flush( &ad, ADFLAGS_HF );
+        ad_close( &ad, ADFLAGS_HF );
+    }
+
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end copyfile:");
+    LOG(log_info, logtype_default, "end copyfile:");
 #endif /* DEBUG */
 
-        return( AFP_OK );
-    }
+    return( AFP_OK );
+}
 
 
-    int deletefile( file )
-    char               *file;
-    {
-        struct adouble ad;
-        int                    adflags, err = AFP_OK;
-        int                    locktype = ADLOCK_WR;
-        int                    openmode = O_RDWR;
+int deletefile( file )
+char           *file;
+{
+    struct adouble     ad;
+    int                        adflags, err = AFP_OK;
+    int                        locktype = ADLOCK_WR;
+    int                        openmode = O_RDWR;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin deletefile:");
+    LOG(log_info, logtype_default, "begin deletefile:");
 #endif /* DEBUG */
 
-        while(1) {
-            /*
-             * If can't open read/write then try again read-only.  If it's open
-             * read-only, we must do a read lock instead of a write lock.
-             */
-            /* try to open both at once */
-            adflags = ADFLAGS_DF|ADFLAGS_HF;
-            memset(&ad, 0, sizeof(ad));
-            if ( ad_open( file, adflags, openmode, 0, &ad ) < 0 ) {
-                switch (errno) {
-                case ENOENT:
-                    adflags = ADFLAGS_DF;
-                    /* that failed. now try to open just the data fork */
-                    memset(&ad, 0, sizeof(ad));
-                    if ( ad_open( file, adflags, openmode, 0, &ad ) < 0 ) {
-                        switch (errno) {
-                        case ENOENT:
-                            return AFPERR_NOOBJ;
-                        case EACCES:
-                            if(openmode == O_RDWR) {
-                                openmode = O_RDONLY;
-                                locktype = ADLOCK_RD;
-                                continue;
-                            } else {
-                                return AFPERR_ACCESS;
-                            }
-                        case EROFS:
-                            return AFPERR_VLOCK;
-                        default:
-                            return AFPERR_PARAM;
+    while(1) {
+        /*
+         * If can't open read/write then try again read-only.  If it's open
+         * read-only, we must do a read lock instead of a write lock.
+         */
+        /* try to open both at once */
+        adflags = ADFLAGS_DF|ADFLAGS_HF;
+        memset(&ad, 0, sizeof(ad));
+        if ( ad_open( file, adflags, openmode, 0, &ad ) < 0 ) {
+            switch (errno) {
+            case ENOENT:
+                adflags = ADFLAGS_DF;
+                /* that failed. now try to open just the data fork */
+                memset(&ad, 0, sizeof(ad));
+                if ( ad_open( file, adflags, openmode, 0, &ad ) < 0 ) {
+                    switch (errno) {
+                    case ENOENT:
+                        return AFPERR_NOOBJ;
+                    case EACCES:
+                        if(openmode == O_RDWR) {
+                            openmode = O_RDONLY;
+                            locktype = ADLOCK_RD;
+                            continue;
+                        } else {
+                            return AFPERR_ACCESS;
                         }
+                    case EROFS:
+                        return AFPERR_VLOCK;
+                    default:
+                        return AFPERR_PARAM;
                     }
-                    break;
+                }
+                break;
 
-                case EACCES:
-                    if(openmode == O_RDWR) {
-                        openmode = O_RDONLY;
-                        locktype = ADLOCK_RD;
-                        continue;
-                    } else {
-                        return AFPERR_ACCESS;
-                    }
-                case EROFS:
-                    return AFPERR_VLOCK;
-                default:
-                    return( AFPERR_PARAM );
+            case EACCES:
+                if(openmode == O_RDWR) {
+                    openmode = O_RDONLY;
+                    locktype = ADLOCK_RD;
+                    continue;
+                } else {
+                    return AFPERR_ACCESS;
                 }
+            case EROFS:
+                return AFPERR_VLOCK;
+            default:
+                return( AFPERR_PARAM );
             }
-            break;     /* from the while */
         }
+        break; /* from the while */
+    }
 
-        if ((adflags & ADFLAGS_HF) &&
-                (ad_tmplock(&ad, ADEID_RFORK, locktype, 0, 0) < 0 )) {
-            ad_close( &ad, adflags );
-            return( AFPERR_BUSY );
-        }
+    if ((adflags & ADFLAGS_HF) &&
+            (ad_tmplock(&ad, ADEID_RFORK, locktype, 0, 0) < 0 )) {
+        ad_close( &ad, adflags );
+        return( AFPERR_BUSY );
+    }
 
-        if (ad_tmplock( &ad, ADEID_DFORK, locktype, 0, 0 ) < 0) {
-            err = AFPERR_BUSY;
-            goto delete_unlock;
-        }
+    if (ad_tmplock( &ad, ADEID_DFORK, locktype, 0, 0 ) < 0) {
+        err = AFPERR_BUSY;
+        goto delete_unlock;
+    }
 
-        if ( unlink( ad_path( file, ADFLAGS_HF )) < 0 ) {
-            switch ( errno ) {
-            case EPERM:
-            case EACCES :
-                err = AFPERR_ACCESS;
-                goto delete_unlock;
-            case EROFS:
-                err = AFPERR_VLOCK;
-                goto delete_unlock;
-            case ENOENT :
-                break;
-            default :
-                err = AFPERR_PARAM;
-                goto delete_unlock;
-            }
+    if ( unlink( ad_path( file, ADFLAGS_HF )) < 0 ) {
+        switch ( errno ) {
+        case EPERM:
+        case EACCES :
+            err = AFPERR_ACCESS;
+            goto delete_unlock;
+        case EROFS:
+            err = AFPERR_VLOCK;
+            goto delete_unlock;
+        case ENOENT :
+            break;
+        default :
+            err = AFPERR_PARAM;
+            goto delete_unlock;
         }
+    }
 
-        if ( unlink( file ) < 0 ) {
-            switch ( errno ) {
-            case EPERM:
-            case EACCES :
-                err = AFPERR_ACCESS;
-                break;
-            case EROFS:
-                err = AFPERR_VLOCK;
-                break;
-            case ENOENT :
-                break;
-            default :
-                err = AFPERR_PARAM;
-                break;
-            }
+    if ( unlink( file ) < 0 ) {
+        switch ( errno ) {
+        case EPERM:
+        case EACCES :
+            err = AFPERR_ACCESS;
+            break;
+        case EROFS:
+            err = AFPERR_VLOCK;
+            break;
+        case ENOENT :
+            break;
+        default :
+            err = AFPERR_PARAM;
+            break;
         }
+    }
 
 delete_unlock:
-        if (adflags & ADFLAGS_HF)
-            ad_tmplock(&ad, ADEID_RFORK, ADLOCK_CLR, 0, 0);
-        ad_tmplock(&ad, ADEID_DFORK, ADLOCK_CLR, 0, 0);
-        ad_close( &ad, adflags );
+    if (adflags & ADFLAGS_HF)
+        ad_tmplock(&ad, ADEID_RFORK, ADLOCK_CLR, 0, 0);
+    ad_tmplock(&ad, ADEID_DFORK, ADLOCK_CLR, 0, 0);
+    ad_close( &ad, adflags );
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end deletefile:");
+    LOG(log_info, logtype_default, "end deletefile:");
 #endif /* DEBUG */
 
-        return err;
-    }
+    return err;
+}
 
 
 #ifdef CNID_DB
-    /* return a file id */
-    int afp_createid(obj, ibuf, ibuflen, rbuf, rbuflen )
-    AFPObj      *obj;
-    char       *ibuf, *rbuf;
-    int                ibuflen, *rbuflen;
-    {
-        struct stat         st;
-        struct adouble ad;
-        struct vol             *vol;
-        struct dir             *dir;
-        char           *path, *upath;
-        int                 len;
-        cnid_t         did, id;
-        u_short                vid;
+/* return a file id */
+int afp_createid(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj      *obj;
+char   *ibuf, *rbuf;
+int            ibuflen, *rbuflen;
+{
+    struct stat         st;
+    struct adouble     ad;
+    struct vol         *vol;
+    struct dir         *dir;
+    char               *path, *upath;
+    int                 len;
+    cnid_t             did, id;
+    u_short            vid;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin afp_createid:");
+    LOG(log_info, logtype_default, "begin afp_createid:");
 #endif /* DEBUG */
 
-        *rbuflen = 0;
-        ibuf += 2;
+    *rbuflen = 0;
+    ibuf += 2;
 
-        memcpy(&vid, ibuf, sizeof(vid));
-        ibuf += sizeof(vid);
+    memcpy(&vid, ibuf, sizeof(vid));
+    ibuf += sizeof(vid);
 
-        if (( vol = getvolbyvid( vid )) == NULL ) {
-            return( AFPERR_PARAM);
-        }
+    if (( vol = getvolbyvid( vid )) == NULL ) {
+        return( AFPERR_PARAM);
+    }
 
-        if (vol->v_flags & AFPVOL_RO)
-            return AFPERR_VLOCK;
+    if (vol->v_flags & AFPVOL_RO)
+        return AFPERR_VLOCK;
 
-        memcpy(&did, ibuf, sizeof( did ));
-        ibuf += sizeof(did);
+    memcpy(&did, ibuf, sizeof( did ));
+    ibuf += sizeof(did);
 
-        if (( dir = dirsearch( vol, did )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( dir = dirsearch( vol, did )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if ( *path == '\0' ) {
-            return( AFPERR_BADTYPE );
-        }
+    if ( *path == '\0' ) {
+        return( AFPERR_BADTYPE );
+    }
 
-        upath = mtoupath(vol, path);
-        if (stat(upath, &st) < 0) {
-            switch (errno) {
-            case EPERM:
-            case EACCES:
-                return AFPERR_ACCESS;
-            case ENOENT:
-                return AFPERR_NOOBJ;
-            default:
-                return AFPERR_PARAM;
-            }
+    upath = mtoupath(vol, path);
+    if (stat(upath, &st) < 0) {
+        switch (errno) {
+        case EPERM:
+        case EACCES:
+            return AFPERR_ACCESS;
+        case ENOENT:
+            return AFPERR_NOOBJ;
+        default:
+            return AFPERR_PARAM;
         }
+    }
 
-        if (id = cnid_lookup(vol->v_db, &st, did, upath, len = strlen(upath))) {
-            memcpy(rbuf, &id, sizeof(id));
-            *rbuflen = sizeof(id);
-            return AFPERR_EXISTID;
-        }
+    if (id = cnid_lookup(vol->v_db, &st, did, upath, len = strlen(upath))) {
+        memcpy(rbuf, &id, sizeof(id));
+        *rbuflen = sizeof(id);
+        return AFPERR_EXISTID;
+    }
 
 #if AD_VERSION > AD_VERSION1
-        memset(&ad, 0, sizeof(ad));
-        if (ad_open( upath, ADFLAGS_HF, O_RDONLY, 0, &ad ) >= 0) {
-            memcpy(&id, ad_entry(&ad, ADEID_DID), sizeof(id));
-            ad_close(&ad, ADFLAGS_HF);
-        }
+    memset(&ad, 0, sizeof(ad));
+    if (ad_open( upath, ADFLAGS_HF, O_RDONLY, 0, &ad ) >= 0) {
+        memcpy(&id, ad_entry(&ad, ADEID_DID), sizeof(id));
+        ad_close(&ad, ADFLAGS_HF);
+    }
 #endif /* AD_VERSION > AD_VERSION1 */
 
-        if (id = cnid_add(vol->v_db, &st, did, upath, len, id)) {
-            memcpy(rbuf, &id, sizeof(id));
-            *rbuflen = sizeof(id);
-            return AFP_OK;
-        }
+    if (id = cnid_add(vol->v_db, &st, did, upath, len, id)) {
+        memcpy(rbuf, &id, sizeof(id));
+        *rbuflen = sizeof(id);
+        return AFP_OK;
+    }
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "ending afp_createid...:");
+    LOG(log_info, logtype_default, "ending afp_createid...:");
 #endif /* DEBUG */
 
-        switch (errno) {
-        case EROFS:
-            return AFPERR_VLOCK;
-            break;
-        case EPERM:
-        case EACCES:
-            return AFPERR_ACCESS;
-            break;
-        default:
-            LOG(log_error, logtype_default, "afp_createid: cnid_add: %s", strerror(errno));
-            return AFPERR_PARAM;
-        }
+    switch (errno) {
+    case EROFS:
+        return AFPERR_VLOCK;
+        break;
+    case EPERM:
+    case EACCES:
+        return AFPERR_ACCESS;
+        break;
+    default:
+        LOG(log_error, logtype_default, "afp_createid: cnid_add: %s", strerror(errno));
+        return AFPERR_PARAM;
     }
+}
 
-    /* resolve a file id */
-    int afp_resolveid(obj, ibuf, ibuflen, rbuf, rbuflen )
-    AFPObj      *obj;
-    char       *ibuf, *rbuf;
-    int                ibuflen, *rbuflen;
-    {
-        struct stat         st;
-        struct vol             *vol;
-        struct dir             *dir;
-        char           *upath;
-        int                 err, buflen;
-        cnid_t         id;
-        u_int16_t              vid, bitmap;
+/* resolve a file id */
+int afp_resolveid(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj      *obj;
+char   *ibuf, *rbuf;
+int            ibuflen, *rbuflen;
+{
+    struct stat         st;
+    struct vol         *vol;
+    struct dir         *dir;
+    char               *upath;
+    int                 err, buflen;
+    cnid_t             id;
+    u_int16_t          vid, bitmap;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin afp_resolveid:");
+    LOG(log_info, logtype_default, "begin afp_resolveid:");
 #endif /* DEBUG */
 
-        *rbuflen = 0;
-        ibuf += 2;
+    *rbuflen = 0;
+    ibuf += 2;
 
-        memcpy(&vid, ibuf, sizeof(vid));
-        ibuf += sizeof(vid);
+    memcpy(&vid, ibuf, sizeof(vid));
+    ibuf += sizeof(vid);
 
-        if (( vol = getvolbyvid( vid )) == NULL ) {
-            return( AFPERR_PARAM);
-        }
+    if (( vol = getvolbyvid( vid )) == NULL ) {
+        return( AFPERR_PARAM);
+    }
 
-        memcpy(&id, ibuf, sizeof( id ));
-        ibuf += sizeof(id);
+    memcpy(&id, ibuf, sizeof( id ));
+    ibuf += sizeof(id);
 
-        if ((upath = cnid_resolve(vol->v_db, &id)) == NULL) {
-            return AFPERR_BADID;
-        }
+    if ((upath = cnid_resolve(vol->v_db, &id)) == NULL) {
+        return AFPERR_BADID;
+    }
 
-        if (( dir = dirsearch( vol, id )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( dir = dirsearch( vol, id )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if ((movecwd(vol, dir) < 0) || (stat(upath, &st) < 0)) {
-            switch (errno) {
-            case EACCES:
-            case EPERM:
-                return AFPERR_ACCESS;
-            case ENOENT:
-                return AFPERR_NOID;
-            default:
-                return AFPERR_PARAM;
-            }
+    if ((movecwd(vol, dir) < 0) || (stat(upath, &st) < 0)) {
+        switch (errno) {
+        case EACCES:
+        case EPERM:
+            return AFPERR_ACCESS;
+        case ENOENT:
+            return AFPERR_NOID;
+        default:
+            return AFPERR_PARAM;
         }
+    }
 
-        /* directories are bad */
-        if (S_ISDIR(st.st_mode))
-            return AFPERR_BADTYPE;
+    /* directories are bad */
+    if (S_ISDIR(st.st_mode))
+        return AFPERR_BADTYPE;
 
-        memcpy(&bitmap, ibuf, sizeof(bitmap));
-        bitmap = ntohs( bitmap );
+    memcpy(&bitmap, ibuf, sizeof(bitmap));
+    bitmap = ntohs( bitmap );
 
-        if ((err = getfilparams(vol, bitmap, utompath(vol, upath), curdir, &st,
-                                rbuf + sizeof(bitmap), &buflen)) != AFP_OK)
-            return err;
+    if ((err = getfilparams(vol, bitmap, utompath(vol, upath), curdir, &st,
+                            rbuf + sizeof(bitmap), &buflen)) != AFP_OK)
+        return err;
 
-        *rbuflen = buflen + sizeof(bitmap);
-        memcpy(rbuf, ibuf, sizeof(bitmap));
+    *rbuflen = buflen + sizeof(bitmap);
+    memcpy(rbuf, ibuf, sizeof(bitmap));
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end afp_resolveid:");
+    LOG(log_info, logtype_default, "end afp_resolveid:");
 #endif /* DEBUG */
 
-        return AFP_OK;
-    }
+    return AFP_OK;
+}
 
-    int afp_deleteid(obj, ibuf, ibuflen, rbuf, rbuflen )
-    AFPObj      *obj;
-    char       *ibuf, *rbuf;
-    int                ibuflen, *rbuflen;
-    {
-        struct stat         st;
-        struct vol             *vol;
-        struct dir             *dir;
-        char                *upath;
-        int                 err;
-        cnid_t         id;
-        u_short                vid;
+int afp_deleteid(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj      *obj;
+char   *ibuf, *rbuf;
+int            ibuflen, *rbuflen;
+{
+    struct stat         st;
+    struct vol         *vol;
+    struct dir         *dir;
+    char                *upath;
+    int                 err;
+    cnid_t             id;
+    u_short            vid;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin afp_deleteid:");
+    LOG(log_info, logtype_default, "begin afp_deleteid:");
 #endif /* DEBUG */
 
-        *rbuflen = 0;
-        ibuf += 2;
+    *rbuflen = 0;
+    ibuf += 2;
 
-        memcpy(&vid, ibuf, sizeof(vid));
-        ibuf += sizeof(vid);
+    memcpy(&vid, ibuf, sizeof(vid));
+    ibuf += sizeof(vid);
 
-        if (( vol = getvolbyvid( vid )) == NULL ) {
-            return( AFPERR_PARAM);
-        }
+    if (( vol = getvolbyvid( vid )) == NULL ) {
+        return( AFPERR_PARAM);
+    }
 
-        if (vol->v_flags & AFPVOL_RO)
-            return AFPERR_VLOCK;
+    if (vol->v_flags & AFPVOL_RO)
+        return AFPERR_VLOCK;
 
-        memcpy(&id, ibuf, sizeof( id ));
-        ibuf += sizeof(id);
+    memcpy(&id, ibuf, sizeof( id ));
+    ibuf += sizeof(id);
 
-        if ((upath = cnid_resolve(vol->v_db, &id)) == NULL) {
-            return AFPERR_NOID;
-        }
+    if ((upath = cnid_resolve(vol->v_db, &id)) == NULL) {
+        return AFPERR_NOID;
+    }
 
-        if (( dir = dirsearch( vol, id )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( dir = dirsearch( vol, id )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        err = AFP_OK;
-        if ((movecwd(vol, dir) < 0) || (stat(upath, &st) < 0)) {
-            switch (errno) {
-            case EACCES:
-            case EPERM:
-                return AFPERR_ACCESS;
-            case ENOENT:
-                /* still try to delete the id */
-                err = AFPERR_NOOBJ;
-                break;
-            default:
-                return AFPERR_PARAM;
-            }
+    err = AFP_OK;
+    if ((movecwd(vol, dir) < 0) || (stat(upath, &st) < 0)) {
+        switch (errno) {
+        case EACCES:
+        case EPERM:
+            return AFPERR_ACCESS;
+        case ENOENT:
+            /* still try to delete the id */
+            err = AFPERR_NOOBJ;
+            break;
+        default:
+            return AFPERR_PARAM;
         }
+    }
 
-        /* directories are bad */
-        if (S_ISDIR(st.st_mode))
-            return AFPERR_BADTYPE;
+    /* directories are bad */
+    if (S_ISDIR(st.st_mode))
+        return AFPERR_BADTYPE;
 
-        if (cnid_delete(vol->v_db, id)) {
-            switch (errno) {
-            case EROFS:
-                return AFPERR_VLOCK;
-            case EPERM:
-            case EACCES:
-                return AFPERR_ACCESS;
-            default:
-                return AFPERR_PARAM;
-            }
+    if (cnid_delete(vol->v_db, id)) {
+        switch (errno) {
+        case EROFS:
+            return AFPERR_VLOCK;
+        case EPERM:
+        case EACCES:
+            return AFPERR_ACCESS;
+        default:
+            return AFPERR_PARAM;
         }
+    }
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "end afp_deleteid:");
+    LOG(log_info, logtype_default, "end afp_deleteid:");
 #endif /* DEBUG */
 
-        return err;
-    }
+    return err;
+}
 #endif /* CNID_DB */
 
 #define APPLETEMP ".AppleTempXXXXXX"
 
-    int afp_exchangefiles(obj, ibuf, ibuflen, rbuf, rbuflen )
-    AFPObj      *obj;
-    char       *ibuf, *rbuf;
-    int                ibuflen, *rbuflen;
-    {
-        struct stat         srcst, destst;
-        struct vol             *vol;
-        struct dir             *dir, *sdir;
-        char           *spath, temp[17], *path, *p;
-        char                *supath, *upath;
-        int                 err;
+int afp_exchangefiles(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj      *obj;
+char   *ibuf, *rbuf;
+int            ibuflen, *rbuflen;
+{
+    struct stat         srcst, destst;
+    struct vol         *vol;
+    struct dir         *dir, *sdir;
+    char               *spath, temp[17], *path, *p;
+    char                *supath, *upath;
+    int                 err;
 #ifdef CNID_DB
-        int                 slen, dlen;
+    int                 slen, dlen;
 #endif /* CNID_DB */
-        u_int32_t              sid, did;
-        u_int16_t              vid;
+    u_int32_t          sid, did;
+    u_int16_t          vid;
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "begin afp_exchangefiles:");
+    LOG(log_info, logtype_default, "begin afp_exchangefiles:");
 #endif /* DEBUG */
 
-        *rbuflen = 0;
-        ibuf += 2;
+    *rbuflen = 0;
+    ibuf += 2;
 
-        memcpy(&vid, ibuf, sizeof(vid));
-        ibuf += sizeof(vid);
+    memcpy(&vid, ibuf, sizeof(vid));
+    ibuf += sizeof(vid);
 
-        if (( vol = getvolbyvid( vid )) == NULL ) {
-            return( AFPERR_PARAM);
-        }
+    if (( vol = getvolbyvid( vid )) == NULL ) {
+        return( AFPERR_PARAM);
+    }
 
-        if (vol->v_flags & AFPVOL_RO)
-            return AFPERR_VLOCK;
+    if (vol->v_flags & AFPVOL_RO)
+        return AFPERR_VLOCK;
 
-        /* source and destination dids */
-        memcpy(&sid, ibuf, sizeof(sid));
-        ibuf += sizeof(sid);
-        memcpy(&did, ibuf, sizeof(did));
-        ibuf += sizeof(did);
+    /* source and destination dids */
+    memcpy(&sid, ibuf, sizeof(sid));
+    ibuf += sizeof(sid);
+    memcpy(&did, ibuf, sizeof(did));
+    ibuf += sizeof(did);
 
-        /* source file */
-        if ((dir = dirsearch( vol, sid )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    /* source file */
+    if ((dir = dirsearch( vol, sid )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if ( *path == '\0' ) {
-            return( AFPERR_BADTYPE );
-        }
+    if ( *path == '\0' ) {
+        return( AFPERR_BADTYPE );
+    }
 
-        upath = mtoupath(vol, path);
-        if (stat(upath, &srcst) < 0) {
-            switch (errno) {
-            case ENOENT:
-                return AFPERR_NOID;
-            case EPERM:
-            case EACCES:
-                return AFPERR_ACCESS;
-            default:
-                return AFPERR_PARAM;
-            }
+    upath = mtoupath(vol, path);
+    if (stat(upath, &srcst) < 0) {
+        switch (errno) {
+        case ENOENT:
+            return AFPERR_NOID;
+        case EPERM:
+        case EACCES:
+            return AFPERR_ACCESS;
+        default:
+            return AFPERR_PARAM;
         }
+    }
 
-        /* save some stuff */
-        sdir = curdir;
-        spath = obj->oldtmp;
-        supath = obj->newtmp;
-        strcpy(spath, path);
-        strcpy(supath, upath); /* this is for the cnid changing */
-        p = ctoupath( vol, sdir, spath);
+    /* save some stuff */
+    sdir = curdir;
+    spath = obj->oldtmp;
+    supath = obj->newtmp;
+    strcpy(spath, path);
+    strcpy(supath, upath); /* this is for the cnid changing */
+    p = ctoupath( vol, sdir, spath);
 
-        /* look for the source cnid. if it doesn't exist, don't worry about
-         * it. */
+    /* look for the source cnid. if it doesn't exist, don't worry about
+     * it. */
 #ifdef CNID_DB
-        sid = cnid_lookup(vol->v_db, &srcst, sdir->d_did, supath,
-                          slen = strlen(supath));
+    sid = cnid_lookup(vol->v_db, &srcst, sdir->d_did, supath,
+                      slen = strlen(supath));
 #endif /* CNID_DB */
 
-        if (( dir = dirsearch( vol, did )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( dir = dirsearch( vol, did )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-            return( AFPERR_PARAM );
-        }
+    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
+        return( AFPERR_PARAM );
+    }
 
-        if ( *path == '\0' ) {
-            return( AFPERR_BADTYPE );
-        }
+    if ( *path == '\0' ) {
+        return( AFPERR_BADTYPE );
+    }
 
-        /* FPExchangeFiles is the only call that can return the SameObj
-         * error */
-        if ((curdir == sdir) && strcmp(spath, path) == 0)
-            return AFPERR_SAMEOBJ;
+    /* FPExchangeFiles is the only call that can return the SameObj
+     * error */
+    if ((curdir == sdir) && strcmp(spath, path) == 0)
+        return AFPERR_SAMEOBJ;
 
-        upath = mtoupath(vol, path);
-        if (stat(upath, &destst) < 0) {
-            switch (errno) {
-            case ENOENT:
-                return AFPERR_NOID;
-            case EPERM:
-            case EACCES:
-                return AFPERR_ACCESS;
-            default:
-                return AFPERR_PARAM;
-            }
+    upath = mtoupath(vol, path);
+    if (stat(upath, &destst) < 0) {
+        switch (errno) {
+        case ENOENT:
+            return AFPERR_NOID;
+        case EPERM:
+        case EACCES:
+            return AFPERR_ACCESS;
+        default:
+            return AFPERR_PARAM;
         }
+    }
 
 #ifdef CNID_DB
-        /* look for destination id. */
-        did = cnid_lookup(vol->v_db, &destst, curdir->d_did, upath,
-                          dlen = strlen(upath));
+    /* look for destination id. */
+    did = cnid_lookup(vol->v_db, &destst, curdir->d_did, upath,
+                      dlen = strlen(upath));
 #endif /* CNID_DB */
 
-        /* construct a temp name.
-         * NOTE: the temp file will be in the dest file's directory. it
-         * will also be inaccessible from AFP. */
-        memcpy(temp, APPLETEMP, sizeof(APPLETEMP));
-        if (!mktemp(temp))
-            return AFPERR_MISC;
+    /* construct a temp name.
+     * NOTE: the temp file will be in the dest file's directory. it
+     * will also be inaccessible from AFP. */
+    memcpy(temp, APPLETEMP, sizeof(APPLETEMP));
+    if (!mktemp(temp))
+        return AFPERR_MISC;
 
-        /* now, quickly rename the file. we error if we can't. */
-        if ((err = renamefile(p, temp, temp, vol_noadouble(vol))) < 0)
-            goto err_exchangefile;
-        of_rename(vol, sdir, spath, curdir, temp);
+    /* now, quickly rename the file. we error if we can't. */
+    if ((err = renamefile(p, temp, temp, vol_noadouble(vol))) < 0)
+        goto err_exchangefile;
+    of_rename(vol, sdir, spath, curdir, temp);
 
-        /* rename destination to source */
-        if ((err = renamefile(path, p, spath, vol_noadouble(vol))) < 0)
-            goto err_src_to_tmp;
-        of_rename(vol, curdir, path, sdir, spath);
+    /* rename destination to source */
+    if ((err = renamefile(path, p, spath, vol_noadouble(vol))) < 0)
+        goto err_src_to_tmp;
+    of_rename(vol, curdir, path, sdir, spath);
 
-        /* rename temp to destination */
-        if ((err = renamefile(temp, upath, path, vol_noadouble(vol))) < 0)
-            goto err_dest_to_src;
-        of_rename(vol, curdir, temp, curdir, path);
+    /* rename temp to destination */
+    if ((err = renamefile(temp, upath, path, vol_noadouble(vol))) < 0)
+        goto err_dest_to_src;
+    of_rename(vol, curdir, temp, curdir, path);
 
 #ifdef CNID_DB
-        /* id's need switching. src -> dest and dest -> src. */
-        if (sid && (cnid_update(vol->v_db, sid, &destst, curdir->d_did,
-                                upath, dlen) < 0)) {
-            switch (errno) {
-            case EPERM:
-            case EACCES:
-                err = AFPERR_ACCESS;
-                break;
-            default:
-                err = AFPERR_PARAM;
-            }
-            goto err_temp_to_dest;
+    /* id's need switching. src -> dest and dest -> src. */
+    if (sid && (cnid_update(vol->v_db, sid, &destst, curdir->d_did,
+                            upath, dlen) < 0)) {
+        switch (errno) {
+        case EPERM:
+        case EACCES:
+            err = AFPERR_ACCESS;
+            break;
+        default:
+            err = AFPERR_PARAM;
         }
+        goto err_temp_to_dest;
+    }
 
-        if (did && (cnid_update(vol->v_db, did, &srcst, sdir->d_did,
-                                supath, slen) < 0)) {
-            switch (errno) {
-            case EPERM:
-            case EACCES:
-                err = AFPERR_ACCESS;
-                break;
-            default:
-                err = AFPERR_PARAM;
-            }
-
-            if (sid)
-                cnid_update(vol->v_db, sid, &srcst, sdir->d_did, supath, slen);
-            goto err_temp_to_dest;
+    if (did && (cnid_update(vol->v_db, did, &srcst, sdir->d_did,
+                            supath, slen) < 0)) {
+        switch (errno) {
+        case EPERM:
+        case EACCES:
+            err = AFPERR_ACCESS;
+            break;
+        default:
+            err = AFPERR_PARAM;
         }
+
+        if (sid)
+            cnid_update(vol->v_db, sid, &srcst, sdir->d_did, supath, slen);
+        goto err_temp_to_dest;
+    }
 #endif /* CNID_DB */
 
 #ifdef DEBUG
-        LOG(log_info, logtype_default, "ending afp_exchangefiles:");
+    LOG(log_info, logtype_default, "ending afp_exchangefiles:");
 #endif /* DEBUG */
 
-        return AFP_OK;
+    return AFP_OK;
 
 
-        /* all this stuff is so that we can unwind a failed operation
-         * properly. */
+    /* all this stuff is so that we can unwind a failed operation
+     * properly. */
 err_temp_to_dest:
-        /* rename dest to temp */
-        renamefile(upath, temp, temp, vol_noadouble(vol));
-        of_rename(vol, curdir, upath, curdir, temp);
+    /* rename dest to temp */
+    renamefile(upath, temp, temp, vol_noadouble(vol));
+    of_rename(vol, curdir, upath, curdir, temp);
 
 err_dest_to_src:
-        /* rename source back to dest */
-        renamefile(p, upath, path, vol_noadouble(vol));
-        of_rename(vol, sdir, spath, curdir, path);
+    /* rename source back to dest */
+    renamefile(p, upath, path, vol_noadouble(vol));
+    of_rename(vol, sdir, spath, curdir, path);
 
 err_src_to_tmp:
-        /* rename temp back to source */
-        renamefile(temp, p, spath, vol_noadouble(vol));
-        of_rename(vol, curdir, temp, sdir, spath);
+    /* rename temp back to source */
+    renamefile(temp, p, spath, vol_noadouble(vol));
+    of_rename(vol, curdir, temp, sdir, spath);
 
 err_exchangefile:
-        return err;
-    }
+    return err;
+}