summaryrefslogtreecommitdiff
path: root/source3/rpc_server/eventlog
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2012-11-13 12:48:53 -0800
committerAndrew Bartlett <abartlet@samba.org>2012-11-13 22:48:19 +0100
commit236977bf4642c035bb22cfcd1cee481c5f6c6da1 (patch)
treefcec6c3d337a98120db720753b07bcf7163a2011 /source3/rpc_server/eventlog
parenta4434297f19a3520d0f2ac242d4e99576d927ecc (diff)
downloadsamba-236977bf4642c035bb22cfcd1cee481c5f6c6da1.tar.gz
samba-236977bf4642c035bb22cfcd1cee481c5f6c6da1.tar.bz2
samba-236977bf4642c035bb22cfcd1cee481c5f6c6da1.zip
Change get_nt_acl_no_snum() to return an NTSTATUS, not a struct security_descriptor *.
Internally change the implementation to use SMB_VFS_GET_NT_ACL() instead of SMB_VFS_FGET_NT_ACL() with a faked-up file struct. Andrew Bartlett Reviewed by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'source3/rpc_server/eventlog')
-rw-r--r--source3/rpc_server/eventlog/srv_eventlog_nt.c11
1 files changed, 7 insertions, 4 deletions
diff --git a/source3/rpc_server/eventlog/srv_eventlog_nt.c b/source3/rpc_server/eventlog/srv_eventlog_nt.c
index a05ea3fcbe..a3e719a38b 100644
--- a/source3/rpc_server/eventlog/srv_eventlog_nt.c
+++ b/source3/rpc_server/eventlog/srv_eventlog_nt.c
@@ -91,12 +91,15 @@ static bool elog_check_access( EVENTLOG_INFO *info, const struct security_token
/* get the security descriptor for the file */
- sec_desc = get_nt_acl_no_snum( info, tdbname, SECINFO_OWNER | SECINFO_GROUP | SECINFO_DACL);
+ status = get_nt_acl_no_snum( info,
+ tdbname,
+ SECINFO_OWNER | SECINFO_GROUP | SECINFO_DACL,
+ &sec_desc);
TALLOC_FREE( tdbname );
- if ( !sec_desc ) {
- DEBUG(5,("elog_check_access: Unable to get NT ACL for %s\n",
- tdbname));
+ if (!NT_STATUS_IS_OK(status)) {
+ DEBUG(5,("elog_check_access: Unable to get NT ACL for %s: %s\n",
+ tdbname, nt_errstr(status)));
return False;
}