diff options
author | Andrew Bartlett <abartlet@samba.org> | 2018-11-15 13:44:23 +1300 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2018-11-20 22:14:17 +0100 |
commit | 8d8303b0aecb472de33f7dbd87c80e939e98034a (patch) | |
tree | ff098a988322150aaf880caa4f63c52afe25efaf /source4 | |
parent | b6241852937e7e2d6254b3e9bba221d34edc5571 (diff) | |
download | samba-8d8303b0aecb472de33f7dbd87c80e939e98034a.tar.gz |
s4-samr: Use GUID_buf_string() in dcesrv_samr_EnumDomainGroups()
This avoids memory allocation
Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Gary Lockyer <gary@catalyst.net.nz>
Diffstat (limited to 'source4')
-rw-r--r-- | source4/rpc_server/samr/dcesrv_samr.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/source4/rpc_server/samr/dcesrv_samr.c b/source4/rpc_server/samr/dcesrv_samr.c index 83600642784..b485b5a2b62 100644 --- a/source4/rpc_server/samr/dcesrv_samr.c +++ b/source4/rpc_server/samr/dcesrv_samr.c @@ -1329,8 +1329,10 @@ static NTSTATUS dcesrv_samr_EnumDomainGroups(struct dcesrv_call_state *dce_call, attrs, 0); if (ret == LDB_ERR_NO_SUCH_OBJECT) { + struct GUID_txt_buf guid_buf; char *guid_str = - GUID_string(mem_ctx, &cache->entries[idx]); + GUID_buf_string(&cache->entries[idx], + &guid_buf); DBG_WARNING("GUID [%s] not found\n", guid_str); continue; } else if (ret != LDB_SUCCESS) { @@ -1342,8 +1344,10 @@ static NTSTATUS dcesrv_samr_EnumDomainGroups(struct dcesrv_call_state *dce_call, rec->msgs[0], "objectSID"); if (objectsid == NULL) { + struct GUID_txt_buf guid_buf; char *guid_str = - GUID_string(mem_ctx, &cache->entries[idx]); + GUID_buf_string(&cache->entries[idx], + &guid_buf); DBG_WARNING("objectSID for GUID [%s] not found\n", guid_str); continue; |