diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2012-11-13 12:48:53 -0800 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2012-11-13 22:48:19 +0100 |
| commit | 236977bf4642c035bb22cfcd1cee481c5f6c6da1 (patch) | |
| tree | fcec6c3d337a98120db720753b07bcf7163a2011 /source3/rpc_server | |
| parent | a4434297f19a3520d0f2ac242d4e99576d927ecc (diff) | |
| download | samba-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')
| -rw-r--r-- | source3/rpc_server/eventlog/srv_eventlog_nt.c | 11 |
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; } |