s4-samr: Use GUID_buf_string() in dcesrv_samr_EnumDomainGroups()
authorAndrew Bartlett <abartlet@samba.org>
Thu, 15 Nov 2018 00:44:23 +0000 (13:44 +1300)
committerAndrew Bartlett <abartlet@samba.org>
Tue, 20 Nov 2018 21:14:17 +0000 (22:14 +0100)
This avoids memory allocation

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Gary Lockyer <gary@catalyst.net.nz>
source4/rpc_server/samr/dcesrv_samr.c

index 8360064278496d3ac656270798c3120b3db7698b..b485b5a2b628ef896674abcf42a4457d3cdec596 100644 (file)
@@ -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;