diff options
author | Jeremy Allison <jra@samba.org> | 2003-06-05 02:44:24 +0000 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2003-06-05 02:44:24 +0000 |
commit | bf50387f492ac273a7c3d22c0f9d298466059e89 (patch) | |
tree | 6cb4ed52d9ad06a0f8e9f7021d8bc2e55b57f28d /source3/rpc_parse | |
parent | 3bdfd57a2dc590896a2e45652ad4f5e3c2358108 (diff) | |
download | samba-bf50387f492ac273a7c3d22c0f9d298466059e89.tar.gz samba-bf50387f492ac273a7c3d22c0f9d298466059e89.tar.bz2 samba-bf50387f492ac273a7c3d22c0f9d298466059e89.zip |
Never second guess tridge.... (always talloc an internal pointer... ).
Jeremy.
(This used to be commit 72ecdb387db60a4afab4d4fabf76f4c1d7ecbeeb)
Diffstat (limited to 'source3/rpc_parse')
-rw-r--r-- | source3/rpc_parse/parse_samr.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/source3/rpc_parse/parse_samr.c b/source3/rpc_parse/parse_samr.c index 2146ccec91..47bf3f390d 100644 --- a/source3/rpc_parse/parse_samr.c +++ b/source3/rpc_parse/parse_samr.c @@ -4614,16 +4614,14 @@ BOOL samr_io_r_query_aliasmem(const char *desc, SAMR_R_QUERY_ALIASMEM * r_u, return False; if (r_u->ptr != 0 && r_u->num_sids != 0) { - uint32 *ptr_sid; + uint32 *ptr_sid = NULL; if(!prs_uint32("num_sids1", ps, depth, &r_u->num_sids1)) return False; - if (UNMARSHALLING(ps)) { - ptr_sid = talloc(ps->mem_ctx, sizeof(uint32) * r_u->num_sids1); - if (!ptr_sid) { - return False; - } + ptr_sid = talloc(ps->mem_ctx, sizeof(uint32) * r_u->num_sids1); + if (!ptr_sid) { + return False; } for (i = 0; i < r_u->num_sids1; i++) { |