summaryrefslogtreecommitdiffstats
path: root/source/rpc_parse
diff options
context:
space:
mode:
authorJean-François Micouleau <jfm@samba.org>2000-10-05 12:41:31 +0000
committerJean-François Micouleau <jfm@samba.org>2000-10-05 12:41:31 +0000
commit83bfe94771e0afbffbb4a1a741cd9983ef612362 (patch)
tree0f1ca32884682f4b1e7380e06bc3748a79d33f46 /source/rpc_parse
parent178e6971005505d2debd74b761ecfaa982336a53 (diff)
downloadsamba-83bfe94771e0afbffbb4a1a741cd9983ef612362.tar.gz
samba-83bfe94771e0afbffbb4a1a741cd9983ef612362.tar.xz
samba-83bfe94771e0afbffbb4a1a741cd9983ef612362.zip
fixing prs_unistr2(). For the special case of an empty (but existing)
string. J.F.
Diffstat (limited to 'source/rpc_parse')
-rw-r--r--source/rpc_parse/parse_prs.c14
1 files changed, 8 insertions, 6 deletions
diff --git a/source/rpc_parse/parse_prs.c b/source/rpc_parse/parse_prs.c
index d6e5797aaf0..401efaadc64 100644
--- a/source/rpc_parse/parse_prs.c
+++ b/source/rpc_parse/parse_prs.c
@@ -617,22 +617,24 @@ BOOL prs_unistr2(BOOL charmode, char *name, prs_struct *ps, int depth, UNISTR2 *
return False;
if (UNMARSHALLING(ps)) {
- str->buffer = (uint16 *)prs_alloc_mem(ps,str->uni_str_len * sizeof(uint16));
+ str->buffer = (uint16 *)prs_alloc_mem(ps,str->uni_max_len * sizeof(uint16));
if (str->buffer == NULL)
return False;
+ memset(str->buffer, '\0', str->uni_max_len * sizeof(uint16));
}
+ /* If the string is empty, we don't have anything to stream */
+ if (str->uni_str_len==0)
+ return True;
+
p = (char *)str->buffer;
/* If we're using big-endian, reverse to get little-endian. */
- if(ps->bigendian_data)
- {
+ if(ps->bigendian_data) {
DBG_RW_PSVAL(charmode, name, depth, ps->data_offset,
ps->io, ps->bigendian_data, q, p,
str->uni_str_len)
- }
- else
- {
+ } else {
DBG_RW_PCVAL(charmode, name, depth, ps->data_offset,
ps->io, q, p, str->uni_str_len * sizeof(uint16))
}