s3-rpc_client: support AES encryption in netr_ServerPasswordSet2 client.
[kai/samba.git] / source3 / modules / vfs_hpuxacl.c
index 9b9fe61d8308f2b4aa3490bf786199e1af008e82..a05ca44f2484bc90b81a6cde127ee180b33aa857 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Unix SMB/Netbios implementation.
  * VFS module to get and set HP-UX ACLs
- * Copyright (C) Michael Adam 2006
+ * Copyright (C) Michael Adam 2006,2008
  *
  * 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
@@ -14,8 +14,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"
+#include "system/filesys.h"
+#include "smbd/smbd.h"
+#include "modules/vfs_hpuxacl.h"
+
 
 /* 
  * including standard header <sys/aclv.h> 
@@ -105,44 +108,45 @@ struct hpux_acl_types {
 /* prototypes for private functions */
 
 static HPUX_ACL_T hpux_acl_init(int count);
-static BOOL smb_acl_to_hpux_acl(SMB_ACL_T smb_acl, 
+static bool smb_acl_to_hpux_acl(SMB_ACL_T smb_acl, 
                HPUX_ACL_T *solariacl, int *count, 
                SMB_ACL_TYPE_T type);
 static SMB_ACL_T hpux_acl_to_smb_acl(HPUX_ACL_T hpuxacl, int count,
-               SMB_ACL_TYPE_T type);
+                                    SMB_ACL_TYPE_T type, TALLOC_CTX *mem_ctx);
 static HPUX_ACL_TAG_T smb_tag_to_hpux_tag(SMB_ACL_TAG_T smb_tag);
 static SMB_ACL_TAG_T hpux_tag_to_smb_tag(HPUX_ACL_TAG_T hpux_tag);
-static BOOL hpux_add_to_acl(HPUX_ACL_T *hpux_acl, int *count,
+static bool hpux_add_to_acl(HPUX_ACL_T *hpux_acl, int *count,
                HPUX_ACL_T add_acl, int add_count, SMB_ACL_TYPE_T type);
-static BOOL hpux_acl_get_file(const char *name, HPUX_ACL_T *hpuxacl, 
+static bool hpux_acl_get_file(const char *name, HPUX_ACL_T *hpuxacl, 
                int *count);
 static SMB_ACL_PERM_T hpux_perm_to_smb_perm(const HPUX_PERM_T perm);
 static HPUX_PERM_T smb_perm_to_hpux_perm(const SMB_ACL_PERM_T perm);
 #if 0
-static BOOL hpux_acl_check(HPUX_ACL_T hpux_acl, int count);
+static bool hpux_acl_check(HPUX_ACL_T hpux_acl, int count);
 #endif
 /* aclsort (internal) and helpers: */
-static BOOL hpux_acl_sort(HPUX_ACL_T acl, int count);
+static bool hpux_acl_sort(HPUX_ACL_T acl, int count);
 static int hpux_internal_aclsort(int acl_count, int calclass, HPUX_ACL_T aclp);
 static void hpux_count_obj(int acl_count, HPUX_ACL_T aclp, 
                struct hpux_acl_types *acl_type_count);
 static void hpux_swap_acl_entries(HPUX_ACE_T *aclp0, HPUX_ACE_T *aclp1);
-static BOOL hpux_prohibited_duplicate_type(int acl_type);
+static bool hpux_prohibited_duplicate_type(int acl_type);
 
-static BOOL hpux_acl_call_present(void);
-static BOOL hpux_aclsort_call_present(void);
+static bool hpux_acl_call_present(void);
+static bool hpux_aclsort_call_present(void);
 
 
 /* public functions - the api */
 
 SMB_ACL_T hpuxacl_sys_acl_get_file(vfs_handle_struct *handle,
                                      const char *path_p,
-                                     SMB_ACL_TYPE_T type)
+                                  SMB_ACL_TYPE_T type,
+                                  TALLOC_CTX *mem_ctx)
 {
        SMB_ACL_T result = NULL;
        int count;
-       HPUX_ACL_T hpux_acl;
-       
+       HPUX_ACL_T hpux_acl = NULL;
+
        DEBUG(10, ("hpuxacl_sys_acl_get_file called for file '%s'.\n", 
                   path_p));
 
@@ -165,12 +169,12 @@ SMB_ACL_T hpuxacl_sys_acl_get_file(vfs_handle_struct *handle,
        if (!hpux_acl_get_file(path_p, &hpux_acl, &count)) {
                goto done;
        }
-       result = hpux_acl_to_smb_acl(hpux_acl, count, type);
+       result = hpux_acl_to_smb_acl(hpux_acl, count, type, mem_ctx);
        if (result == NULL) {
                DEBUG(10, ("conversion hpux_acl -> smb_acl failed (%s).\n",
                           strerror(errno)));
        }
-       
+
  done:
        DEBUG(10, ("hpuxacl_sys_acl_get_file %s.\n",
                   ((result == NULL) ? "failed" : "succeeded" )));
@@ -184,18 +188,11 @@ SMB_ACL_T hpuxacl_sys_acl_get_file(vfs_handle_struct *handle,
  */
 SMB_ACL_T hpuxacl_sys_acl_get_fd(vfs_handle_struct *handle,
                                 files_struct *fsp,
-                                int fd)
+                                TALLOC_CTX *mem_ctx)
 {
         /* 
         * HPUX doesn't have the facl call. Fake it using the path.... JRA. 
         */
-       /* For all I see, the info should already be in the fsp
-        * parameter, but get it again to be safe --- necessary? */
-        files_struct *file_struct_p = file_find_fd(fd);
-        if (file_struct_p == NULL) {
-                errno = EBADF;
-                return NULL;
-        }
         /*
          * We know we're in the same conn context. So we
          * can use the relative path.
@@ -203,8 +200,10 @@ SMB_ACL_T hpuxacl_sys_acl_get_fd(vfs_handle_struct *handle,
        DEBUG(10, ("redirecting call of hpuxacl_sys_acl_get_fd to "
                "hpuxacl_sys_acl_get_file (no facl syscall on HPUX).\n"));
 
-        return hpuxacl_sys_acl_get_file(handle, file_struct_p->fsp_name, 
-                       SMB_ACL_TYPE_ACCESS);
+        return hpuxacl_sys_acl_get_file(handle,
+                                       fsp->fsp_name->base_name,
+                                       SMB_ACL_TYPE_ACCESS,
+                                       mem_ctx);
 }
 
 
@@ -214,13 +213,19 @@ int hpuxacl_sys_acl_set_file(vfs_handle_struct *handle,
                             SMB_ACL_T theacl)
 {
        int ret = -1;
-       SMB_STRUCT_STAT s;
-       HPUX_ACL_T hpux_acl;
+       HPUX_ACL_T hpux_acl = NULL;
        int count;
-       
+       struct smb_filename *smb_fname = NULL;
+       NTSTATUS status;
+
        DEBUG(10, ("hpuxacl_sys_acl_set_file called for file '%s'\n",
                   name));
 
+       status = create_synthetic_smb_fname(talloc_tos(), name, NULL, NULL,
+                                           &smb_fname);
+       if (!NT_STATUS_IS_OK(status)) {
+               goto done;
+       }
 
        if(hpux_acl_call_present() == False) {
                /* Looks like we don't have the acl() system call on HPUX. 
@@ -250,11 +255,16 @@ int hpuxacl_sys_acl_set_file(vfs_handle_struct *handle,
         * that has _not_ been specified in "type" from the file first 
         * and concatenate it with the acl provided.
         */
-       if (SMB_VFS_STAT(handle->conn, name, &s) != 0) {
+       if (lp_posix_pathnames()) {
+               ret = SMB_VFS_LSTAT(handle->conn, smb_fname);
+       } else {
+               ret = SMB_VFS_STAT(handle->conn, smb_fname);
+       }
+       if (ret != 0) {
                DEBUG(10, ("Error in stat call: %s\n", strerror(errno)));
                goto done;
        }
-       if (S_ISDIR(s.st_mode)) {
+       if (S_ISDIR(smb_fname->st.st_ex_mode)) {
                HPUX_ACL_T other_acl; 
                int other_count;
                SMB_ACL_TYPE_T other_type;
@@ -263,7 +273,8 @@ int hpuxacl_sys_acl_set_file(vfs_handle_struct *handle,
                        ? SMB_ACL_TYPE_DEFAULT
                        : SMB_ACL_TYPE_ACCESS;
                DEBUGADD(10, ("getting acl from filesystem\n"));
-               if (!hpux_acl_get_file(name, &other_acl, &other_count)) {
+               if (!hpux_acl_get_file(smb_fname->base_name, &other_acl,
+                                      &other_count)) {
                        DEBUG(10, ("error getting acl from directory\n"));
                        goto done;
                }
@@ -291,14 +302,16 @@ int hpuxacl_sys_acl_set_file(vfs_handle_struct *handle,
        }
        DEBUG(10, ("resulting acl is valid.\n"));
 
-       ret = acl(CONST_DISCARD(char *, name), ACL_SET, count, hpux_acl);
+       ret = acl(discard_const_p(char, smb_fname->base_name), ACL_SET, count,
+                 hpux_acl);
        if (ret != 0) {
                DEBUG(0, ("ERROR calling acl: %s\n", strerror(errno)));
        }
-       
+
  done:
        DEBUG(10, ("hpuxacl_sys_acl_set_file %s.\n",
                   ((ret != 0) ? "failed" : "succeeded")));
+       TALLOC_FREE(smb_fname);
        SAFE_FREE(hpux_acl);
        return ret;
 }
@@ -308,18 +321,11 @@ int hpuxacl_sys_acl_set_file(vfs_handle_struct *handle,
  */
 int hpuxacl_sys_acl_set_fd(vfs_handle_struct *handle,
                              files_struct *fsp,
-                             int fd, SMB_ACL_T theacl)
+                             SMB_ACL_T theacl)
 {
         /*
          * HPUX doesn't have the facl call. Fake it using the path.... JRA.
          */
-       /* For all I see, the info should already be in the fsp
-        * parameter, but get it again to be safe --- necessary? */
-        files_struct *file_struct_p = file_find_fd(fd);
-        if (file_struct_p == NULL) {
-                errno = EBADF;
-                return -1;
-        }
         /*
          * We know we're in the same conn context. So we
          * can use the relative path.
@@ -327,8 +333,9 @@ int hpuxacl_sys_acl_set_fd(vfs_handle_struct *handle,
        DEBUG(10, ("redirecting call of hpuxacl_sys_acl_set_fd to "
                "hpuxacl_sys_acl_set_file (no facl syscall on HPUX)\n"));
 
-        return hpuxacl_sys_acl_set_file(handle, file_struct_p->fsp_name,
-                       SMB_ACL_TYPE_ACCESS, theacl);
+        return hpuxacl_sys_acl_set_file(handle,
+                                       fsp->fsp_name->base_name,
+                                       SMB_ACL_TYPE_ACCESS, theacl);
 }
 
 
@@ -357,7 +364,7 @@ int hpuxacl_sys_acl_delete_def_file(vfs_handle_struct *handle,
        int count;
 
        DEBUG(10, ("entering hpuxacl_sys_acl_delete_def_file.\n"));
-       
+
        smb_acl = hpuxacl_sys_acl_get_file(handle, path, 
                                           SMB_ACL_TYPE_ACCESS);
        if (smb_acl == NULL) {
@@ -374,15 +381,15 @@ int hpuxacl_sys_acl_delete_def_file(vfs_handle_struct *handle,
                DEBUG(10, ("resulting acl is not valid!\n"));
                goto done;
        }
-       ret = acl(CONST_DISCARD(char *, path), ACL_SET, count, hpux_acl);
+       ret = acl(discard_const_p(char, path), ACL_SET, count, hpux_acl);
        if (ret != 0) {
                DEBUG(10, ("settinge file acl failed!\n"));
        }
-       
+
  done:
        DEBUG(10, ("hpuxacl_sys_acl_delete_def_file %s.\n",
                   ((ret != 0) ? "failed" : "succeeded" )));
-       SAFE_FREE(smb_acl);
+       TALLOC_FREE(smb_acl);
        return ret;
 }
 
@@ -405,16 +412,16 @@ static HPUX_ACL_T hpux_acl_init(int count)
  * Convert the SMB acl to the ACCESS or DEFAULT part of a 
  * hpux ACL, as desired.
  */
-static BOOL smb_acl_to_hpux_acl(SMB_ACL_T smb_acl, 
+static bool smb_acl_to_hpux_acl(SMB_ACL_T smb_acl, 
                                   HPUX_ACL_T *hpux_acl, int *count, 
                                   SMB_ACL_TYPE_T type)
 {
-       BOOL ret = False;
+       bool ret = False;
        int i;
        int check_which, check_rc;
 
        DEBUG(10, ("entering smb_acl_to_hpux_acl\n"));
-       
+
        *hpux_acl = NULL;
        *count = 0;
 
@@ -432,13 +439,13 @@ static BOOL smb_acl_to_hpux_acl(SMB_ACL_T smb_acl,
                switch(hpux_entry.a_type) {
                case USER:
                        DEBUG(10, ("got tag type USER with uid %d\n", 
-                                  smb_entry->uid));
-                       hpux_entry.a_id = (uid_t)smb_entry->uid;
+                                  smb_entry->info.user.uid));
+                       hpux_entry.a_id = (uid_t)smb_entry->info.user.uid;
                        break;
                case GROUP:
                        DEBUG(10, ("got tag type GROUP with gid %d\n", 
-                                  smb_entry->gid));
-                       hpux_entry.a_id = (uid_t)smb_entry->gid;
+                                  smb_entry->info.group.gid));
+                       hpux_entry.a_id = (uid_t)smb_entry->info.group.gid;
                        break;
                default:
                        break;
@@ -447,7 +454,7 @@ static BOOL smb_acl_to_hpux_acl(SMB_ACL_T smb_acl,
                        DEBUG(10, ("adding default bit to hpux ace\n"));
                        hpux_entry.a_type |= ACL_DEFAULT;
                }
-               
+
                hpux_entry.a_perm = 
                        smb_perm_to_hpux_perm(smb_entry->a_perm);
                DEBUG(10, ("assembled the following hpux ace:\n"));
@@ -472,7 +479,7 @@ static BOOL smb_acl_to_hpux_acl(SMB_ACL_T smb_acl,
 
        ret = True;
        goto done;
-       
+
  fail:
        SAFE_FREE(*hpux_acl);
  done:
@@ -486,27 +493,24 @@ static BOOL smb_acl_to_hpux_acl(SMB_ACL_T smb_acl,
  * soaris acl to the SMB_ACL format.
  */
 static SMB_ACL_T hpux_acl_to_smb_acl(HPUX_ACL_T hpux_acl, int count, 
-                                       SMB_ACL_TYPE_T type)
+                                    SMB_ACL_TYPE_T type, TALLOC_CTX *mem_ctx)
 {
        SMB_ACL_T result;
        int i;
 
-       if ((result = sys_acl_init(0)) == NULL) {
+       if ((result = sys_acl_init(mem_ctx)) == NULL) {
                DEBUG(10, ("error allocating memory for SMB_ACL\n"));
                goto fail;
        }
        for (i = 0; i < count; i++) {
                SMB_ACL_ENTRY_T smb_entry;
                SMB_ACL_PERM_T smb_perm;
-               
+
                if (!_IS_OF_TYPE(hpux_acl[i], type)) {
                        continue;
                }
-               result = SMB_REALLOC(result, 
-                                    sizeof(struct smb_acl_t) +
-                                    (sizeof(struct smb_acl_entry) *
-                                     (result->count + 1)));
-               if (result == NULL) {
+               result->acl = talloc_realloc(result, result->acl, struct smb_acl_entry, result->count + 1);
+               if (result->acl == NULL) {
                        DEBUG(10, ("error reallocating memory for SMB_ACL\n"));
                        goto fail;
                }
@@ -529,9 +533,8 @@ static SMB_ACL_T hpux_acl_to_smb_acl(HPUX_ACL_T hpux_acl, int count,
                result->count += 1;
        }
        goto done;
-       
  fail:
-       SAFE_FREE(result);
+       TALLOC_FREE(result);
  done:
        DEBUG(10, ("hpux_acl_to_smb_acl %s\n",
                   ((result == NULL) ? "failed" : "succeeded")));
@@ -546,7 +549,7 @@ static HPUX_ACL_TAG_T smb_tag_to_hpux_tag(SMB_ACL_TAG_T smb_tag)
 
        DEBUG(10, ("smb_tag_to_hpux_tag\n"));
        DEBUGADD(10, (" --> got smb tag 0x%04x\n", smb_tag));
-       
+
        switch (smb_tag) {
        case SMB_ACL_USER:
                hpux_tag = USER;
@@ -570,7 +573,7 @@ static HPUX_ACL_TAG_T smb_tag_to_hpux_tag(SMB_ACL_TAG_T smb_tag)
                DEBUGADD(10, (" !!! unknown smb tag type 0x%04x\n", smb_tag));
                break;
        }
-       
+
        DEBUGADD(10, (" --> determined hpux tag 0x%04x\n", hpux_tag));
 
        return hpux_tag;
@@ -582,7 +585,7 @@ static SMB_ACL_TAG_T hpux_tag_to_smb_tag(HPUX_ACL_TAG_T hpux_tag)
 
        DEBUG(10, ("hpux_tag_to_smb_tag:\n"));
        DEBUGADD(10, (" --> got hpux tag 0x%04x\n", hpux_tag)); 
-       
+
        hpux_tag &= ~ACL_DEFAULT; 
 
        switch (hpux_tag) {
@@ -611,7 +614,7 @@ static SMB_ACL_TAG_T hpux_tag_to_smb_tag(HPUX_ACL_TAG_T hpux_tag)
        }
 
        DEBUGADD(10, (" --> determined smb tag 0x%04x\n", smb_tag));
-       
+
        return smb_tag;
 }
 
@@ -641,14 +644,14 @@ static HPUX_PERM_T smb_perm_to_hpux_perm(const SMB_ACL_PERM_T perm)
 }
 
 
-static BOOL hpux_acl_get_file(const char *name, HPUX_ACL_T *hpux_acl, 
+static bool hpux_acl_get_file(const char *name, HPUX_ACL_T *hpux_acl, 
                                 int *count)
 {
-       BOOL result = False;
+       bool result = False;
        static HPUX_ACE_T dummy_ace;
 
        DEBUG(10, ("hpux_acl_get_file called for file '%s'\n", name));
-       
+
        /* 
         * The original code tries some INITIAL_ACL_SIZE
         * and only did the ACL_CNT call upon failure
@@ -662,7 +665,7 @@ static BOOL hpux_acl_get_file(const char *name, HPUX_ACL_T *hpux_acl,
         * instantiated or malloced each time this function is
         * called). Btw: the count parameter does not seem to matter...
         */
-       *count = acl(CONST_DISCARD(char *, name), ACL_CNT, 0, &dummy_ace);
+       *count = acl(discard_const_p(char, name), ACL_CNT, 0, &dummy_ace);
        if (*count < 0) {
                DEBUG(10, ("acl ACL_CNT failed: %s\n", strerror(errno)));
                goto done;
@@ -672,7 +675,7 @@ static BOOL hpux_acl_get_file(const char *name, HPUX_ACL_T *hpux_acl,
                DEBUG(10, ("error allocating memory for hpux acl...\n"));
                goto done;
        }
-       *count = acl(CONST_DISCARD(char *, name), ACL_GET, *count, *hpux_acl);
+       *count = acl(discard_const_p(char, name), ACL_GET, *count, *hpux_acl);
        if (*count < 0) {
                DEBUG(10, ("acl ACL_GET failed: %s\n", strerror(errno)));
                goto done;
@@ -704,12 +707,12 @@ static BOOL hpux_acl_get_file(const char *name, HPUX_ACL_T *hpux_acl,
  * time. If it should become necessary to add all of an ACL, one 
  * would have to replace this parameter by another type.
  */
-static BOOL hpux_add_to_acl(HPUX_ACL_T *hpux_acl, int *count,
+static bool hpux_add_to_acl(HPUX_ACL_T *hpux_acl, int *count,
                               HPUX_ACL_T add_acl, int add_count, 
                               SMB_ACL_TYPE_T type)
 {
        int i;
-       
+
        if ((type != SMB_ACL_TYPE_ACCESS) && (type != SMB_ACL_TYPE_DEFAULT)) 
        {
                DEBUG(10, ("invalid acl type given: %d\n", type));
@@ -750,7 +753,7 @@ static BOOL hpux_add_to_acl(HPUX_ACL_T *hpux_acl, int *count,
  * happen aclsort() will fail and return an error and someone will
  * have to fix it...)
  */
-static BOOL hpux_acl_sort(HPUX_ACL_T hpux_acl, int count)
+static bool hpux_acl_sort(HPUX_ACL_T hpux_acl, int count)
 {
        int fixmask = (count <= 4);
 
@@ -872,7 +875,7 @@ static void hpux_swap_acl_entries(HPUX_ACE_T *aclp0, HPUX_ACE_T *aclp1)
  * False - If the ACL type doesn't match any of the prohibited types.
  */ 
 
-static BOOL hpux_prohibited_duplicate_type(int acl_type)
+static bool hpux_prohibited_duplicate_type(int acl_type)
 {
        switch(acl_type) {
                case USER:
@@ -943,7 +946,7 @@ static int hpux_internal_aclsort(int acl_count, int calclass, HPUX_ACL_T aclp)
        struct hpux_acl_types acl_obj_count;
        int n_class_obj_perm = 0;
        int i, j;
+
        DEBUG(10,("Entering hpux_internal_aclsort. (calclass = %d)\n", calclass));
 
        if (hpux_aclsort_call_present()) {
@@ -1055,13 +1058,13 @@ or DEF_USER_OBJ or DEF_GROUP_OBJ or DEF_OTHER_OBJ\n"));
  * calls if it isn't there.                            
  */
 
-static BOOL hpux_acl_call_present(void)
+static bool hpux_acl_call_present(void)
 {
 
        shl_t handle = NULL;
        void *value;
        int ret_val=0;
-       static BOOL already_checked = False;
+       static bool already_checked = False;
 
        if(already_checked)
                return True;
@@ -1090,12 +1093,12 @@ static BOOL hpux_acl_call_present(void)
  * a dispatcher function could be handy...
  */
 
-static BOOL hpux_aclsort_call_present(void) 
+static bool hpux_aclsort_call_present(void) 
 {
        shl_t handle = NULL;
        void *value;
        int ret_val = 0;
-       static BOOL already_checked = False;
+       static bool already_checked = False;
 
        if (already_checked) {
                return True;
@@ -1123,11 +1126,11 @@ static BOOL hpux_aclsort_call_present(void)
  *   concrete error messages for debugging...
  *   (acl sort just says that the acl is invalid...)
  */
-static BOOL hpux_acl_check(HPUX_ACL_T hpux_acl, int count)
+static bool hpux_acl_check(HPUX_ACL_T hpux_acl, int count)
 {
        int check_rc;
        int check_which;
-       
+
        check_rc = aclcheck(hpux_acl, count, &check_which);
        if (check_rc != 0) {
                DEBUG(10, ("acl is not valid:\n"));
@@ -1150,37 +1153,20 @@ static BOOL hpux_acl_check(HPUX_ACL_T hpux_acl, int count)
 
 /* VFS operations structure */
 
-static vfs_op_tuple hpuxacl_op_tuples[] = {
-       /* Disk operations */
-       {SMB_VFS_OP(hpuxacl_sys_acl_get_file),
-        SMB_VFS_OP_SYS_ACL_GET_FILE,
-        SMB_VFS_LAYER_TRANSPARENT},
-
-       {SMB_VFS_OP(hpuxacl_sys_acl_get_fd),
-        SMB_VFS_OP_SYS_ACL_GET_FD,
-        SMB_VFS_LAYER_TRANSPARENT},
-
-       {SMB_VFS_OP(hpuxacl_sys_acl_set_file),
-        SMB_VFS_OP_SYS_ACL_SET_FILE,
-        SMB_VFS_LAYER_TRANSPARENT},
-
-       {SMB_VFS_OP(hpuxacl_sys_acl_set_fd),
-        SMB_VFS_OP_SYS_ACL_SET_FD,
-        SMB_VFS_LAYER_TRANSPARENT},
-
-       {SMB_VFS_OP(hpuxacl_sys_acl_delete_def_file),
-        SMB_VFS_OP_SYS_ACL_DELETE_DEF_FILE,
-        SMB_VFS_LAYER_TRANSPARENT},
-
-       {SMB_VFS_OP(NULL),
-        SMB_VFS_OP_NOOP,
-        SMB_VFS_LAYER_NOOP}
+static struct vfs_fn_pointers hpuxacl_fns = {
+       .sys_acl_get_file_fn = hpuxacl_sys_acl_get_file,
+       .sys_acl_get_fd_fn = hpuxacl_sys_acl_get_fd,
+       .sys_acl_blob_get_file_fn = posix_sys_acl_blob_get_file,
+       .sys_acl_blob_get_fd_fn = posix_sys_acl_blob_get_fd,
+       .sys_acl_set_file_fn = hpuxacl_sys_acl_set_file,
+       .sys_acl_set_fd_fn = hpuxacl_sys_acl_set_fd,
+       .sys_acl_delete_def_file_fn = hpuxacl_sys_acl_delete_def_file,
 };
 
 NTSTATUS vfs_hpuxacl_init(void)
 {
        return smb_register_vfs(SMB_VFS_INTERFACE_VERSION, "hpuxacl",
-                               hpuxacl_op_tuples);
+                               &hpuxacl_fns);
 }
 
 /* ENTE */