s4:ntvfs/posix: name->dos.attrib isn't initialized in pvfs_access_check_create()

That's why we have the 'container' parameter to indicate
the caller wants to create a directory.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Mar 22 17:39:11 CET 2011 on sn-devel-104
(cherry picked from commit b8d0994627ef237fdc92bebd9c9a26f9d62764bc)

diff --git a/source4/ntvfs/posix/pvfs_acl.c b/source4/ntvfs/posix/pvfs_acl.c
index addd6808ca3c40b358b7159d7bee5401ed8a8e5a..67b544d4dee550d2961de2ca97009d32c81fe1f6 100644 (file)
--- a/source4/ntvfs/posix/pvfs_acl.c
+++ b/source4/ntvfs/posix/pvfs_acl.c
@@ -698,7 +698,7 @@ NTSTATUS pvfs_access_check_create(struct pvfs_state *pvfs,
        status = pvfs_resolve_parent(pvfs, req, name, &parent);
        NT_STATUS_NOT_OK_RETURN(status);
 
-       if (name->dos.attrib & FILE_ATTRIBUTE_DIRECTORY) {
+       if (container) {
                parent_mask = SEC_DIR_ADD_SUBDIR;
        } else {
                parent_mask = SEC_DIR_ADD_FILE;