]> arthur.barton.de Git - netatalk.git/blobdiff - etc/afpd/fork.c
Remove an #error if flock locks are enabled.
[netatalk.git] / etc / afpd / fork.c
index c238d7a5b94b64f4741e030a1c1f177b4fbc164a..77156a54cb29c4d3f23963dc270e396d10aa543b 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: fork.c,v 1.28 2002-05-29 17:40:36 jmarcus Exp $
+ * $Id: fork.c,v 1.40 2002-11-14 17:13:45 srittau Exp $
  *
  * Copyright (c) 1990,1993 Regents of The University of Michigan.
  * All Rights Reserved.  See COPYRIGHT.
@@ -48,6 +48,7 @@
 #include "volume.h"
 
 #define BYTELOCK_MAX 0x7FFFFFFFU
+#define BYTELOCK_MAXL 0x7FFFFFFFFFFFFFFFULL
 
 struct ofork           *writtenfork;
 extern int getmetadata(struct vol *vol,
@@ -62,44 +63,39 @@ char                *buf;
 int                    *buflen;
 const u_int16_t     attrbits;
 {
-#ifndef USE_LASTDID
-    struct stat                lst, *lstp;
-#endif /* !USE_LASTDID */
     struct stat                st;
-    struct extmap      *em;
-    char                   *data, *nameoff = NULL, *upath;
-    int                            bit = 0;
-    u_int32_t       aint;
-    u_int16_t          ashort;
+    char               *upath;
 
     struct adouble     *adp;
-    struct dir      *dir;
-    struct vol      *vol;
+    struct dir         *dir;
+    struct vol         *vol;
     
+
+    /* can only get the length of the opened fork */
+    if ( ( (bitmap & ((1<<FILPBIT_DFLEN) | (1<<FILPBIT_EXTDFLEN))) 
+                  && (ofork->of_flags & AFPFORK_RSRC)) 
+        ||
+          ( (bitmap & ((1<<FILPBIT_RFLEN) | (1<<FILPBIT_EXTRFLEN))) 
+                  && (ofork->of_flags & AFPFORK_DATA))) {
+        return( AFPERR_BITMAP );
+    }
+
     if ( ad_hfileno( ofork->of_ad ) == -1 ) {
         adp = NULL;
     } else {
-        aint = ad_getentrylen( ofork->of_ad, ADEID_RFORK );
-        if ( ad_refresh( ofork->of_ad ) < 0 ) {
-            LOG(log_error, logtype_afpd, "getforkparams: ad_refresh: %s", strerror(errno) );
-            return( AFPERR_PARAM );
-        }
-        /* See afp_closefork() for why this is bad */
-        ad_setentrylen( ofork->of_ad, ADEID_RFORK, aint );
         adp = ofork->of_ad;
     }
 
-    /* can only get the length of the opened fork */
-    if (((bitmap & (1<<FILPBIT_DFLEN)) && (ofork->of_flags & AFPFORK_RSRC)) ||
-            ((bitmap & (1<<FILPBIT_RFLEN)) && (ofork->of_flags & AFPFORK_DATA))) {
-        return( AFPERR_BITMAP );
-    }
+    vol = ofork->of_vol;
+    dir = ofork->of_dir;
 
-    if ( bitmap & ( 1<<FILPBIT_DFLEN | 1<<FILPBIT_FNUM |
-                    (1 << FILPBIT_CDATE) | (1 << FILPBIT_MDATE) |
-                    (1 << FILPBIT_BDATE))) {
-        upath = mtoupath(ofork->of_vol, ofork->of_name);
+    if ( bitmap & ( (1<<FILPBIT_DFLEN) | (1<<FILPBIT_EXTDFLEN) | 
+                    (1<<FILPBIT_FNUM) | (1 << FILPBIT_CDATE) | 
+                    (1 << FILPBIT_MDATE) | (1 << FILPBIT_BDATE))) {
         if ( ad_dfileno( ofork->of_ad ) == -1 ) {
+            upath = mtoupath(vol, ofork->of_name);
+            if (movecwd(vol, dir) < 0)
+                return( AFPERR_NOOBJ );
             if ( stat( upath, &st ) < 0 )
                 return( AFPERR_NOOBJ );
         } else {
@@ -108,8 +104,6 @@ const u_int16_t     attrbits;
             }
         }
     }
-    dir = ofork->of_dir;
-    vol = ofork->of_vol;
     return getmetadata(vol, bitmap, ofork->of_name, dir, &st, buf, buflen, adp, attrbits );    
 }
 
@@ -142,10 +136,6 @@ static int getforkmode(struct adouble *adp, int eid, int what)
     return ad_testlock(adp, eid,  what);
 }
 
-/* -------------------------- 
-   a lot of races, some can be remove. but I try first to get the semantic right
-*/
-
 static int afp_setmode(struct adouble *adp, int eid, int access, int ofrefnum)
 {
     int ret;
@@ -232,7 +222,10 @@ int                ibuflen, *rbuflen;
     u_int32_t           did;
     u_int16_t          vid, bitmap, access, ofrefnum, attrbits = 0;
     char               fork, *path, *upath;
-
+    struct stat         *st;
+    u_int16_t           bshort;
+    struct path         *s_path;
+    
     ibuf++;
     fork = *ibuf++;
     memcpy(&vid, ibuf, sizeof( vid ));
@@ -246,7 +239,7 @@ int         ibuflen, *rbuflen;
     memcpy(&did, ibuf, sizeof( did ));
     ibuf += sizeof( int );
 
-    if (( dir = dirsearch( vol, did )) == NULL ) {
+    if (( dir = dirlookup( vol, did )) == NULL ) {
         return( AFPERR_NOOBJ );
     }
 
@@ -261,62 +254,82 @@ int               ibuflen, *rbuflen;
         return AFPERR_VLOCK;
     }
 
-    if (( path = cname( vol, dir, &ibuf )) == NULL ) {
-        return( AFPERR_NOOBJ );
+    if (( s_path = cname( vol, dir, &ibuf )) == NULL ) {
+        return AFPERR_NOOBJ;
     }
 
-    if ( fork == OPENFORK_DATA ) {
-        eid = ADEID_DFORK;
-        adflags = ADFLAGS_DF|ADFLAGS_HF;
-    } else {
-        eid = ADEID_RFORK;
-        adflags = ADFLAGS_HF;
+    if (*s_path->m_name == '\0') {
+       /* it's a dir ! */
+       return  AFPERR_BADTYPE;
+    }
+
+    /* stat() data fork st is set because it's not a dir */
+    switch ( s_path->st_errno ) {
+    case 0:
+        break;
+    case ENOENT:
+        return AFPERR_NOOBJ;
+    case EACCES:
+        return (access & OPENACC_WR) ? AFPERR_LOCK : AFPERR_ACCESS;
+    default:
+        LOG(log_error, logtype_afpd, "afp_openfork: ad_open: %s", strerror(errno) );
+        return AFPERR_PARAM;
+    }
+    /* FIXME should we check first ? */
+    upath = s_path->u_name;
+    if (check_access(upath, access ) < 0) {
+        return AFPERR_ACCESS;
     }
 
+    st   = &s_path->st;
     /* XXX: this probably isn't the best way to do this. the already
        open bits should really be set if the fork is opened by any
        program, not just this one. however, that's problematic to do
        if we can't write lock files somewhere. opened is also passed to 
-       ad_open so that we can keep file locks together. */
-    if ((opened = of_findname(vol, curdir, path))) {
-        attrbits = ((opened->of_flags & AFPFORK_RSRC) ? ATTRBIT_ROPEN : 0) |
-                   ((opened->of_flags & AFPFORK_DATA) ? ATTRBIT_DOPEN : 0);
+       ad_open so that we can keep file locks together.
+       FIXME: add the fork we are opening? 
+    */
+    if ((opened = of_findname(s_path))) {
+        attrbits = ((opened->of_ad->ad_df.adf_refcount > 0) ? ATTRBIT_DOPEN : 0);
+        attrbits |= ((opened->of_ad->ad_hf.adf_refcount > opened->of_ad->ad_df.adf_refcount)? ATTRBIT_ROPEN : 0);
+                   
         adsame = opened->of_ad;
     }
 
+    if ( fork == OPENFORK_DATA ) {
+        eid = ADEID_DFORK;
+        adflags = ADFLAGS_DF|ADFLAGS_HF;
+    } else {
+        eid = ADEID_RFORK;
+        adflags = ADFLAGS_HF;
+    }
+
+    path = s_path->m_name;
     if (( ofork = of_alloc(vol, curdir, path, &ofrefnum, eid,
-                           adsame)) == NULL ) {
+                           adsame, st)) == NULL ) {
         return( AFPERR_NFILE );
     }
+
+    ret = AFPERR_NOOBJ;
     if (access & OPENACC_WR) {
         /* try opening in read-write mode */
-        upath = mtoupath(vol, path);
-        ret = AFPERR_NOOBJ;
         if (ad_open(upath, adflags, O_RDWR, 0, ofork->of_ad) < 0) {
             switch ( errno ) {
             case EROFS:
                 ret = AFPERR_VLOCK;
             case EACCES:
                 goto openfork_err;
-
                 break;
             case ENOENT:
-                {
-                    struct stat st;
-
-                    /* see if client asked for the data fork */
-                    if (fork == OPENFORK_DATA) {
-                        if (ad_open(upath, ADFLAGS_DF, O_RDWR, 0, ofork->of_ad) < 0) {
-                            goto openfork_err;
-                        }
-                        adflags = ADFLAGS_DF;
-
-                    } else if (stat(upath, &st) == 0) {
-                        /* here's the deal. we only try to create the resource
-                         * fork if the user wants to open it for write acess. */
-                        if (ad_open(upath, adflags, O_RDWR | O_CREAT, 0666, ofork->of_ad) < 0)
-                            goto openfork_err;
-                    } else
+                if (fork == OPENFORK_DATA) {
+                    if (ad_open(upath, ADFLAGS_DF, O_RDWR, 0, ofork->of_ad) < 0) {
+                        goto openfork_err;
+                    }
+                    adflags = ADFLAGS_DF;
+
+                    /* here's the deal. we only try to create the resource
+                    * fork if the user wants to open it for write acess. */
+                    if (ad_open(upath, adflags, O_RDWR | O_CREAT, 0666, ofork->of_ad) < 0)
                         goto openfork_err;
                 }
                 break;
@@ -338,7 +351,6 @@ int         ibuflen, *rbuflen;
         }
     } else {
         /* try opening in read-only mode */
-        upath = mtoupath(vol, path);
         ret = AFPERR_NOOBJ;
         if (ad_open(upath, adflags, O_RDONLY, 0, ofork->of_ad) < 0) {
             switch ( errno ) {
@@ -353,19 +365,12 @@ int               ibuflen, *rbuflen;
                 adflags = ADFLAGS_DF;
                 break;
             case ENOENT:
-                {
-                    struct stat st;
-
-                    /* see if client asked for the data fork */
-                    if (fork == OPENFORK_DATA) {
-                        if (ad_open(upath, ADFLAGS_DF, O_RDONLY, 0, ofork->of_ad) < 0) {
-                            goto openfork_err;
-                        }
-                        adflags = ADFLAGS_DF;
-
-                    } else if (stat(upath, &st) != 0) {
+                /* see if client asked for the data fork */
+                if (fork == OPENFORK_DATA) {
+                    if (ad_open(upath, ADFLAGS_DF, O_RDONLY, 0, ofork->of_ad) < 0) {
                         goto openfork_err;
                     }
+                    adflags = ADFLAGS_DF;
                 }
                 break;
             case EMFILE :
@@ -404,6 +409,16 @@ int                ibuflen, *rbuflen;
     memcpy(rbuf, &bitmap, sizeof( u_int16_t ));
     rbuf += sizeof( u_int16_t );
 
+    /* check  WriteInhibit bit, the test is done here, after some Mac trafic capture */
+    ad_getattr(ofork->of_ad, &bshort);
+    if ((bshort & htons(ATTRBIT_NOWRITE)) && (access & OPENACC_WR)) {
+        ad_close( ofork->of_ad, adflags );
+        of_dealloc( ofork );
+        ofrefnum = 0;
+        memcpy(rbuf, &ofrefnum, sizeof(ofrefnum));
+        return(AFPERR_OLOCK);
+    }
+
     /*
      * synchronization locks:
      */
@@ -536,13 +551,59 @@ afp_setfork_err:
  */
 #define ENDBIT(a)  ((a) & 0x80)
 #define UNLOCKBIT(a) ((a) & 0x01)
-int afp_bytelock(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
+
+static off_t get_off_t(ibuf, is64)
+char   **ibuf;
+int     is64;
+{
+    u_int32_t             temp;
+    off_t                 ret;
+
+    memcpy(&temp, *ibuf, sizeof( temp ));
+    ret = ntohl(temp); /* ntohl is unsigned */
+    *ibuf += sizeof(temp);
+
+    if (is64) {
+        memcpy(&temp, *ibuf, sizeof( temp ));
+        *ibuf += sizeof(temp);
+        ret = ntohl(temp)| (ret << 32);
+    }
+    return ret;
+}
+
+/* ---------------------- */
+static int set_off_t(offset, rbuf, is64)
+off_t   offset;
+char   *rbuf;
+int     is64;
+{
+    u_int32_t  temp;
+    int        ret;
+
+    ret = 0;
+    if (is64) {
+        temp = htonl(offset >> 32);
+        memcpy(rbuf, &temp, sizeof( temp ));
+        rbuf += sizeof(temp);
+        ret = sizeof( temp );
+        offset &= 0xffffffff;
+    }
+    temp = htonl(offset);
+    memcpy(rbuf, &temp, sizeof( temp ));
+    ret += sizeof( temp );
+
+    return ret;
+}
+
+/* ---------------------- */
+static int byte_lock(obj, ibuf, ibuflen, rbuf, rbuflen, is64 )
+AFPObj  *obj;
 char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+int    ibuflen, *rbuflen;
+int     is64;
 {
     struct ofork       *ofork;
-    int32_t             offset, length;
+    off_t               offset, length;
     int                 eid;
     u_int16_t          ofrefnum;
     u_int8_t            flags;
@@ -568,20 +629,30 @@ int               ibuflen, *rbuflen;
     } else
         return AFPERR_PARAM;
 
-    memcpy(&offset, ibuf, sizeof( offset ));
-    offset = ntohl(offset);
-    ibuf += sizeof(offset);
-
-    memcpy(&length, ibuf, sizeof( length ));
-    length = ntohl(length);
-    if (length == 0xFFFFFFFF)
-        length = BYTELOCK_MAX;
-    else if (length <= 0) {
-        return AFPERR_PARAM;
-    } else if ((length >= AD_FILELOCK_BASE) &&
-               (ad_hfileno(ofork->of_ad) == -1))
-        return AFPERR_LOCK;
+    offset = get_off_t(&ibuf, is64);
+    length = get_off_t(&ibuf, is64);
 
+    if (is64) {
+        if (length == -1)
+            length = BYTELOCK_MAXL;
+        else if (length <= 0) {
+            return AFPERR_PARAM;
+        } else if ((length >= AD_FILELOCK_BASE) &&
+                   (ad_hfileno(ofork->of_ad) == -1)) {
+            return AFPERR_LOCK;
+        }
+    }
+    else {
+        if (length == 0xFFFFFFFF)
+            length = BYTELOCK_MAX;
+        else if (!length || (length & (1 << 31))) {
+            return AFPERR_PARAM;
+        } else if ((length >= AD_FILELOCK_BASE) &&
+                   (ad_hfileno(ofork->of_ad) == -1)) {
+            return AFPERR_LOCK;
+        }
+    }
+    
     if (ENDBIT(flags))
         offset += ad_size(ofork->of_ad, eid);
 
@@ -612,15 +683,31 @@ int               ibuflen, *rbuflen;
             break;
         }
     }
-
-    offset = htonl(offset);
-    memcpy(rbuf, &offset, sizeof( offset ));
-    *rbuflen = sizeof( offset );
+    *rbuflen = set_off_t (offset, rbuf, is64);
     return( AFP_OK );
 }
-#undef UNLOCKBIT
 
+/* --------------------------- */
+int afp_bytelock(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj  *obj;
+char   *ibuf, *rbuf;
+int    ibuflen, *rbuflen;
+{
+   return byte_lock ( obj, ibuf, ibuflen, rbuf, rbuflen , 0);
+}
+
+/* --------------------------- */
+int afp_bytelock_ext(obj, ibuf, ibuflen, rbuf, rbuflen )
+AFPObj  *obj;
+char   *ibuf, *rbuf;
+int    ibuflen, *rbuflen;
+{
+   return byte_lock ( obj, ibuf, ibuflen, rbuf, rbuflen , 1);
+}
+
+#undef UNLOCKBIT
 
+/* --------------------------- */
 static __inline__ int crlf( of )
 struct ofork   *of;
 {
@@ -701,18 +788,32 @@ static __inline__ ssize_t read_file(struct ofork *ofork, int eid,
     return AFP_OK;
 }
 
-int afp_read(obj, ibuf, ibuflen, rbuf, rbuflen)
+/* -----------------------------
+ * with ddp, afp_read can return fewer bytes than in reqcount 
+ * so return EOF only if read actually past end of file not
+ * if offset +reqcount > size of file
+ * e.g.:
+ * getfork size ==> 10430
+ * read fork offset 0 size 10752 ????  ==> 4264 bytes (without EOF)
+ * read fork offset 4264 size 6128 ==> 4264 (without EOF)
+ * read fork offset 9248 size 1508 ==> 1182 (EOF)
+ * 10752 is a bug in Mac 7.5.x finder 
+ *
+ * with dsi, should we check that reqcount < server quantum? 
+*/
+static int read_fork(obj, ibuf, ibuflen, rbuf, rbuflen, is64)
 AFPObj      *obj;
 char   *ibuf, *rbuf;
 int            ibuflen, *rbuflen;
+int is64;
 {
     struct ofork       *ofork;
     off_t              size;
-    int32_t            offset, saveoff, reqcount, savereqcount;
-    int                        cc, err, saveerr, eid, xlate = 0;
+    off_t              offset, saveoff, reqcount, savereqcount;
+    int                        cc, err, eid, xlate = 0;
     u_int16_t          ofrefnum;
     u_char             nlmask, nlchar;
-
+    
     ibuf += 2;
     memcpy(&ofrefnum, ibuf, sizeof( ofrefnum ));
     ibuf += sizeof( u_short );
@@ -727,17 +828,16 @@ int               ibuflen, *rbuflen;
         err = AFPERR_ACCESS;
         goto afp_read_err;
     }
+    offset   = get_off_t(&ibuf, is64);
+    reqcount = get_off_t(&ibuf, is64);
 
-    memcpy(&offset, ibuf, sizeof( offset ));
-    offset = ntohl( offset );
-    ibuf += sizeof( offset );
-    memcpy(&reqcount, ibuf, sizeof( reqcount ));
-    reqcount = ntohl( reqcount );
-    ibuf += sizeof( reqcount );
-
-    nlmask = *ibuf++;
-    nlchar = *ibuf++;
-
+    if (is64) {
+        nlmask = nlchar = 0;
+    }
+    else {
+        nlmask = *ibuf++;
+        nlchar = *ibuf++;
+    }
     /* if we wanted to be picky, we could add in the following
      * bit: if (afp_version == 11 && !(nlmask == 0xFF || !nlmask))
      */
@@ -770,16 +870,8 @@ int                ibuflen, *rbuflen;
         goto afp_read_err;
     }
 
-    /* subtract off the offset */
-    size -= offset;
     savereqcount = reqcount;
-    if (reqcount > size) {
-       reqcount = size;
-        err = AFPERR_EOF;
-    }
-
     saveoff = offset;
-    /* if EOF lock on the old reqcount, some prg may need it */
     if (ad_tmplock(ofork->of_ad, eid, ADLOCK_RD, saveoff, savereqcount) < 0) {
         err = AFPERR_LOCK;
         goto afp_read_err;
@@ -787,14 +879,11 @@ int               ibuflen, *rbuflen;
 
 #define min(a,b)       ((a)<(b)?(a):(b))
     *rbuflen = min( reqcount, *rbuflen );
-    saveerr = err;
     err = read_file(ofork, eid, offset, nlmask, nlchar, rbuf, rbuflen,
                     xlate);
     if (err < 0)
         goto afp_read_done;
-    if (saveerr < 0) {
-       err = saveerr;
-    }
+
     /* dsi can stream requests. we can only do this if we're not checking
      * for an end-of-line character. oh well. */
     if ((obj->proto == AFPPROTO_DSI) && (*rbuflen < reqcount) && !nlmask) {
@@ -802,9 +891,15 @@ int                ibuflen, *rbuflen;
 
         if (obj->options.flags & OPTION_DEBUG) {
             printf( "(read) reply: %d/%d, %d\n", *rbuflen,
-                    reqcount, dsi->clientID);
+                    (int) reqcount, dsi->clientID);
             bprint(rbuf, *rbuflen);
         }
+        /* subtract off the offset */
+        size -= offset;
+        if (reqcount > size) {
+          reqcount = size;
+           err = AFPERR_EOF;
+        }
 
         offset += *rbuflen;
 
@@ -873,6 +968,25 @@ afp_read_err:
     return err;
 }
 
+/* ---------------------- */
+int afp_read(obj, ibuf, ibuflen, rbuf, rbuflen)
+AFPObj  *obj;
+char   *ibuf, *rbuf;
+int    ibuflen, *rbuflen;
+{
+    return read_fork(obj, ibuf, ibuflen, rbuf, rbuflen, 0);
+}
+
+/* ---------------------- */
+int afp_read_ext(obj, ibuf, ibuflen, rbuf, rbuflen)
+AFPObj  *obj;
+char   *ibuf, *rbuf;
+int    ibuflen, *rbuflen;
+{
+    return read_fork(obj, ibuf, ibuflen, rbuf, rbuflen, 1);
+}
+
+/* ---------------------- */
 int afp_flush(obj, ibuf, ibuflen, rbuf, rbuflen )
 AFPObj      *obj;
 char   *ibuf, *rbuf;
@@ -922,7 +1036,8 @@ int flushfork( ofork )
 struct ofork   *ofork;
 {
     struct timeval tv;
-    int len, err = 0, doflush = 0;
+
+    int err = 0, doflush = 0;
 
     if ( ad_dfileno( ofork->of_ad ) != -1 &&
             fsync( ad_dfileno( ofork->of_ad )) < 0 ) {
@@ -931,32 +1046,21 @@ struct ofork     *ofork;
         err = -1;
     }
 
-    if ( ad_hfileno( ofork->of_ad ) != -1 ) {
+    if ( ad_hfileno( ofork->of_ad ) != -1 && 
+           (ofork->of_flags & AFPFORK_RSRC)) {
 
         /* read in the rfork length */
-        len = ad_getentrylen(ofork->of_ad, ADEID_RFORK);
         ad_refresh(ofork->of_ad);
 
         /* set the date if we're dirty */
-        if ((ofork->of_flags & AFPFORK_DIRTY) &&
-                (gettimeofday(&tv, NULL) == 0)) {
+        if ((ofork->of_flags & AFPFORK_DIRTY) && !gettimeofday(&tv, NULL)) {
             ad_setdate(ofork->of_ad, AD_DATE_MODIFY|AD_DATE_UNIX, tv.tv_sec);
             ofork->of_flags &= ~AFPFORK_DIRTY;
             doflush++;
         }
 
-        /* if we're actually flushing this fork, make sure to set the
-         * length. otherwise, just use the stored length */
-        if ((ofork->of_flags & AFPFORK_RSRC) &&
-                (len != ad_getentrylen(ofork->of_ad, ADEID_RFORK))) {
-            ad_setentrylen(ofork->of_ad, ADEID_RFORK, len);
-            doflush++;
-        }
-
-
-        /* flush the header (if it is a resource fork) */
-        if (ofork->of_flags & AFPFORK_RSRC)
-            if (doflush && (ad_flush(ofork->of_ad, ADFLAGS_HF) < 0))
+        /* flush the header */
+        if (doflush && ad_flush(ofork->of_ad, ADFLAGS_HF) < 0)
                 err = -1;
 
         if (fsync( ad_hfileno( ofork->of_ad )) < 0)
@@ -977,7 +1081,7 @@ int                ibuflen, *rbuflen;
 {
     struct ofork       *ofork;
     struct timeval      tv;
-    int                        adflags, aint, doflush = 0;
+    int                        adflags, doflush = 0;
     u_int16_t          ofrefnum;
 
     *rbuflen = 0;
@@ -997,26 +1101,19 @@ int              ibuflen, *rbuflen;
 
     if ( ad_hfileno( ofork->of_ad ) != -1 ) {
         adflags |= ADFLAGS_HF;
-
-        aint = ad_getentrylen( ofork->of_ad, ADEID_RFORK );
-        ad_refresh( ofork->of_ad );
-        if ((ofork->of_flags & AFPFORK_DIRTY) &&
-                (gettimeofday(&tv, NULL) == 0)) {
-            ad_setdate(ofork->of_ad, AD_DATE_MODIFY | AD_DATE_UNIX,
-                       tv.tv_sec);
-            doflush++;
-        }
-
         /*
          * Only set the rfork's length if we're closing the rfork.
          */
-        if ((ofork->of_flags & AFPFORK_RSRC) && aint !=
-                ad_getentrylen( ofork->of_ad, ADEID_RFORK )) {
-            ad_setentrylen( ofork->of_ad, ADEID_RFORK, aint );
-            doflush++;
-        }
-        if ( doflush ) {
-            ad_flush( ofork->of_ad, adflags );
+        if ((ofork->of_flags & AFPFORK_RSRC)) {
+            ad_refresh( ofork->of_ad );
+            if ((ofork->of_flags & AFPFORK_DIRTY) &&
+                      !gettimeofday(&tv, NULL)) {
+                ad_setdate(ofork->of_ad, AD_DATE_MODIFY | AD_DATE_UNIX,tv.tv_sec);
+               doflush++;
+            }
+            if ( doflush ) {
+                ad_flush( ofork->of_ad, adflags );
+            }
         }
     }
 
@@ -1069,13 +1166,14 @@ static __inline__ ssize_t write_file(struct ofork *ofork, int eid,
 /* FPWrite. NOTE: on an error, we always use afp_write_err as
  * the client may have sent us a bunch of data that's not reflected 
  * in reqcount et al. */
-int afp_write(obj, ibuf, ibuflen, rbuf, rbuflen)
+static int write_fork(obj, ibuf, ibuflen, rbuf, rbuflen, is64)
 AFPObj              *obj;
 char                *ibuf, *rbuf;
 int                 ibuflen, *rbuflen;
+int                 is64;
 {
     struct ofork       *ofork;
-    int32_t            offset, saveoff, reqcount;
+    off_t              offset, saveoff, reqcount;
     int                        endflag, eid, xlate = 0, err = AFP_OK;
     u_int16_t          ofrefnum;
     ssize_t             cc;
@@ -1086,12 +1184,9 @@ int                 ibuflen, *rbuflen;
     ibuf++;
     memcpy(&ofrefnum, ibuf, sizeof( ofrefnum ));
     ibuf += sizeof( ofrefnum );
-    memcpy(&offset, ibuf, sizeof( offset ));
-    offset = ntohl( offset );
-    ibuf += sizeof( offset );
-    memcpy(&reqcount, ibuf, sizeof( reqcount ));
-    reqcount = ntohl( reqcount );
-    ibuf += sizeof( reqcount );
+
+    offset   = get_off_t(&ibuf, is64);
+    reqcount = get_off_t(&ibuf, is64);
 
     if (( ofork = of_find( ofrefnum )) == NULL ) {
         LOG(log_error, logtype_afpd, "afp_write: of_find");
@@ -1136,8 +1231,7 @@ int                 ibuflen, *rbuflen;
 
     if (!reqcount) { /* handle request counts of 0 */
         err = AFP_OK;
-        offset = htonl(offset);
-        memcpy(rbuf, &offset, sizeof(offset));
+        *rbuflen = set_off_t (offset, rbuf, is64);
         goto afp_write_err;
     }
 
@@ -1239,13 +1333,7 @@ int                 ibuflen, *rbuflen;
     if ( ad_hfileno( ofork->of_ad ) != -1 )
         ofork->of_flags |= AFPFORK_DIRTY;
 
-    offset = htonl( offset );
-#if defined(__GNUC__) && defined(HAVE_GCC_MEMCPY_BUG)
-    bcopy(&offset, rbuf, sizeof(offset));
-#else /* __GNUC__ && HAVE_GCC_MEMCPY_BUG */
-    memcpy(rbuf, &offset, sizeof(offset));
-#endif /* __GNUC__ && HAVE_GCC_MEMCPY_BUG */
-    *rbuflen = sizeof(offset);
+    *rbuflen = set_off_t (offset, rbuf, is64);
     return( AFP_OK );
 
 afp_write_err:
@@ -1258,7 +1346,27 @@ afp_write_err:
     return err;
 }
 
+/* ---------------------------- */
+int afp_write(obj, ibuf, ibuflen, rbuf, rbuflen)
+AFPObj              *obj;
+char                *ibuf, *rbuf;
+int                 ibuflen, *rbuflen;
+{
+    return write_fork(obj, ibuf, ibuflen, rbuf, rbuflen, 0);
+}
 
+/* ---------------------------- 
+ * FIXME need to deal with SIGXFSZ signal
+*/
+int afp_write_ext(obj, ibuf, ibuflen, rbuf, rbuflen)
+AFPObj              *obj;
+char                *ibuf, *rbuf;
+int                 ibuflen, *rbuflen;
+{
+    return write_fork(obj, ibuf, ibuflen, rbuf, rbuflen, 1);
+}
+
+/* ---------------------------- */
 int afp_getforkparams(obj, ibuf, ibuflen, rbuf, rbuflen )
 AFPObj      *obj;
 char   *ibuf, *rbuf;
@@ -1267,6 +1375,7 @@ int               ibuflen, *rbuflen;
     struct ofork       *ofork;
     int                        buflen, ret;
     u_int16_t          ofrefnum, bitmap;
+    u_int16_t          attrbits = 0;
 
     ibuf += 2;
     memcpy(&ofrefnum, ibuf, sizeof( ofrefnum ));
@@ -1280,9 +1389,18 @@ int              ibuflen, *rbuflen;
         LOG(log_error, logtype_afpd, "afp_getforkparams: of_find");
         return( AFPERR_PARAM );
     }
+    attrbits = ((ofork->of_ad->ad_df.adf_refcount > 0) ? ATTRBIT_DOPEN : 0);
+    attrbits |= ((ofork->of_ad->ad_hf.adf_refcount > ofork->of_ad->ad_df.adf_refcount) ? ATTRBIT_ROPEN : 0);
+
+    if ( ad_hfileno( ofork->of_ad ) != -1 ) {
+        if ( ad_refresh( ofork->of_ad ) < 0 ) {
+            LOG(log_error, logtype_afpd, "getforkparams: ad_refresh: %s", strerror(errno) );
+            return( AFPERR_PARAM );
+        }
+    }
 
     if (( ret = getforkparams( ofork, bitmap,
-                               rbuf + sizeof( u_short ), &buflen, 0 )) != AFP_OK ) {
+                               rbuf + sizeof( u_short ), &buflen, attrbits )) != AFP_OK ) {
         return( ret );
     }