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 | 72ecdb387db60a4afab4d4fabf76f4c1d7ecbeeb (patch) | |
tree | df86716ea9a0a6cb3fbf7c5e4e6a380d1a0a29bc /source | |
parent | 824175854421f7c27d31ad673a8790dd018ae350 (diff) | |
download | samba-72ecdb387db60a4afab4d4fabf76f4c1d7ecbeeb.tar.gz |
Never second guess tridge.... (always talloc an internal pointer... ).
Jeremy.
Diffstat (limited to 'source')
-rw-r--r-- | source/rpc_parse/parse_samr.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/source/rpc_parse/parse_samr.c b/source/rpc_parse/parse_samr.c index 2146ccec91f..47bf3f390dd 100644 --- a/source/rpc_parse/parse_samr.c +++ b/source/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++) { |