summaryrefslogtreecommitdiff
path: root/source3/smbd/service.c
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2011-07-04 18:35:21 +0200
committerStefan Metzmacher <metze@samba.org>2011-07-05 13:28:08 +0200
commit720fa46f9443ccbe471b265f1c2b9cb9782a3c26 (patch)
tree8bf810cd262bdf2b1b207f339a690dfed271b363 /source3/smbd/service.c
parent1c022d2e414607633323e65abbc63bb3aeaaa6a4 (diff)
downloadsamba-720fa46f9443ccbe471b265f1c2b9cb9782a3c26.tar.gz
samba-720fa46f9443ccbe471b265f1c2b9cb9782a3c26.tar.bz2
samba-720fa46f9443ccbe471b265f1c2b9cb9782a3c26.zip
s3: Calculate&store the maximum share access mask
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'source3/smbd/service.c')
-rw-r--r--source3/smbd/service.c37
1 files changed, 15 insertions, 22 deletions
diff --git a/source3/smbd/service.c b/source3/smbd/service.c
index c1d4dd1799..5c410be02a 100644
--- a/source3/smbd/service.c
+++ b/source3/smbd/service.c
@@ -641,28 +641,21 @@ connection_struct *make_connection_snum(struct smbd_server_connection *sconn,
*
*/
- {
- bool can_write = False;
-
- can_write = share_access_check(
- conn->session_info->security_token,
- lp_servicename(snum), FILE_WRITE_DATA, NULL);
-
- if (!can_write) {
- if (!share_access_check(
- conn->session_info->security_token,
- lp_servicename(snum), FILE_READ_DATA,
- NULL)) {
- /* No access, read or write. */
- DEBUG(0,("make_connection: connection to %s "
- "denied due to security "
- "descriptor.\n",
- lp_servicename(snum)));
- *pstatus = NT_STATUS_ACCESS_DENIED;
- goto err_root_exit;
- } else {
- conn->read_only = True;
- }
+ share_access_check(conn->session_info->security_token,
+ lp_servicename(snum), MAXIMUM_ALLOWED_ACCESS,
+ &conn->share_access);
+
+ if ((conn->share_access & FILE_WRITE_DATA) == 0) {
+ if ((conn->share_access & FILE_READ_DATA) == 0) {
+ /* No access, read or write. */
+ DEBUG(0,("make_connection: connection to %s "
+ "denied due to security "
+ "descriptor.\n",
+ lp_servicename(snum)));
+ *pstatus = NT_STATUS_ACCESS_DENIED;
+ goto err_root_exit;
+ } else {
+ conn->read_only = True;
}
}
/* Initialise VFS function pointers */