s3: VFS: solarisacl: Remove solarisacl_sys_acl_delete_def_file().
authorJeremy Allison <jra@samba.org>
Sat, 15 May 2021 01:22:23 +0000 (18:22 -0700)
committerJeremy Allison <jra@samba.org>
Tue, 18 May 2021 17:29:34 +0000 (17:29 +0000)
No longer used.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Noel Power <npower@samba.org>
source3/modules/vfs_solarisacl.c
source3/modules/vfs_solarisacl.h

index 78edb0719e5b1e9b9010e821344dc2730d64973e..d35721b9b04001f532b8c7fb042b457985f8445b 100644 (file)
@@ -297,60 +297,6 @@ int solarisacl_sys_acl_set_fd(vfs_handle_struct *handle,
        return ret;
 }
 
-/*
- * delete the default ACL of a directory
- *
- * This is achieved by fetching the access ACL and rewriting it 
- * directly, via the solaris system call: the SETACL call on 
- * directories writes both the access and the default ACL as provided.
- *
- * XXX: posix acl_delete_def_file returns an error if
- * the file referred to by path is not a directory.
- * this function does not complain but the actions 
- * have no effect on a file other than a directory.
- * But sys_acl_delete_default_file is only called in
- * smbd/posixacls.c after having checked that the file
- * is a directory, anyways. So implementing the extra
- * check is considered unnecessary. --- Agreed? XXX
- */
-int solarisacl_sys_acl_delete_def_file(vfs_handle_struct *handle,
-                               const struct smb_filename *smb_fname)
-{
-       SMB_ACL_T smb_acl;
-       int ret = -1;
-       SOLARIS_ACL_T solaris_acl = NULL;
-       int count;
-
-       DEBUG(10, ("entering solarisacl_sys_acl_delete_def_file.\n"));
-       
-       smb_acl = solarisacl_sys_acl_get_file(handle, smb_fname->base_name,
-                                             SMB_ACL_TYPE_ACCESS, talloc_tos());
-       if (smb_acl == NULL) {
-               DEBUG(10, ("getting file acl failed!\n"));
-               goto done;
-       }
-       if (!smb_acl_to_solaris_acl(smb_acl, &solaris_acl, &count, 
-                                   SMB_ACL_TYPE_ACCESS))
-       {
-               DEBUG(10, ("conversion smb_acl -> solaris_acl failed.\n"));
-               goto done;
-       }
-       if (!solaris_acl_sort(solaris_acl, count)) {
-               DEBUG(10, ("resulting acl is not valid!\n"));
-               goto done;
-       }
-       ret = acl(smb_fname->base_name, SETACL, count, solaris_acl);
-       if (ret != 0) {
-               DEBUG(10, ("settinge file acl failed!\n"));
-       }
-       
- done:
-       DEBUG(10, ("solarisacl_sys_acl_delete_def_file %s.\n",
-                  ((ret != 0) ? "failed" : "succeeded" )));
-       TALLOC_FREE(smb_acl);
-       return ret;
-}
-
 /*
  * delete the default ACL of a directory
  *
@@ -831,7 +777,6 @@ static struct vfs_fn_pointers solarisacl_fns = {
        .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_fd_fn = solarisacl_sys_acl_set_fd,
-       .sys_acl_delete_def_file_fn = solarisacl_sys_acl_delete_def_file,
        .sys_acl_delete_def_fd_fn = solarisacl_sys_acl_delete_def_fd,
 };
 
index 9debe4feea9bcdd728441032a80786670c4349c5..2a9290573e4d8f0425f6f4f433bfacdf0aa67705 100644 (file)
@@ -38,9 +38,6 @@ int solarisacl_sys_acl_set_fd(vfs_handle_struct *handle,
                              files_struct *fsp,
                              SMB_ACL_T theacl);
 
-int solarisacl_sys_acl_delete_def_file(vfs_handle_struct *handle,
-                               const struct smb_filename *smb_fname);
-
 int solarisacl_sys_acl_delete_def_fd(vfs_handle_struct *handle,
                                files_struct *fsp);