Fix all warnings in source3 with gcc4.3.
[samba.git] / source3 / modules / vfs_recycle.c
index 5372a7e64c15b64232f1fc663a133cf5fce0f3ca..cb985e1be22f45bdf982afac63bd3e010aa62948 100644 (file)
@@ -6,10 +6,11 @@
  * Copyright (C) 2002, Alexander Bokovoy - cascaded VFS adoption,
  * Copyright (C) 2002, Juergen Hasch - added some options.
  * Copyright (C) 2002, Simo Sorce
+ * Copyright (C) 2002, Stefan (metze) Metzmacher
  *
  * 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
- * the Free Software Foundation; either version 2 of the License, or
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
@@ -18,8 +19,7 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ * along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
 #include "includes.h"
@@ -30,180 +30,191 @@ static int vfs_recycle_debug_level = DBGC_VFS;
 
 #undef DBGC_CLASS
 #define DBGC_CLASS vfs_recycle_debug_level
-
-static const char *delimiter = "|";            /* delimiter for options */
-
-/* One per connection */
-
-typedef struct recycle_bin_struct
-{
-       TALLOC_CTX *ctx;
-       char    *repository;            /* name of the recycle bin directory */
-       BOOL    keep_dir_tree;          /* keep directory structure of deleted file in recycle bin */
-       BOOL    versions;               /* create versions of deleted files with identical name */
-       BOOL    touch;                  /* touch access date of deleted file */
-       char    *exclude;               /* which files to exclude */
-       char    *exclude_dir;           /* which directories to exclude */
-       char    *noversions;            /* which files to exclude from versioning */
-       SMB_OFF_T maxsize;              /* maximum file size to be saved */
-} recycle_bin_struct;
-
-/* VFS operations */
-static struct vfs_ops default_vfs_ops;   /* For passthrough operation */
-
-static int recycle_connect(struct connection_struct *conn, const char *service, const char *user);
-static void recycle_disconnect(struct connection_struct *conn);
-static int recycle_unlink(connection_struct *, const char *);
-
-#define VFS_OP(x) ((void *) x)
+static int recycle_connect(vfs_handle_struct *handle, const char *service, const char *user);
+static void recycle_disconnect(vfs_handle_struct *handle);
+static int recycle_unlink(vfs_handle_struct *handle, const char *name);
 
 static vfs_op_tuple recycle_ops[] = {
 
        /* Disk operations */
-       {VFS_OP(recycle_connect),       SMB_VFS_OP_CONNECT,     SMB_VFS_LAYER_TRANSPARENT},
-       {VFS_OP(recycle_disconnect),    SMB_VFS_OP_DISCONNECT,  SMB_VFS_LAYER_TRANSPARENT},
+       {SMB_VFS_OP(recycle_connect),   SMB_VFS_OP_CONNECT,     SMB_VFS_LAYER_TRANSPARENT},
+       {SMB_VFS_OP(recycle_disconnect),        SMB_VFS_OP_DISCONNECT,  SMB_VFS_LAYER_TRANSPARENT},
 
        /* File operations */
-       {VFS_OP(recycle_unlink),        SMB_VFS_OP_UNLINK,      SMB_VFS_LAYER_TRANSPARENT},
+       {SMB_VFS_OP(recycle_unlink),    SMB_VFS_OP_UNLINK,      SMB_VFS_LAYER_TRANSPARENT},
 
-       {NULL,                          SMB_VFS_OP_NOOP,        SMB_VFS_LAYER_NOOP}
+       {SMB_VFS_OP(NULL),              SMB_VFS_OP_NOOP,        SMB_VFS_LAYER_NOOP}
 };
 
-static BOOL check_bool_param(const char *value)
+static int recycle_connect(vfs_handle_struct *handle, const char *service, const char *user)
 {
-       if (strwicmp(value, "yes") == 0 ||
-           strwicmp(value, "true") == 0 ||
-           strwicmp(value, "1") == 0)
-               return True;
+       DEBUG(10,("recycle_connect() connect to service[%s] as user[%s].\n",
+               service,user));
 
-       return False;
+       return SMB_VFS_NEXT_CONNECT(handle, service, user);
 }
 
-/**
- * VFS initialisation function.
- *
- * @retval initialised vfs_op_tuple array
- **/
-static vfs_op_tuple *recycle_init(const struct vfs_ops *def_vfs_ops,
-                       struct smb_vfs_handle_struct *vfs_handle)
+static void recycle_disconnect(vfs_handle_struct *handle)
 {
-       DEBUG(10, ("Initializing VFS module recycle\n"));
-       memcpy(&default_vfs_ops, def_vfs_ops, sizeof(struct vfs_ops));
-       vfs_recycle_debug_level = debug_add_class("vfs_recycle_bin");
-       if (vfs_recycle_debug_level == -1) {
-               vfs_recycle_debug_level = DBGC_VFS;
-               DEBUG(0, ("vfs_recycle: Couldn't register custom debugging class!\n"));
-       } else {
-               DEBUG(0, ("vfs_recycle: Debug class number of 'vfs_recycle': %d\n", vfs_recycle_debug_level));
-       }
+       DEBUG(10,("recycle_disconnect() connect to service[%s].\n",
+               lp_servicename(SNUM(handle->conn))));
 
-       return recycle_ops;
+       SMB_VFS_NEXT_DISCONNECT(handle);
 }
 
-static int recycle_connect(struct connection_struct *conn, const char *service, const char *user)
+static const char *recycle_repository(vfs_handle_struct *handle)
 {
-       TALLOC_CTX *ctx = NULL;
-       recycle_bin_struct *recbin;
-       char *servicename;
-       char *tmp_str;
+       const char *tmp_str = NULL;
+       
 
-       DEBUG(10, ("Called for service %s (%d) as user %s\n", service, SNUM(conn), user));
+       tmp_str = lp_parm_const_string(SNUM(handle->conn), "recycle", "repository",".recycle");
 
-       if (!(ctx = talloc_init("recycle bin"))) {
-               DEBUG(0, ("Failed to allocate memory in VFS module recycle_bin\n"));
-               return 0;
-       }
+       DEBUG(10, ("recycle: repository = %s\n", tmp_str));
+       
+       return tmp_str;
+}
 
-       recbin = talloc(ctx,sizeof(recycle_bin_struct));
-       if ( recbin == NULL) {
-               DEBUG(0, ("Failed to allocate memory in VFS module recycle_bin\n"));
-               return -1;
-       }
-       recbin->ctx = ctx;
-
-       /* Set defaults */
-       recbin->repository = talloc_strdup(ctx, ".recycle");
-       ALLOC_CHECK(recbin->repository, error);
-       recbin->keep_dir_tree = False;
-       recbin->versions = False;
-       recbin->touch = False;
-       recbin->exclude = "";
-       recbin->exclude_dir = "";
-       recbin->noversions = "";
-       recbin->maxsize = 0;
-
-       /* parse configuration options */
-       servicename = talloc_strdup(recbin->ctx, lp_servicename(SNUM(conn)));
-       DEBUG(10, ("servicename = %s\n",servicename));
-       if ((tmp_str = lp_parm_string(servicename, "vfs_recycle_bin", "repository")) != NULL) {
-               recbin->repository = talloc_sub_conn(ctx, conn, tmp_str);
-               ALLOC_CHECK(recbin->repository, error);
-               trim_string(recbin->repository, "/", "/");
-               DEBUG(5, ("recycle.bin: repository = %s\n", recbin->repository));
-       }
-       if ((tmp_str = lp_parm_string(servicename, "vfs_recycle_bin", "keeptree")) != NULL) {
-               if (check_bool_param(tmp_str) == True)
-                       recbin->keep_dir_tree = True;
-               DEBUG(5, ("recycle.bin: keeptree = %s\n", tmp_str));
-       }
-       if ((tmp_str = lp_parm_string(servicename, "vfs_recycle_bin", "versions")) != NULL) {
-               if (check_bool_param(tmp_str) == True)
-                       recbin->versions = True;
-               DEBUG(5, ("recycle.bin: versions = %s\n", tmp_str));
-       }
-       if ((tmp_str = lp_parm_string(servicename, "vfs_recycle_bin", "touch")) != NULL) {
-               if (check_bool_param(tmp_str) == True)
-                       recbin->touch = True;
-               DEBUG(5, ("recycle.bin: touch = %s\n", tmp_str));
-       }
-       if ((tmp_str = lp_parm_string(servicename, "vfs_recycle_bin", "maxsize")) != NULL) {
-               recbin->maxsize = strtoul(tmp_str, NULL, 10);
-               if (recbin->maxsize == 0) {
-                       recbin->maxsize = -1;
-                       DEBUG(5, ("recycle.bin: maxsize = -infinite-\n"));
-               } else {
-                       DEBUG(5, ("recycle.bin: maxsize = %ld\n", (long int)recbin->maxsize));
-               }
-       }
-       if ((tmp_str = lp_parm_string(servicename, "vfs_recycle_bin", "exclude")) != NULL) {
-               recbin->exclude = talloc_strdup(ctx, tmp_str);
-               ALLOC_CHECK(recbin->exclude, error);
-               DEBUG(5, ("recycle.bin: exclude = %s\n", recbin->exclude));
-       }
-       if ((tmp_str = lp_parm_string(servicename,"vfs_recycle_bin", "exclude_dir")) != NULL) {
-               recbin->exclude_dir = talloc_strdup(ctx, tmp_str);
-               ALLOC_CHECK(recbin->exclude_dir, error);
-               DEBUG(5, ("recycle.bin: exclude_dir = %s\n", recbin->exclude_dir));
-       }
-       if ((tmp_str = lp_parm_string(servicename,"vfs_recycle_bin", "noversions")) != NULL) {
-               recbin->noversions = talloc_strdup(ctx, tmp_str);
-               ALLOC_CHECK(recbin->noversions, error);
-               DEBUG(5, ("recycle.bin: noversions = %s\n", recbin->noversions));
-       }
+static bool recycle_keep_dir_tree(vfs_handle_struct *handle)
+{
+       bool ret;
+       
+       ret = lp_parm_bool(SNUM(handle->conn), "recycle", "keeptree", False);
+
+       DEBUG(10, ("recycle_bin: keeptree = %s\n", ret?"True":"False"));
+       
+       return ret;
+}
+
+static bool recycle_versions(vfs_handle_struct *handle)
+{
+       bool ret;
+
+       ret = lp_parm_bool(SNUM(handle->conn), "recycle", "versions", False);
+
+       DEBUG(10, ("recycle: versions = %s\n", ret?"True":"False"));
+       
+       return ret;
+}
+
+static bool recycle_touch(vfs_handle_struct *handle)
+{
+       bool ret;
+
+       ret = lp_parm_bool(SNUM(handle->conn), "recycle", "touch", False);
+
+       DEBUG(10, ("recycle: touch = %s\n", ret?"True":"False"));
+       
+       return ret;
+}
+
+static bool recycle_touch_mtime(vfs_handle_struct *handle)
+{
+       bool ret;
+
+       ret = lp_parm_bool(SNUM(handle->conn), "recycle", "touch_mtime", False);
+
+       DEBUG(10, ("recycle: touch_mtime = %s\n", ret?"True":"False"));
+       
+       return ret;
+}
+
+static const char **recycle_exclude(vfs_handle_struct *handle)
+{
+       const char **tmp_lp;
+       
+       tmp_lp = lp_parm_string_list(SNUM(handle->conn), "recycle", "exclude", NULL);
+
+       DEBUG(10, ("recycle: exclude = %s ...\n", tmp_lp?*tmp_lp:""));
+       
+       return tmp_lp;
+}
+
+static const char **recycle_exclude_dir(vfs_handle_struct *handle)
+{
+       const char **tmp_lp;
+       
+       tmp_lp = lp_parm_string_list(SNUM(handle->conn), "recycle", "exclude_dir", NULL);
+
+       DEBUG(10, ("recycle: exclude_dir = %s ...\n", tmp_lp?*tmp_lp:""));
+       
+       return tmp_lp;
+}
+
+static const char **recycle_noversions(vfs_handle_struct *handle)
+{
+       const char **tmp_lp;
+       
+       tmp_lp = lp_parm_string_list(SNUM(handle->conn), "recycle", "noversions", NULL);
+
+       DEBUG(10, ("recycle: noversions = %s\n", tmp_lp?*tmp_lp:""));
+       
+       return tmp_lp;
+}
+
+static SMB_OFF_T recycle_maxsize(vfs_handle_struct *handle)
+{
+       SMB_OFF_T maxsize;
+       
+       maxsize = conv_str_size(lp_parm_const_string(SNUM(handle->conn),
+                                           "recycle", "maxsize", NULL));
+
+       DEBUG(10, ("recycle: maxsize = %lu\n", (long unsigned int)maxsize));
+       
+       return maxsize;
+}
+
+static SMB_OFF_T recycle_minsize(vfs_handle_struct *handle)
+{
+       SMB_OFF_T minsize;
+       
+       minsize = conv_str_size(lp_parm_const_string(SNUM(handle->conn),
+                                           "recycle", "minsize", NULL));
+
+       DEBUG(10, ("recycle: minsize = %lu\n", (long unsigned int)minsize));
+       
+       return minsize;
+}
+
+static mode_t recycle_directory_mode(vfs_handle_struct *handle)
+{
+       int dirmode;
+       const char *buff;
 
-       conn->vfs_private = (void *)recbin;
-       return default_vfs_ops.connect(conn, service, user);
+       buff = lp_parm_const_string(SNUM(handle->conn), "recycle", "directory_mode", NULL);
 
-error:
-       talloc_destroy(ctx);
-       return -1;
+       if (buff != NULL ) {
+               sscanf(buff, "%o", &dirmode);
+       } else {
+               dirmode=S_IRUSR | S_IWUSR | S_IXUSR;
+       }
+
+       DEBUG(10, ("recycle: directory_mode = %o\n", dirmode));
+       return (mode_t)dirmode;
 }
 
-static void recycle_disconnect(struct connection_struct *conn)
+static mode_t recycle_subdir_mode(vfs_handle_struct *handle)
 {
-       DEBUG(10, ("Disconnecting VFS module recycle bin\n"));
-       if (conn->vfs_private) {
-               talloc_destroy(((recycle_bin_struct *)conn->vfs_private)->ctx);
-               conn->vfs_private = NULL;
+       int dirmode;
+       const char *buff;
+
+       buff = lp_parm_const_string(SNUM(handle->conn), "recycle", "subdir_mode", NULL);
+
+       if (buff != NULL ) {
+               sscanf(buff, "%o", &dirmode);
+       } else {
+               dirmode=recycle_directory_mode(handle);
        }
-       default_vfs_ops.disconnect(conn);
+
+       DEBUG(10, ("recycle: subdir_mode = %o\n", dirmode));
+       return (mode_t)dirmode;
 }
 
-static BOOL recycle_directory_exist(connection_struct *conn, const char *dname)
+static bool recycle_directory_exist(vfs_handle_struct *handle, const char *dname)
 {
        SMB_STRUCT_STAT st;
 
-       if (default_vfs_ops.stat(conn, dname, &st) == 0) {
+       if (SMB_VFS_NEXT_STAT(handle, dname, &st) == 0) {
                if (S_ISDIR(st.st_mode)) {
                        return True;
                }
@@ -212,11 +223,11 @@ static BOOL recycle_directory_exist(connection_struct *conn, const char *dname)
        return False;
 }
 
-static BOOL recycle_file_exist(connection_struct *conn, const char *fname)
+static bool recycle_file_exist(vfs_handle_struct *handle, const char *fname)
 {
        SMB_STRUCT_STAT st;
 
-       if (default_vfs_ops.stat(conn, fname, &st) == 0) {
+       if (SMB_VFS_NEXT_STAT(handle, fname, &st) == 0) {
                if (S_ISREG(st.st_mode)) {
                        return True;
                }
@@ -231,13 +242,15 @@ static BOOL recycle_file_exist(connection_struct *conn, const char *fname)
  * @param fname file name
  * @return size in bytes
  **/
-static SMB_OFF_T recycle_get_file_size(connection_struct *conn, const char *fname)
+static SMB_OFF_T recycle_get_file_size(vfs_handle_struct *handle, const char *fname)
 {
        SMB_STRUCT_STAT st;
-       if (default_vfs_ops.stat(conn, fname, &st) != 0) {
-               DEBUG(0,("recycle.bin: stat for %s returned %s\n", fname, strerror(errno)));
+
+       if (SMB_VFS_NEXT_STAT(handle, fname, &st) != 0) {
+               DEBUG(0,("recycle: stat for %s returned %s\n", fname, strerror(errno)));
                return (SMB_OFF_T)0;
        }
+
        return(st.st_size);
 }
 
@@ -247,42 +260,49 @@ static SMB_OFF_T recycle_get_file_size(connection_struct *conn, const char *fnam
  * @param dname Directory tree to be created
  * @return Returns True for success
  **/
-static BOOL recycle_create_dir(connection_struct *conn, const char *dname)
+static bool recycle_create_dir(vfs_handle_struct *handle, const char *dname)
 {
-       int len;
+       size_t len;
        mode_t mode;
        char *new_dir = NULL;
        char *tmp_str = NULL;
        char *token;
        char *tok_str;
-       BOOL ret = False;
+       bool ret = False;
+       char *saveptr;
 
-       mode = S_IREAD | S_IWRITE | S_IEXEC;
+       mode = recycle_directory_mode(handle);
 
-       tmp_str = strdup(dname);
+       tmp_str = SMB_STRDUP(dname);
        ALLOC_CHECK(tmp_str, done);
        tok_str = tmp_str;
 
-       len = strlen(dname);
-       new_dir = (char *)malloc(len + 1);
+       len = strlen(dname)+1;
+       new_dir = (char *)SMB_MALLOC(len + 1);
        ALLOC_CHECK(new_dir, done);
        *new_dir = '\0';
+       if (dname[0] == '/') {
+               /* Absolute path. */
+               safe_strcat(new_dir,"/",len);
+       }
 
        /* Create directory tree if neccessary */
-       for(token = strtok(tok_str, "/"); token; token = strtok(NULL, "/")) {
+       for(token = strtok_r(tok_str, "/", &saveptr); token;
+           token = strtok_r(NULL, "/", &saveptr)) {
                safe_strcat(new_dir, token, len);
-               if (recycle_directory_exist(conn, new_dir))
-                       DEBUG(10, ("recycle.bin: dir %s already exists\n", new_dir));
+               if (recycle_directory_exist(handle, new_dir))
+                       DEBUG(10, ("recycle: dir %s already exists\n", new_dir));
                else {
-                       DEBUG(5, ("recycle.bin: creating new dir %s\n", new_dir));
-                       if (default_vfs_ops.mkdir(conn, new_dir, mode) != 0) {
-                               DEBUG(1,("recycle.bin: mkdir failed for %s with error: %s\n", new_dir, strerror(errno)));
+                       DEBUG(5, ("recycle: creating new dir %s\n", new_dir));
+                       if (SMB_VFS_NEXT_MKDIR(handle, new_dir, mode) != 0) {
+                               DEBUG(1,("recycle: mkdir failed for %s with error: %s\n", new_dir, strerror(errno)));
                                ret = False;
                                goto done;
                        }
                }
                safe_strcat(new_dir, "/", len);
-               }
+               mode = recycle_subdir_mode(handle);
+       }
 
        ret = True;
 done:
@@ -292,35 +312,52 @@ done:
 }
 
 /**
- * Check if needle is contained exactly in haystack
- * @param haystack list of parameters separated by delimimiter character
- * @param needle string to be matched exactly to haystack
- * @return True if found
+ * Check if any of the components of "exclude_list" are contained in path.
+ * Return True if found
  **/
-static BOOL checkparam(const char *haystack, const char *needle)
+
+static bool matchdirparam(const char **dir_exclude_list, char *path)
 {
-       char *token;
-       char *tok_str;
-       char *tmp_str;
-       BOOL ret = False;
+       char *startp = NULL, *endp = NULL;
 
-       if (haystack == NULL || strlen(haystack) == 0 || needle == NULL || strlen(needle) == 0) {
+       if (dir_exclude_list == NULL || dir_exclude_list[0] == NULL ||
+               *dir_exclude_list[0] == '\0' || path == NULL || *path == '\0') {
                return False;
        }
 
-       tmp_str = strdup(haystack);
-       ALLOC_CHECK(tmp_str, done);
-       token = tok_str = tmp_str;
+       /* 
+        * Walk the components of path, looking for matches with the
+        * exclude list on each component. 
+        */
 
-       for(token = strtok(tok_str, delimiter); token; token = strtok(NULL, delimiter)) {
-               if(strcmp(token, needle) == 0) {
-                       ret = True;
-                       goto done;
+       for (startp = path; startp; startp = endp) {
+               int i;
+
+               while (*startp == '/') {
+                       startp++;
+               }
+               endp = strchr(startp, '/');
+               if (endp) {
+                       *endp = '\0';
+               }
+
+               for(i=0; dir_exclude_list[i] ; i++) {
+                       if(unix_wild_match(dir_exclude_list[i], startp)) {
+                               /* Repair path. */
+                               if (endp) {
+                                       *endp = '/';
+                               }
+                               return True;
+                       }
+               }
+
+               /* Repair path. */
+               if (endp) {
+                       *endp = '/';
                }
        }
-done:
-       SAFE_FREE(tmp_str);
-       return ret;
+
+       return False;
 }
 
 /**
@@ -329,123 +366,130 @@ done:
  * @param needle string to be matched exectly to haystack including pattern matching
  * @return True if found
  **/
-static BOOL matchparam(const char *haystack, const char *needle)
+static bool matchparam(const char **haystack_list, const char *needle)
 {
-       char *token;
-       char *tok_str;
-       char *tmp_str;
-       BOOL ret = False;
+       int i;
 
-       if (haystack == NULL || strlen(haystack) == 0 || needle == NULL || strlen(needle) == 0) {
+       if (haystack_list == NULL || haystack_list[0] == NULL ||
+               *haystack_list[0] == '\0' || needle == NULL || *needle == '\0') {
                return False;
        }
 
-       tmp_str = strdup(haystack);
-       ALLOC_CHECK(tmp_str, done);
-       token = tok_str = tmp_str;
-
-       for(token = strtok(tok_str, delimiter); token; token = strtok(NULL, delimiter)) {
-               if (!unix_wild_match(token, needle)) {
-                       ret = True;
-                       goto done;
+       for(i=0; haystack_list[i] ; i++) {
+               if(unix_wild_match(haystack_list[i], needle)) {
+                       return True;
                }
        }
-done:
-       SAFE_FREE(tmp_str);
-       return ret;
+
+       return False;
 }
 
 /**
- * Touch access date
+ * Touch access or modify date
  **/
-static void recycle_touch(connection_struct *conn, const char *fname)
+static void recycle_do_touch(vfs_handle_struct *handle, const char *fname,
+                            bool touch_mtime)
 {
        SMB_STRUCT_STAT st;
-       struct utimbuf tb;
-       time_t currtime;
+       struct timespec ts[2];
+       int ret, err;
 
-       if (default_vfs_ops.stat(conn, fname, &st) != 0) {
-               DEBUG(0,("recycle.bin: stat for %s returned %s\n", fname, strerror(errno)));
+       if (SMB_VFS_NEXT_STAT(handle, fname, &st) != 0) {
+               DEBUG(0,("recycle: stat for %s returned %s\n",
+                        fname, strerror(errno)));
                return;
        }
-       currtime = time(&currtime);
-       tb.actime = currtime;
-       tb.modtime = st.st_mtime;
+       ts[0] = timespec_current(); /* atime */
+       ts[1] = touch_mtime ? ts[0] : get_mtimespec(&st); /* mtime */
 
-       if (default_vfs_ops.utime(conn, fname, &tb) == -1 )
-               DEBUG(0, ("recycle.bin: touching %s failed, reason = %s\n", fname, strerror(errno)));
+       become_root();
+       ret = SMB_VFS_NEXT_NTIMES(handle, fname, ts);
+       err = errno;
+       unbecome_root();
+       if (ret == -1 ) {
+               DEBUG(0, ("recycle: touching %s failed, reason = %s\n",
+                         fname, strerror(err)));
        }
+}
 
 /**
  * Check if file should be recycled
  **/
-static int recycle_unlink(connection_struct *conn, const char *inname)
+static int recycle_unlink(vfs_handle_struct *handle, const char *file_name)
 {
-       recycle_bin_struct *recbin;
-       char *file_name = NULL;
+       connection_struct *conn = handle->conn;
        char *path_name = NULL;
                char *temp_name = NULL;
        char *final_name = NULL;
-       char *base;
-       int i;
-       SMB_BIG_UINT dfree, dsize, bsize;
-       SMB_OFF_T file_size, space_avail;
-       BOOL exist;
+       const char *base;
+       char *repository = NULL;
+       int i = 1;
+       SMB_OFF_T maxsize, minsize;
+       SMB_OFF_T file_size; /* space_avail;    */
+       bool exist;
        int rc = -1;
 
-       file_name = strdup(inname);
-       ALLOC_CHECK(file_name, done);
-
-       if (conn->vfs_private)
-               recbin = (recycle_bin_struct *)conn->vfs_private;
-       else {
-               DEBUG(0, ("Recycle bin not initialized!\n"));
-               rc = default_vfs_ops.unlink(conn, file_name);
-               goto done;
-       }
-
-       if(!recbin->repository || *(recbin->repository) == '\0') {
-               DEBUG(3, ("Recycle path not set, purging %s...\n", file_name));
-               rc = default_vfs_ops.unlink(conn, file_name);
+       repository = talloc_sub_advanced(NULL, lp_servicename(SNUM(conn)),
+                                       conn->server_info->unix_name,
+                                       conn->connectpath,
+                                       conn->server_info->utok.gid,
+                                       conn->server_info->sanitized_username,
+                                       pdb_get_domain(conn->server_info->sam_account),
+                                       recycle_repository(handle));
+       ALLOC_CHECK(repository, done);
+       /* shouldn't we allow absolute path names here? --metze */
+       /* Yes :-). JRA. */
+       trim_char(repository, '\0', '/');
+       
+       if(!repository || *(repository) == '\0') {
+               DEBUG(3, ("recycle: repository path not set, purging %s...\n", file_name));
+               rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
                goto done;
        }
 
        /* we don't recycle the recycle bin... */
-       if (strncmp(file_name, recbin->repository, strlen(recbin->repository)) == 0) {
-               DEBUG(3, ("File is within recycling bin, unlinking ...\n"));
-               rc = default_vfs_ops.unlink(conn, file_name);
+       if (strncmp(file_name, repository, strlen(repository)) == 0) {
+               DEBUG(3, ("recycle: File is within recycling bin, unlinking ...\n"));
+               rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
                goto done;
        }
 
-       file_size = recycle_get_file_size(conn, file_name);
+       file_size = recycle_get_file_size(handle, file_name);
        /* it is wrong to purge filenames only because they are empty imho
         *   --- simo
         *
        if(fsize == 0) {
-               DEBUG(3, ("File %s is empty, purging...\n", file_name));
-               rc = default_vfs_ops.unlink(conn,file_name);
+               DEBUG(3, ("recycle: File %s is empty, purging...\n", file_name));
+               rc = SMB_VFS_NEXT_UNLINK(handle,file_name);
                goto done;
        }
         */
 
-       /* FIXME: this is wrong, we should check the hole size of the recycle bin is
+       /* FIXME: this is wrong, we should check the whole size of the recycle bin is
         * not greater then maxsize, not the size of the single file, also it is better
         * to remove older files
         */
-       if(recbin->maxsize > 0 && file_size > recbin->maxsize) {
-               DEBUG(3, ("File %s exceeds maximum recycle size, purging... \n", file_name));
-               rc = default_vfs_ops.unlink(conn, file_name);
+       maxsize = recycle_maxsize(handle);
+       if(maxsize > 0 && file_size > maxsize) {
+               DEBUG(3, ("recycle: File %s exceeds maximum recycle size, purging... \n", file_name));
+               rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
+               goto done;
+       }
+       minsize = recycle_minsize(handle);
+       if(minsize > 0 && file_size < minsize) {
+               DEBUG(3, ("recycle: File %s lowers minimum recycle size, purging... \n", file_name));
+               rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
                goto done;
        }
 
        /* FIXME: this is wrong: moving files with rename does not change the disk space
         * allocation
         *
-       space_avail = default_vfs_ops.disk_free(conn, ".", True, &bsize, &dfree, &dsize) * 1024L;
+       space_avail = SMB_VFS_NEXT_DISK_FREE(handle, ".", True, &bsize, &dfree, &dsize) * 1024L;
        DEBUG(5, ("space_avail = %Lu, file_size = %Lu\n", space_avail, file_size));
        if(space_avail < file_size) {
-               DEBUG(3, ("Not enough diskspace, purging file %s\n", file_name));
-               rc = default_vfs_ops.unlink(conn, file_name);
+               DEBUG(3, ("recycle: Not enough diskspace, purging file %s\n", file_name));
+               rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
                goto done;
        }
         */
@@ -454,98 +498,112 @@ static int recycle_unlink(connection_struct *conn, const char *inname)
        base = strrchr(file_name, '/');
        if (base == NULL) {
                base = file_name;
-               path_name = strdup("/");
+               path_name = SMB_STRDUP("/");
                ALLOC_CHECK(path_name, done);
        }
        else {
-               path_name = strdup(file_name);
+               path_name = SMB_STRDUP(file_name);
                ALLOC_CHECK(path_name, done);
                path_name[base - file_name] = '\0';
                base++;
        }
 
-       DEBUG(10, ("recycle.bin: fname = %s\n", file_name));    /* original filename with path */
-       DEBUG(10, ("recycle.bin: fpath = %s\n", path_name));    /* original path */
-       DEBUG(10, ("recycle.bin: base = %s\n", base));          /* filename without path */
+       DEBUG(10, ("recycle: fname = %s\n", file_name));        /* original filename with path */
+       DEBUG(10, ("recycle: fpath = %s\n", path_name));        /* original path */
+       DEBUG(10, ("recycle: base = %s\n", base));              /* filename without path */
 
-       if (matchparam(recbin->exclude, base)) {
-               DEBUG(3, ("recycle.bin: file %s is excluded \n", base));
-               rc = default_vfs_ops.unlink(conn, file_name);
+       if (matchparam(recycle_exclude(handle), base)) {
+               DEBUG(3, ("recycle: file %s is excluded \n", base));
+               rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
                goto done;
        }
 
-       /* FIXME: this check will fail if we have more than one level of directories,
-        * we shoud check for every level 1, 1/2, 1/2/3, 1/2/3/4 .... 
-        *      ---simo
-        */
-       if (checkparam(recbin->exclude_dir, path_name)) {
-               DEBUG(3, ("recycle.bin: directory %s is excluded \n", path_name));
-               rc = default_vfs_ops.unlink(conn, file_name);
+       if (matchdirparam(recycle_exclude_dir(handle), path_name)) {
+               DEBUG(3, ("recycle: directory %s is excluded \n", path_name));
+               rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
                goto done;
        }
 
-       /* see if we need to recreate the original directory structure in the recycle bin */
-       if (recbin->keep_dir_tree == True) {
-               asprintf(&temp_name, "%s/%s", recbin->repository, path_name);
+       if (recycle_keep_dir_tree(handle) == True) {
+               if (asprintf(&temp_name, "%s/%s", repository, path_name) == -1) {
+                       ALLOC_CHECK(temp_name, done);
+               }
        } else {
-               temp_name = strdup(recbin->repository);
+               temp_name = SMB_STRDUP(repository);
        }
        ALLOC_CHECK(temp_name, done);
 
-       exist = recycle_directory_exist(conn, temp_name);
+       exist = recycle_directory_exist(handle, temp_name);
        if (exist) {
-               DEBUG(10, ("recycle.bin: Directory already exists\n"));
+               DEBUG(10, ("recycle: Directory already exists\n"));
        } else {
-               DEBUG(10, ("recycle.bin: Creating directory %s\n", temp_name));
-               if (recycle_create_dir(conn, temp_name) == False) {
-                       DEBUG(3, ("Could not create directory, purging %s...\n", file_name));
-                       rc = default_vfs_ops.unlink(conn, file_name);
+               DEBUG(10, ("recycle: Creating directory %s\n", temp_name));
+               if (recycle_create_dir(handle, temp_name) == False) {
+                       DEBUG(3, ("recycle: Could not create directory, purging %s...\n", file_name));
+                       rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
                        goto done;
                }
        }
 
-       final_name = (char *)malloc(PATH_MAX);
-       ALLOC_CHECK(final_name, done);
-       snprintf(final_name, PATH_MAX, "%s/%s", temp_name, base);
-       DEBUG(10, ("recycle.bin: recycled file name%s\n", temp_name));          /* new filename with path */
+       if (asprintf(&final_name, "%s/%s", temp_name, base) == -1) {
+               ALLOC_CHECK(final_name, done);
+       }
+       DEBUG(10, ("recycle: recycled file name: %s\n", final_name));           /* new filename with path */
 
        /* check if we should delete file from recycle bin */
-       if (recycle_file_exist(conn, final_name)) {
-               if (recbin->versions == False || matchparam(recbin->noversions, base) == True) {
-                       DEBUG(3, ("recycle.bin: Removing old file %s from recycle bin\n", final_name));
-                       if (default_vfs_ops.unlink(conn, final_name) != 0) {
-                               DEBUG(1, ("recycle.bin: Error deleting old file: %s\n", strerror(errno)));
+       if (recycle_file_exist(handle, final_name)) {
+               if (recycle_versions(handle) == False || matchparam(recycle_noversions(handle), base) == True) {
+                       DEBUG(3, ("recycle: Removing old file %s from recycle bin\n", final_name));
+                       if (SMB_VFS_NEXT_UNLINK(handle, final_name) != 0) {
+                               DEBUG(1, ("recycle: Error deleting old file: %s\n", strerror(errno)));
                        }
                }
        }
 
        /* rename file we move to recycle bin */
        i = 1;
-       while (recycle_file_exist(conn, final_name)) {
-               snprintf(final_name, PATH_MAX, "%s/Copy #%d of %s", temp_name, i++, base);
+       while (recycle_file_exist(handle, final_name)) {
+               SAFE_FREE(final_name);
+               if (asprintf(&final_name, "%s/Copy #%d of %s", temp_name, i++, base) == -1) {
+                       ALLOC_CHECK(final_name, done);
+               }
        }
 
-       DEBUG(10, ("recycle.bin: Moving %s to %s\n", file_name, final_name));
-       rc = default_vfs_ops.rename(conn, file_name, final_name);
+       DEBUG(10, ("recycle: Moving %s to %s\n", file_name, final_name));
+       rc = SMB_VFS_NEXT_RENAME(handle, file_name, final_name);
        if (rc != 0) {
-               DEBUG(3, ("recycle.bin: Move error %d (%s), purging file %s (%s)\n", errno, strerror(errno), file_name, final_name));
-               rc = default_vfs_ops.unlink(conn, file_name);
+               DEBUG(3, ("recycle: Move error %d (%s), purging file %s (%s)\n", errno, strerror(errno), file_name, final_name));
+               rc = SMB_VFS_NEXT_UNLINK(handle, file_name);
                goto done;
        }
 
        /* touch access date of moved file */
-       if (recbin->touch == True )
-               recycle_touch(conn, final_name);
+       if (recycle_touch(handle) == True || recycle_touch_mtime(handle))
+               recycle_do_touch(handle, final_name, recycle_touch_mtime(handle));
 
 done:
-       SAFE_FREE(file_name);
        SAFE_FREE(path_name);
        SAFE_FREE(temp_name);
        SAFE_FREE(final_name);
+       TALLOC_FREE(repository);
        return rc;
 }
 
-int vfs_recycle_init(void)
-{  
-   return smb_register_vfs("recycle", recycle_init, SMB_VFS_INTERFACE_VERSION);
+NTSTATUS vfs_recycle_init(void);
+NTSTATUS vfs_recycle_init(void)
+{
+       NTSTATUS ret = smb_register_vfs(SMB_VFS_INTERFACE_VERSION, "recycle", recycle_ops);
+
+       if (!NT_STATUS_IS_OK(ret))
+               return ret;
+       
+       vfs_recycle_debug_level = debug_add_class("recycle");
+       if (vfs_recycle_debug_level == -1) {
+               vfs_recycle_debug_level = DBGC_VFS;
+               DEBUG(0, ("vfs_recycle: Couldn't register custom debugging class!\n"));
+       } else {
+               DEBUG(10, ("vfs_recycle: Debug class number of 'recycle': %d\n", vfs_recycle_debug_level));
+       }
+       
+       return ret;
 }