diff options
| author | Gerald Carter <jerry@samba.org> | 2003-12-10 21:13:44 +0000 |
|---|---|---|
| committer | Gerald Carter <jerry@samba.org> | 2003-12-10 21:13:44 +0000 |
| commit | ae8d35b2a2cead8a21a7c4f4bfbefa830384470e (patch) | |
| tree | 8807d430c524dbe96bf642acce0f24352179ee7f | |
| parent | 87fddf6a988dfcdb3f1d3a715df585b6c6efa9d7 (diff) | |
| download | samba-ae8d35b2a2cead8a21a7c4f4bfbefa830384470e.tar.gz samba-ae8d35b2a2cead8a21a7c4f4bfbefa830384470e.tar.bz2 samba-ae8d35b2a2cead8a21a7c4f4bfbefa830384470e.zip | |
Fix UNISTR2 length bug in LsaQueryInfo(3) that cause SID resolution to fail on local files on on domain members; bug 875
(This used to be commit c6594e35573186966a4d57404f1c06b98670db06)
| -rw-r--r-- | source3/include/rpc_misc.h | 2 | ||||
| -rw-r--r-- | source3/rpc_parse/parse_misc.c | 2 | ||||
| -rw-r--r-- | source3/rpc_server/srv_lsa_nt.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/source3/include/rpc_misc.h b/source3/include/rpc_misc.h index aaaad55c20..29ae212185 100644 --- a/source3/include/rpc_misc.h +++ b/source3/include/rpc_misc.h @@ -403,5 +403,5 @@ typedef struct buffer4_info } BUFFER4; -enum unistr2_term_codes { UNI_FLAGS_NONE = 0, UNI_STR_TERMINATE = 1, UNI_MAXLEN_TERMINATE = 2 }; +enum unistr2_term_codes { UNI_FLAGS_NONE = 0, UNI_STR_TERMINATE = 1, UNI_MAXLEN_TERMINATE = 2, UNI_BROKEN_NON_NULL = 3 }; #endif /* _RPC_MISC_H */ diff --git a/source3/rpc_parse/parse_misc.c b/source3/rpc_parse/parse_misc.c index afdf0fc4c9..976ba20681 100644 --- a/source3/rpc_parse/parse_misc.c +++ b/source3/rpc_parse/parse_misc.c @@ -914,7 +914,7 @@ void init_unistr2(UNISTR2 *str, const char *buf, enum unistr2_term_codes flags) str->uni_max_len = num_chars; str->offset = 0; str->uni_str_len = num_chars; - if (num_chars && (flags == UNI_MAXLEN_TERMINATE)) + if ( num_chars && ((flags == UNI_MAXLEN_TERMINATE) || (flags == UNI_BROKEN_NON_NULL)) ) str->uni_max_len++; } diff --git a/source3/rpc_server/srv_lsa_nt.c b/source3/rpc_server/srv_lsa_nt.c index e545d8c267..2b2cb91924 100644 --- a/source3/rpc_server/srv_lsa_nt.c +++ b/source3/rpc_server/srv_lsa_nt.c @@ -66,7 +66,7 @@ static void init_dom_query(DOM_QUERY *d_q, const char *dom_name, DOM_SID *dom_si /* this string is supposed to be non-null terminated. */ /* But the maxlen in this UNISTR2 must include the terminating null. */ - init_unistr2(&d_q->uni_domain_name, dom_name, UNI_MAXLEN_TERMINATE); + init_unistr2(&d_q->uni_domain_name, dom_name, UNI_BROKEN_NON_NULL); /* * I'm not sure why this really odd combination of length |