sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2024-12-22 13:34:15 +03:00
Code

smbd: Remove "flags2" from open_file_ntcreate()

Browse Source 
"flags" carried just the O_ACCMODE bits, "flags2" everything
else. Unify them.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
This commit is contained in:
Volker Lendecke 2023-09-05 15:55:26 +02:00 committed by Ralph Boehme
parent 02d9321ce0
commit f701faf667
1 changed files with 22 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
source3/smbd/open.c
View File

@ -3777,7 +3777,6 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
{ {
struct smb_filename *smb_fname = fsp->fsp_name; struct smb_filename *smb_fname = fsp->fsp_name;
int flags=0; int flags=0;
int flags2=0;
bool file_existed = VALID_STAT(smb_fname->st); bool file_existed = VALID_STAT(smb_fname->st);
bool def_acl = False; bool def_acl = False;
bool posix_open = False; bool posix_open = False;
@ -3965,7 +3964,7 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
return NT_STATUS_INVALID_PARAMETER; return NT_STATUS_INVALID_PARAMETER;
} }
flags2 = disposition_to_open_flags(create_disposition); flags = disposition_to_open_flags(create_disposition);
/* We only care about matching attributes on file exists and /* We only care about matching attributes on file exists and
* overwrite. */ * overwrite. */
@ -4002,7 +4001,7 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
open_access_mask = access_mask; open_access_mask = access_mask;
if (flags2 & O_TRUNC) { if (flags & O_TRUNC) {
open_access_mask |= FILE_WRITE_DATA; /* This will cause oplock breaks. */ open_access_mask |= FILE_WRITE_DATA; /* This will cause oplock breaks. */
} }
@ -4032,7 +4031,7 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
* mean the same thing under DOS and Unix. * mean the same thing under DOS and Unix.
*/ */
flags = calculate_open_access_flags(access_mask, private_flags); flags |= calculate_open_access_flags(access_mask, private_flags);
/* /*
* Currently we only look at FILE_WRITE_THROUGH for create options. * Currently we only look at FILE_WRITE_THROUGH for create options.
@ -4040,12 +4039,12 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
#if defined(O_SYNC) #if defined(O_SYNC)
if ((create_options & FILE_WRITE_THROUGH) && lp_strict_sync(SNUM(conn))) { if ((create_options & FILE_WRITE_THROUGH) && lp_strict_sync(SNUM(conn))) {
flags2 |= O_SYNC; flags |= O_SYNC;
} }
#endif /* O_SYNC */ #endif /* O_SYNC */
if (posix_open && (access_mask & FILE_APPEND_DATA)) { if (posix_open && (access_mask & FILE_APPEND_DATA)) {
flags2 |= O_APPEND; flags |= O_APPEND;
} }
if (!posix_open && !CAN_WRITE(conn)) { if (!posix_open && !CAN_WRITE(conn)) {
@ -4054,7 +4053,7 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
* O_CREAT or O_TRUNC are set, but for compatibility with * O_CREAT or O_TRUNC are set, but for compatibility with
* older versions of Samba we just AND them out. * older versions of Samba we just AND them out.
*/ */
flags2 &= ~(O_CREAT|O_TRUNC); flags &= ~(O_CREAT | O_TRUNC);
} }
/* /*
@ -4069,13 +4068,13 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
* oplock we must periodically poll for available open * oplock we must periodically poll for available open
* using O_NONBLOCK. * using O_NONBLOCK.
*/ */
flags2 |= O_NONBLOCK; flags |= O_NONBLOCK;
/* /*
* Ensure we can't write on a read-only share or file. * Ensure we can't write on a read-only share or file.
*/ */
if (flags != O_RDONLY && file_existed && if (((flags & O_ACCMODE) != O_RDONLY) && file_existed &&
(!CAN_WRITE(conn) || IS_DOS_READONLY(existing_dos_attributes))) { (!CAN_WRITE(conn) || IS_DOS_READONLY(existing_dos_attributes))) {
DEBUG(5,("open_file_ntcreate: write access requested for " DEBUG(5,("open_file_ntcreate: write access requested for "
"file %s on read only %s\n", "file %s on read only %s\n",
@ -4101,8 +4100,7 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
fsp->posix_flags |= FSP_POSIX_FLAGS_ALL; fsp->posix_flags |= FSP_POSIX_FLAGS_ALL;
} }
if ((create_options & FILE_DELETE_ON_CLOSE) && if ((create_options & FILE_DELETE_ON_CLOSE) && (flags & O_CREAT) &&
(flags2 & O_CREAT) &&
!file_existed) { !file_existed) {
/* Delete on close semantics for new files. */ /* Delete on close semantics for new files. */
status = can_set_delete_on_close(fsp, status = can_set_delete_on_close(fsp,
@ -4117,21 +4115,22 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
* Ensure we pay attention to default ACLs on directories if required. * Ensure we pay attention to default ACLs on directories if required.
*/ */
if ((flags2 & O_CREAT) && lp_inherit_acls(SNUM(conn)) && if ((flags & O_CREAT) && lp_inherit_acls(SNUM(conn)) &&
(def_acl = directory_has_default_acl_fsp(parent_dir_fname->fsp))) (def_acl = directory_has_default_acl_fsp(parent_dir_fname->fsp))) {
{
unx_mode = (0777 & lp_create_mask(SNUM(conn))); unx_mode = (0777 & lp_create_mask(SNUM(conn)));
} }
DEBUG(4,("calling open_file with flags=0x%X flags2=0x%X mode=0%o, " DEBUG(4,
("calling open_file with flags=0x%X mode=0%o, "
"access_mask = 0x%x, open_access_mask = 0x%x\n", "access_mask = 0x%x, open_access_mask = 0x%x\n",
(unsigned int)flags, (unsigned int)flags2, (unsigned int)flags,
(unsigned int)unx_mode, (unsigned int)access_mask, (unsigned int)unx_mode,
(unsigned int)access_mask,
(unsigned int)open_access_mask)); (unsigned int)open_access_mask));
{ {
struct vfs_open_how how = { struct vfs_open_how how = {
.flags = flags | flags2, .flags = flags,
.mode = unx_mode, .mode = unx_mode,
}; };
fsp_open = open_file(req, fsp_open = open_file(req,
@ -4255,7 +4254,7 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
if (new_file_created) { if (new_file_created) {
info = FILE_WAS_CREATED; info = FILE_WAS_CREATED;
} else { } else {
if (flags2 & O_TRUNC) { if (flags & O_TRUNC) {
info = FILE_WAS_OVERWRITTEN; info = FILE_WAS_OVERWRITTEN;
} else { } else {
info = FILE_WAS_OPENED; info = FILE_WAS_OPENED;
@ -4328,8 +4327,7 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
} }
/* Should we atomically (to the client at least) truncate ? */ /* Should we atomically (to the client at least) truncate ? */
if ((!new_file_created) && if ((!new_file_created) && (flags & O_TRUNC) &&
(flags2 & O_TRUNC) &&
(S_ISREG(fsp->fsp_name->st.st_ex_mode))) { (S_ISREG(fsp->fsp_name->st.st_ex_mode))) {
int ret; int ret;