diff options
| author | Karolin Seeger <kseeger@samba.org> | 2009-01-09 16:52:12 +0100 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2009-01-09 17:00:22 +0100 |
| commit | b516a2abcadb4a58fe7058ddbb4e38f53a467426 (patch) | |
| tree | 451cc5b9c0179c724bf8c291d2a9eebdc54ec247 /source3/utils/net_rpc_rights.c | |
| parent | 86b1b82d9ef29f0431d27d62ce2ec289078a6802 (diff) | |
| download | samba-b516a2abcadb4a58fe7058ddbb4e38f53a467426.tar.gz samba-b516a2abcadb4a58fe7058ddbb4e38f53a467426.tar.bz2 samba-b516a2abcadb4a58fe7058ddbb4e38f53a467426.zip | |
s3/net: Display error message if user does not exist.
net rpc rights grant: Verify if the username can be resolved to a SID and
display a proper error message if it does not. Otherwise users might think
setting privileges worked fine, but in fact it does not.
Karolin
Diffstat (limited to 'source3/utils/net_rpc_rights.c')
| -rw-r--r-- | source3/utils/net_rpc_rights.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/source3/utils/net_rpc_rights.c b/source3/utils/net_rpc_rights.c index 2bc5efe3cf..ddcfff3685 100644 --- a/source3/utils/net_rpc_rights.c +++ b/source3/utils/net_rpc_rights.c @@ -449,8 +449,11 @@ static NTSTATUS rpc_rights_grant_internal(struct net_context *c, } result = name_to_sid(pipe_hnd, mem_ctx, &sid, argv[0]); + if (NT_STATUS_EQUAL(result, NT_STATUS_NONE_MAPPED)) + result = NT_STATUS_NO_SUCH_USER; + if (!NT_STATUS_IS_OK(result)) - return result; + goto done; result = rpccli_lsa_open_policy2(pipe_hnd, mem_ctx, true, SEC_RIGHTS_MAXIMUM_ALLOWED, |