]> arthur.barton.de Git - netatalk.git/blobdiff - libatalk/vfs/vfs.c
speed up validupath_adouble for the common case: it's not a file with a leading ...
[netatalk.git] / libatalk / vfs / vfs.c
index b160646e2b076fc0cb95239806f903e655f379f2..a423bf63357eaa01ddeeccb5cd1d5b95c4b39dbe 100644 (file)
@@ -1,5 +1,6 @@
 /*
     Copyright (c) 2004 Didier Gautheron
+    Copyright (c) 2009 Frank Lahm
  
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
 
 #include <atalk/afp.h>    
 #include <atalk/adouble.h>
-#include <atalk/vfs.h>
 #include <atalk/ea.h>
 #include <atalk/acl.h>
 #include <atalk/logger.h>
 #include <atalk/util.h>
 #include <atalk/volume.h>
+#include <atalk/vfs.h>
 #include <atalk/directory.h>
 #include <atalk/unix.h>
 
@@ -85,15 +86,20 @@ for_each_adouble(const char *from, const char *name, rf_loop fn, void *data, int
 
 static int netatalk_name(const char *name)
 {
-    return strcasecmp(name,".AppleDB") &&
-        strcasecmp(name,".AppleDouble") &&
+    return strcasecmp(name,".AppleDouble") &&
+        strcasecmp(name,".AppleDB") &&
         strcasecmp(name,".AppleDesktop");
 }
 
 static int validupath_adouble(VFS_FUNC_ARGS_VALIDUPATH)
 {
-    return (vol->v_flags & AFPVOL_USEDOTS) ? 
-        netatalk_name(name) && strcasecmp(name,".Parent"): name[0] != '.';
+    if (name[0] != '.')
+        return 1;
+    
+    if (!(vol->v_flags & AFPVOL_USEDOTS))
+        return 0;
+        
+    return netatalk_name(name) && strcasecmp(name,".Parent");
 }                                           
 
 /* ----------------- */
@@ -102,7 +108,7 @@ static int RF_chown_adouble(VFS_FUNC_ARGS_CHOWN)
     struct stat st;
     char        *ad_p;
 
-    ad_p = vol->vfs->ad_path(path, ADFLAGS_HF );
+    ad_p = vol->ad_path(path, ADFLAGS_HF );
 
     if ( stat( ad_p, &st ) < 0 )
         return 0; /* ignore */
@@ -153,13 +159,13 @@ static int adouble_setfilmode(const char * name, mode_t mode, struct stat *st, m
 
 static int RF_setfilmode_adouble(VFS_FUNC_ARGS_SETFILEMODE)
 {
-    return adouble_setfilmode(vol->vfs->ad_path( name, ADFLAGS_HF ), mode, st, vol->v_umask);
+    return adouble_setfilmode(vol->ad_path(name, ADFLAGS_HF ), mode, st, vol->v_umask);
 }
 
 /* ----------------- */
 static int RF_setdirunixmode_adouble(VFS_FUNC_ARGS_SETDIRUNIXMODE)
 {
-    char *adouble = vol->vfs->ad_path( name, ADFLAGS_DIR );
+    char *adouble = vol->ad_path(name, ADFLAGS_DIR );
     int  dropbox = vol->v_flags;
 
     if (dir_rx_set(mode)) {
@@ -167,7 +173,7 @@ static int RF_setdirunixmode_adouble(VFS_FUNC_ARGS_SETDIRUNIXMODE)
             return -1;
     }
 
-    if (adouble_setfilmode(vol->vfs->ad_path( name, ADFLAGS_DIR ), mode, st, vol->v_umask) < 0) 
+    if (adouble_setfilmode(vol->ad_path(name, ADFLAGS_DIR ), mode, st, vol->v_umask) < 0) 
         return -1;
 
     if (!dir_rx_set(mode)) {
@@ -200,7 +206,7 @@ static int RF_setdirmode_adouble(VFS_FUNC_ARGS_SETDIRMODE)
 {
     int   dropbox = vol->v_flags;
     mode_t hf_mode = ad_hf_mode(mode);
-    char  *adouble = vol->vfs->ad_path( name, ADFLAGS_DIR );
+    char  *adouble = vol->ad_path(name, ADFLAGS_DIR );
     char  *adouble_p = ad_dir(adouble);
 
     if (dir_rx_set(mode)) {
@@ -241,7 +247,7 @@ static int RF_setdirowner_adouble(VFS_FUNC_ARGS_SETDIROWNER)
     owner.uid = uid;
     owner.gid = gid;
 
-    adouble_p = ad_dir(vol->vfs->ad_path( name, ADFLAGS_DIR ));
+    adouble_p = ad_dir(vol->ad_path(name, ADFLAGS_DIR ));
 
     if (for_each_adouble("setdirowner", adouble_p, setdirowner_adouble_loop, &owner, noadouble, vol->v_umask)) 
         return -1;
@@ -266,7 +272,7 @@ static int RF_setdirowner_adouble(VFS_FUNC_ARGS_SETDIROWNER)
 /* ----------------- */
 static int RF_deletefile_adouble(VFS_FUNC_ARGS_DELETEFILE)
 {
-       return netatalk_unlink(vol->vfs->ad_path( file, ADFLAGS_HF));
+       return netatalk_unlink(vol->ad_path(file, ADFLAGS_HF));
 }
 
 /* ----------------- */
@@ -275,8 +281,8 @@ static int RF_renamefile_adouble(VFS_FUNC_ARGS_RENAMEFILE)
     char  adsrc[ MAXPATHLEN + 1];
     int   err = 0;
 
-    strcpy( adsrc, vol->vfs->ad_path( src, 0 ));
-    if (unix_rename( adsrc, vol->vfs->ad_path( dst, 0 )) < 0) {
+    strcpy( adsrc, vol->ad_path(src, 0 ));
+    if (unix_rename( adsrc, vol->ad_path(dst, 0 )) < 0) {
         struct stat st;
 
         err = errno;
@@ -296,7 +302,7 @@ static int RF_renamefile_adouble(VFS_FUNC_ARGS_RENAMEFILE)
             ad_init(&ad, vol->v_adouble, vol->v_ad_options); 
             if (!ad_open(dst, ADFLAGS_HF, O_RDWR | O_CREAT, 0666, &ad)) {
                ad_close(&ad, ADFLAGS_HF);
-               if (!unix_rename( adsrc, vol->vfs->ad_path( dst, 0 )) ) 
+               if (!unix_rename( adsrc, vol->ad_path(dst, 0 )) ) 
                    err = 0;
                 else 
                    err = errno;
@@ -388,7 +394,7 @@ static int RF_chown_ads(VFS_FUNC_ARGS_CHOWN)
     owner.gid = gid;
 
 
-    ad_p = ad_dir(vol->vfs->ad_path(path, ADFLAGS_HF ));
+    ad_p = ad_dir(vol->ad_path(path, ADFLAGS_HF ));
 
     if ( stat( ad_p, &st ) < 0 ) {
        /* ignore */
@@ -493,13 +499,13 @@ static int ads_setfilmode(const char * name, mode_t mode, struct stat *st, mode_
 
 static int RF_setfilmode_ads(VFS_FUNC_ARGS_SETFILEMODE)
 {
-    return ads_setfilmode(ad_dir(vol->vfs->ad_path( name, ADFLAGS_HF )), mode, st, vol->v_umask);
+    return ads_setfilmode(ad_dir(vol->ad_path(name, ADFLAGS_HF )), mode, st, vol->v_umask);
 }
 
 /* ------------------- */
 static int RF_setdirunixmode_ads(VFS_FUNC_ARGS_SETDIRUNIXMODE)
 {
-    char *adouble = vol->vfs->ad_path( name, ADFLAGS_DIR );
+    char *adouble = vol->ad_path(name, ADFLAGS_DIR );
     char   ad_p[ MAXPATHLEN + 1];
     int dropbox = vol->v_flags;
 
@@ -516,7 +522,7 @@ static int RF_setdirunixmode_ads(VFS_FUNC_ARGS_SETDIRUNIXMODE)
             return -1;
     }
 
-    if (ads_setfilmode(ad_dir(vol->vfs->ad_path( name, ADFLAGS_DIR)), mode, st, vol->v_umask) < 0)
+    if (ads_setfilmode(ad_dir(vol->ad_path(name, ADFLAGS_DIR)), mode, st, vol->v_umask) < 0)
         return -1;
 
     if (!dir_rx_set(mode)) {
@@ -564,7 +570,7 @@ static int setdirmode_ads_loop(struct dirent *de _U_, char *name, void *data, in
 
 static int RF_setdirmode_ads(VFS_FUNC_ARGS_SETDIRMODE)
 {
-    char *adouble = vol->vfs->ad_path( name, ADFLAGS_DIR );
+    char *adouble = vol->ad_path(name, ADFLAGS_DIR );
     char   ad_p[ MAXPATHLEN + 1];
     struct dir_mode param;
 
@@ -627,7 +633,7 @@ static int RF_setdirowner_ads(VFS_FUNC_ARGS_SETDIROWNER)
     owner.uid = uid;
     owner.gid = gid;
 
-    strlcpy(adouble_p, ad_dir(vol->vfs->ad_path( name, ADFLAGS_DIR )), sizeof(adouble_p));
+    strlcpy(adouble_p, ad_dir(vol->ad_path(name, ADFLAGS_DIR )), sizeof(adouble_p));
 
     if (for_each_adouble("setdirowner", ad_dir(adouble_p), setdirowner_ads_loop, &owner, noadouble, 0)) 
         return -1;
@@ -652,7 +658,7 @@ static int RF_setdirowner_ads(VFS_FUNC_ARGS_SETDIROWNER)
 /* ------------------- */
 static int RF_deletefile_ads(VFS_FUNC_ARGS_DELETEFILE)
 {
-    char *ad_p = ad_dir(vol->vfs->ad_path(file, ADFLAGS_HF ));
+    char *ad_p = ad_dir(vol->ad_path(file, ADFLAGS_HF ));
 
     return ads_delete_rf(ad_p);
 }
@@ -663,8 +669,8 @@ static int RF_renamefile_ads(VFS_FUNC_ARGS_RENAMEFILE)
     char  adsrc[ MAXPATHLEN + 1];
     int   err = 0;
 
-    strcpy( adsrc, ad_dir(vol->vfs->ad_path( src, 0 )));
-    if (unix_rename( adsrc, ad_dir(vol->vfs->ad_path( dst, 0 ))) < 0) {
+    strcpy( adsrc, ad_dir(vol->ad_path(src, 0 )));
+    if (unix_rename( adsrc, ad_dir(vol->ad_path(dst, 0 ))) < 0) {
         struct stat st;
 
         err = errno;
@@ -687,7 +693,7 @@ static int RF_renamefile_ads(VFS_FUNC_ARGS_RENAMEFILE)
 
                /* We must delete it */
                RF_deletefile_ads(vol, dst );
-               if (!unix_rename( adsrc, ad_dir(vol->vfs->ad_path( dst, 0 ))) ) 
+               if (!unix_rename( adsrc, ad_dir(vol->ad_path(dst, 0 ))) ) 
                    err = 0;
                 else 
                    err = errno;
@@ -718,19 +724,19 @@ static int RF_renamedir_osx(VFS_FUNC_ARGS_RENAMEDIR)
 {
     /* We simply move the corresponding ad file as well */
     char   tempbuf[258]="._";
-    return rename(vol->vfs->ad_path(oldpath,0),strcat(tempbuf,newpath));
+    return rename(vol->ad_path(oldpath,0),strcat(tempbuf,newpath));
 }
 
 /* ---------------- */
 static int RF_deletecurdir_osx(VFS_FUNC_ARGS_DELETECURDIR)
 {
-    return netatalk_unlink( vol->vfs->ad_path(".",0) );
+    return netatalk_unlink( vol->ad_path(".",0) );
 }
 
 /* ---------------- */
 static int RF_setdirunixmode_osx(VFS_FUNC_ARGS_SETDIRUNIXMODE)
 {
-    return adouble_setfilmode(vol->vfs->ad_path( name, ADFLAGS_DIR ), mode, st, vol->v_umask);
+    return adouble_setfilmode(vol->ad_path(name, ADFLAGS_DIR ), mode, st, vol->v_umask);
 }
 
 /* ---------------- */
@@ -751,9 +757,9 @@ static int RF_renamefile_osx(VFS_FUNC_ARGS_RENAMEFILE)
     char  adsrc[ MAXPATHLEN + 1];
     int   err = 0;
 
-    strcpy( adsrc, vol->vfs->ad_path( src, 0 ));
+    strcpy( adsrc, vol->ad_path(src, 0 ));
 
-    if (unix_rename( adsrc, vol->vfs->ad_path( dst, 0 )) < 0) {
+    if (unix_rename( adsrc, vol->ad_path(dst, 0 )) < 0) {
         struct stat st;
 
         err = errno;
@@ -778,25 +784,19 @@ static int RF_renamefile_osx(VFS_FUNC_ARGS_RENAMEFILE)
  * following funcs are called in order to give them a chance.
  */
 
-/* 
- * Currently the maximum will be:
- * main adouble module + EA module + ACL module + NULL = 4.
- * NULL is an end of array marker.
- */
-static struct vfs_ops *vfs[4] = { NULL };
-
 /* 
  * Define most VFS funcs with macros as they all do the same.
  * Only "ad_path" and "validupath" will NOT do stacking and only
  * call the func from the first module.
  */
+
 #define VFS_MFUNC(name, args, vars) \
     static int vfs_ ## name(args) \
     { \
         int i = 0, ret = AFP_OK, err; \
-        while (vfs[i]) { \
-            if (vfs[i]->vfs_ ## name) { \
-                err = vfs[i]->vfs_ ## name (vars); \
+        while (vol->vfs_modules[i]) { \
+            if (vol->vfs_modules[i]->vfs_ ## name) { \
+                err = vol->vfs_modules[i]->vfs_ ## name (vars); \
                 if ((ret == AFP_OK) && (err != AFP_OK)) \
                     ret = err; \
             } \
@@ -823,22 +823,16 @@ VFS_MFUNC(ea_list, VFS_FUNC_ARGS_EA_LIST, VFS_FUNC_VARS_EA_LIST)
 VFS_MFUNC(ea_set, VFS_FUNC_ARGS_EA_SET, VFS_FUNC_VARS_EA_SET)
 VFS_MFUNC(ea_remove, VFS_FUNC_ARGS_EA_REMOVE, VFS_FUNC_VARS_EA_REMOVE)
 
-static char *vfs_path(const char *path, int flags)
-{
-    return vfs[0]->ad_path(path, flags);
-}
-
 static int vfs_validupath(VFS_FUNC_ARGS_VALIDUPATH)
 {
-    return vfs[0]->vfs_validupath(VFS_FUNC_VARS_VALIDUPATH);
+    return vol->vfs_modules[0]->vfs_validupath(VFS_FUNC_VARS_VALIDUPATH);
 }
 
 /*
  * These function pointers get called from the lib users via vol->vfs->func.
  * These funcs are defined via the macros above.
  */
-struct vfs_ops vfs_master_funcs = {
-    vfs_path,
+static struct vfs_ops vfs_master_funcs = {
     vfs_validupath,
     vfs_chown,
     vfs_renamedir,
@@ -864,7 +858,6 @@ struct vfs_ops vfs_master_funcs = {
  */
 
 static struct vfs_ops netatalk_adouble = {
-    /* vfs_path:          */ ad_path,
     /* vfs_validupath:    */ validupath_adouble,
     /* vfs_chown:         */ RF_chown_adouble,
     /* vfs_renamedir:     */ RF_renamedir_adouble,
@@ -874,12 +867,11 @@ static struct vfs_ops netatalk_adouble = {
     /* vfs_setdirunixmode:*/ RF_setdirunixmode_adouble,
     /* vfs_setdirowner:   */ RF_setdirowner_adouble,
     /* vfs_deletefile:    */ RF_deletefile_adouble,
-    /* vfs_renamefile:    */ RF_renamefile_adouble
-    /* NULL, ...          */
+    /* vfs_renamefile:    */ RF_renamefile_adouble,
+    NULL
 };
 
 static struct vfs_ops netatalk_adouble_osx = {
-    /* vfs_path:          */ ad_path_osx,
     /* vfs_validupath:    */ validupath_osx,
     /* vfs_chown:         */ RF_chown_adouble,
     /* vfs_renamedir:     */ RF_renamedir_osx,
@@ -889,12 +881,12 @@ static struct vfs_ops netatalk_adouble_osx = {
     /* vfs_setdirunixmode:*/ RF_setdirunixmode_osx,
     /* vfs_setdirowner:   */ RF_setdirowner_osx,
     /* vfs_deletefile:    */ RF_deletefile_adouble,
-    /* vfs_renamefile:    */ RF_renamefile_osx
+    /* vfs_renamefile:    */ RF_renamefile_osx,
+    NULL
 };
 
 /* samba sfm format. ad_path shouldn't be set her */
 static struct vfs_ops netatalk_adouble_sfm = {
-    /* vfs_path:          */ ad_path_sfm,
     /* vfs_validupath:    */ validupath_adouble,
     /* vfs_chown:         */ RF_chown_ads,
     /* vfs_renamedir:     */ RF_renamedir_adouble,
@@ -905,22 +897,22 @@ static struct vfs_ops netatalk_adouble_sfm = {
     /* vfs_setdirowner:   */ RF_setdirowner_ads,
     /* vfs_deletefile:    */ RF_deletefile_ads,
     /* vfs_renamefile:    */ RF_renamefile_ads,
+    NULL
 };
 
 /* 
  * Secondary vfs modules for Extended Attributes
  */
 
-struct vfs_ops netatalk_ea_adouble = {
-    /* vfs_path:          */ NULL,
+static struct vfs_ops netatalk_ea_adouble = {
     /* vfs_validupath:    */ NULL,
     /* vfs_chown:         */ ea_chown,
     /* vfs_renamedir:     */ NULL, /* ok */
     /* vfs_deletecurdir:  */ NULL, /* ok */
-    /* vfs_setfilmode:    */ NULL,
-    /* vfs_setdirmode:    */ NULL,
-    /* vfs_setdirunixmode:*/ NULL,
-    /* vfs_setdirowner:   */ NULL,
+    /* vfs_setfilmode:    */ ea_chmod_file,
+    /* vfs_setdirmode:    */ NULL, /* ok */
+    /* vfs_setdirunixmode:*/ ea_chmod_dir,
+    /* vfs_setdirowner:   */ NULL, /* ok */
     /* vfs_deletefile:    */ ea_deletefile,
     /* vfs_renamefile:    */ ea_renamefile,
     /* vfs_copyfile       */ ea_copyfile,
@@ -934,8 +926,7 @@ struct vfs_ops netatalk_ea_adouble = {
 };
 
 #ifdef HAVE_SOLARIS_EAS
-struct vfs_ops netatalk_ea_solaris = {
-    /* ad_path:           */ NULL,
+static struct vfs_ops netatalk_ea_solaris = {
     /* validupath:        */ NULL,
     /* rf_chown:          */ NULL,
     /* rf_renamedir:      */ NULL,
@@ -958,12 +949,11 @@ struct vfs_ops netatalk_ea_solaris = {
 #endif
 
 /* 
- * Tertiary attributes for ACLs
+ * Tertiary VFS modules for ACLs
  */
 
 #ifdef HAVE_NFSv4_ACLS
-struct vfs_ops netatalk_solaris_acl_adouble = {
-    /* ad_path:           */ NULL,
+static struct vfs_ops netatalk_solaris_acl_adouble = {
     /* validupath:        */ NULL,
     /* rf_chown:          */ NULL,
     /* rf_renamedir:      */ NULL,
@@ -976,7 +966,8 @@ struct vfs_ops netatalk_solaris_acl_adouble = {
     /* rf_renamefile:     */ NULL,
     /* vfs_copyfile       */ NULL,
     /* rf_acl:            */ RF_solaris_acl,
-    /* rf_remove_acl      */ RF_remove_acl
+    /* rf_remove_acl      */ RF_remove_acl,
+    NULL
 };
 #endif
 
@@ -987,13 +978,16 @@ void initvol_vfs(struct vol *vol)
 
     /* Default adouble stuff */
     if (vol->v_adouble == AD_VERSION2_OSX) {
-        vfs[0] = &netatalk_adouble_osx;
+        vol->vfs_modules[0] = &netatalk_adouble_osx;
+        vol->ad_path = ad_path_osx;
     }
     else if (vol->v_adouble == AD_VERSION1_SFM) {
-        vfs[0] = &netatalk_adouble_sfm;
+        vol->vfs_modules[0] = &netatalk_adouble_sfm;
+        vol->ad_path = ad_path_sfm;
     }
     else {
-        vfs[0] = &netatalk_adouble;
+        vol->vfs_modules[0] = &netatalk_adouble;
+        vol->ad_path = ad_path;
     }
 
     /* Extended Attributes */
@@ -1001,7 +995,7 @@ void initvol_vfs(struct vol *vol)
 
 #ifdef HAVE_SOLARIS_EAS
         LOG(log_debug, logtype_afpd, "initvol_vfs: Enabling EA support with Solaris native EAs.");
-        vfs[1] = &netatalk_ea_solaris;
+        vol->vfs_modules[1] = &netatalk_ea_solaris;
 #else
         LOG(log_error, logtype_afpd, "initvol_vfs: Can't enable Solaris EA support.");
         goto enable_adea;
@@ -1010,12 +1004,11 @@ void initvol_vfs(struct vol *vol)
     enable_adea:
         /* default: AFPVOL_EA_AD */
         LOG(log_debug, logtype_afpd, "initvol_vfs: Enabling EA support with adouble files.");
-        vfs[1] = &netatalk_ea_adouble;
+        vol->vfs_modules[1] = &netatalk_ea_adouble;
     }
 
     /* ACLs */
 #ifdef HAVE_NFSv4_ACLS
-    vfs[2] = &netatalk_solaris_acl_adouble;
+    vol->vfs_modules[2] = &netatalk_solaris_acl_adouble;
 #endif
 }
-