]> arthur.barton.de Git - netatalk.git/blobdiff - libatalk/adouble/ad_attr.c
Only update adouble file with CNID on mismatch.
[netatalk.git] / libatalk / adouble / ad_attr.c
index 7168f8fecb5ba0da972bc872efb3fec336f55ce4..e90b6b8e0268c9ff41dc7c23e14ae5198fe49f87 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: ad_attr.c,v 1.5 2005-04-28 20:49:51 bfernhomberg Exp $
+ * $Id: ad_attr.c,v 1.14 2010-01-06 14:05:15 franklahm Exp $
  */
 
 #ifdef HAVE_CONFIG_H
 #define FILEIOFF_ATTR 14
 #define AFPFILEIOFF_ATTR 2
 
+/* 
+   Note:
+   the "shared" and "invisible" attributes are opaque and stored and
+   retrieved from the FinderFlags. This fixes Bug #2802236:
+   <https://sourceforge.net/tracker/?func=detail&aid=2802236&group_id=8642&atid=108642>
+ */
 int ad_getattr(const struct adouble *ad, u_int16_t *attr)
 {
-   *attr = 0;
-        
-   if (ad->ad_version == AD_VERSION1) {
-       if (ad_getentryoff(ad, ADEID_FILEI)) {
-           memcpy(attr, ad_entry(ad, ADEID_FILEI) + FILEIOFF_ATTR,
-                 sizeof(u_int16_t));
-       }
-   }
+    u_int16_t fflags;
+    *attr = 0;
+
+    if (ad->ad_version == AD_VERSION1) {
+        if (ad_getentryoff(ad, ADEID_FILEI)) {
+            memcpy(attr, ad_entry(ad, ADEID_FILEI) + FILEIOFF_ATTR,
+                   sizeof(u_int16_t));
+        }
+    }
 #if AD_VERSION == AD_VERSION2
-   else if (ad->ad_version == AD_VERSION2) {
-       if (ad_getentryoff(ad, ADEID_AFPFILEI)) {
-           memcpy(attr, ad_entry(ad, ADEID_AFPFILEI) + AFPFILEIOFF_ATTR,
-                 sizeof(u_int16_t));
-       }
-   }      
+    else if (ad->ad_version == AD_VERSION2) {
+        if (ad_getentryoff(ad, ADEID_AFPFILEI)) {
+            memcpy(attr, ad_entry(ad, ADEID_AFPFILEI) + AFPFILEIOFF_ATTR, 2);
+
+            /* Now get opaque flags from FinderInfo */
+            memcpy(&fflags, ad_entry(ad, ADEID_FINDERI) + FINDERINFO_FRFLAGOFF, 2);
+            if (fflags & htons(FINDERINFO_INVISIBLE))
+                *attr |= htons(ATTRBIT_INVISIBLE);
+            else
+                *attr &= htons(~ATTRBIT_INVISIBLE);
+ /*
+   This one is tricky, I actually got it wrong the first time:
+   for directories bit 1<<1 is ATTRBIT_EXPFLDR and is NOT opaque !
+ */
+            if ( ! (ad->ad_adflags & ADFLAGS_DIR)) {
+                if (fflags & htons(FINDERINFO_ISHARED))
+                    *attr |= htons(ATTRBIT_MULTIUSER);
+                else
+                    *attr &= htons(~ATTRBIT_MULTIUSER);
+            }
+        }
+    }
 #endif
-   else 
-      return -1;
+    else
+        return -1;
+
+    *attr |= htons(ad->ad_open_forks);
 
-   return 0;
+    return 0;
 }
 
 /* ----------------- */
-int ad_setattr(const struct adouble *ad, const u_int16_t attr)
+int ad_setattr(const struct adouble *ad, const u_int16_t attribute)
 {
-   if (ad->ad_version == AD_VERSION1) {
-       if (ad_getentryoff(ad, ADEID_FILEI)) {
-           memcpy(ad_entry(ad, ADEID_FILEI) + FILEIOFF_ATTR, &attr,
-                  sizeof(attr));
-       }
-   }      
+    u_int16_t *fflags;
+
+    /* we don't save open forks indicator */
+    u_int16_t attr = attribute & ~htons(ATTRBIT_DOPEN | ATTRBIT_ROPEN);
+
+    /* Proactively (10.4 does indeed try to set ATTRBIT_MULTIUSER (=ATTRBIT_EXPFLDR)
+       for dirs with SetFile -a M <dir> ) disable all flags not defined for dirs. */
+    if (ad->ad_adflags & ADFLAGS_DIR)
+        attr &= ~(ATTRBIT_MULTIUSER | ATTRBIT_NOWRITE | ATTRBIT_NOCOPY);
+
+    if (ad->ad_version == AD_VERSION1) {
+        if (ad_getentryoff(ad, ADEID_FILEI)) {
+            memcpy(ad_entry(ad, ADEID_FILEI) + FILEIOFF_ATTR, &attr,
+                   sizeof(attr));
+        }
+    }
 #if AD_VERSION == AD_VERSION2
-   else if (ad->ad_version == AD_VERSION2) {
-       if (ad_getentryoff(ad, ADEID_AFPFILEI)) {
-            memcpy(ad_entry(ad, ADEID_AFPFILEI) + AFPFILEIOFF_ATTR, &attr,
-                   sizeof(attr));
-       }
-   }      
+    else if (ad->ad_version == AD_VERSION2) {
+        if (ad_getentryoff(ad, ADEID_AFPFILEI)) {
+            memcpy(ad_entry(ad, ADEID_AFPFILEI) + AFPFILEIOFF_ATTR, &attr, sizeof(attr));
+            
+            /* Now set opaque flags in FinderInfo too */
+            fflags = (u_int16_t *)ad_entry(ad, ADEID_FINDERI) + FINDERINFO_FRFLAGOFF;
+            if (attr & htons(ATTRBIT_INVISIBLE))
+                *fflags |= htons(FINDERINFO_INVISIBLE);
+            else
+                *fflags &= htons(~FINDERINFO_INVISIBLE);
+
+            /* See above comment in ad_getattr() */
+            if (attr & htons(ATTRBIT_MULTIUSER)) {
+                if ( ! (ad->ad_adflags & ADFLAGS_DIR) )
+                    *fflags |= htons(FINDERINFO_ISHARED);
+            } else
+                    *fflags &= htons(~FINDERINFO_ISHARED);
+        }
+    }
 #endif
-   else 
-      return -1;
+    else
+        return -1;
 
-   return 0;
+    return 0;
 }
 
-/* -------------- 
+/* --------------
  * save file/folder ID in AppleDoubleV2 netatalk private parameters
  * return 1 if resource fork has been modified
-*/
+ */
 #if AD_VERSION == AD_VERSION2
 int ad_setid (struct adouble *adp, const dev_t dev, const ino_t ino , const u_int32_t id, const cnid_t did, const void *stamp)
 {
-    if (adp->ad_flags == AD_VERSION2  && ( adp->ad_options & ADVOL_CACHE) &&
-                ad_getentryoff(adp, ADEID_PRIVDEV) &&
-                sizeof(dev_t) == ADEDLEN_PRIVDEV && sizeof(ino_t) == ADEDLEN_PRIVINO) 
-    {
+    if ((adp->ad_flags == AD_VERSION2) && (adp->ad_options & ADVOL_CACHE)) {
+
+        /* ad_getid depends on this to detect presence of ALL entries */
+        ad_setentrylen( adp, ADEID_PRIVID, sizeof(id));
+        memcpy(ad_entry( adp, ADEID_PRIVID ), &id, sizeof(id));
+
         ad_setentrylen( adp, ADEID_PRIVDEV, sizeof(dev_t));
-       if ((adp->ad_options & ADVOL_NODEV)) {
+        if ((adp->ad_options & ADVOL_NODEV)) {
             memset(ad_entry( adp, ADEID_PRIVDEV ), 0, sizeof(dev_t));
-        }
-       else {
+        } else {
             memcpy(ad_entry( adp, ADEID_PRIVDEV ), &dev, sizeof(dev_t));
         }
 
         ad_setentrylen( adp, ADEID_PRIVINO, sizeof(ino_t));
         memcpy(ad_entry( adp, ADEID_PRIVINO ), &ino, sizeof(ino_t));
 
-       ad_setentrylen( adp, ADEID_PRIVID, sizeof(id));
-        memcpy(ad_entry( adp, ADEID_PRIVID ), &id, sizeof(id));
-
         ad_setentrylen( adp, ADEID_DID, sizeof(did));
         memcpy(ad_entry( adp, ADEID_DID ), &did, sizeof(did));
 
@@ -97,43 +143,53 @@ int ad_setid (struct adouble *adp, const dev_t dev, const ino_t ino , const u_in
 /* ----------------------------- */
 u_int32_t ad_getid (struct adouble *adp, const dev_t st_dev, const ino_t st_ino , const cnid_t did, const void *stamp)
 {
-u_int32_t aint = 0;
-dev_t  dev;
-ino_t  ino;
-cnid_t a_did;
-char   temp[ADEDLEN_PRIVSYN];
+    u_int32_t aint = 0;
+    dev_t  dev;
+    ino_t  ino;
+    cnid_t a_did;
+    char   temp[ADEDLEN_PRIVSYN];
 
-    /* look in AD v2 header 
+    /* look in AD v2 header
      * note inode and device are opaques and not in network order
      * only use the ID if adouble is writable for us.
-    */
-    if (adp && ( adp->ad_options & ADVOL_CACHE) && ( adp->ad_hf.adf_flags & O_RDWR )
-            && sizeof(dev_t) == ad_getentrylen(adp, ADEID_PRIVDEV)
-            && sizeof(ino_t) == ad_getentrylen(adp,ADEID_PRIVINO)
-            && sizeof(temp) == ad_getentrylen(adp,ADEID_PRIVSYN)
-            && sizeof(cnid_t) == ad_getentrylen(adp, ADEID_DID)
-            && sizeof(cnid_t) == ad_getentrylen(adp, ADEID_PRIVID)
-    ) {
+     */
+    if (adp
+        && (adp->ad_options & ADVOL_CACHE)
+        && (adp->ad_md->adf_flags & O_RDWR )
+        && (sizeof(dev_t) == ad_getentrylen(adp, ADEID_PRIVDEV)) /* One check to ensure ALL values are there */
+        ) {
         memcpy(&dev, ad_entry(adp, ADEID_PRIVDEV), sizeof(dev_t));
         memcpy(&ino, ad_entry(adp, ADEID_PRIVINO), sizeof(ino_t));
         memcpy(temp, ad_entry(adp, ADEID_PRIVSYN), sizeof(temp));
         memcpy(&a_did, ad_entry(adp, ADEID_DID), sizeof(cnid_t));
 
-        if (  ((adp->ad_options & ADVOL_NODEV) || dev == st_dev)
-              && ino == st_ino && a_did == did 
-              && !memcmp(stamp, temp, sizeof(temp))) { 
+        if ( ((adp->ad_options & ADVOL_NODEV) || dev == st_dev)
+             && ino == st_ino
+             && (!did || a_did == did)
+             && (memcmp(stamp, temp, sizeof(temp)) == 0) ) {
             memcpy(&aint, ad_entry(adp, ADEID_PRIVID), sizeof(aint));
             return aint;
         }
     }
-    return 0; 
+    return 0;
 }
 
+/* ----------------------------- */
+u_int32_t ad_forcegetid (struct adouble *adp)
+{
+    u_int32_t aint = 0;
+
+    if (adp && (adp->ad_options & ADVOL_CACHE)) {
+        memcpy(&aint, ad_entry(adp, ADEID_PRIVID), sizeof(aint));
+        return aint;
+    }
+    return 0;
+}
 #endif
 
-/* ----------------- 
+/* -----------------
  * set resource fork filename attribute.
-*/
+ */
 int ad_setname(struct adouble *ad, const char *path)
 {
     if (ad_getentryoff(ad, ADEID_NAME)) {