r22573: Fix old bug mixing free() and talloc_free()

when the add_sid_to_array_XX code was moved
from malloc to talloc. Found running valgrind
and rpcclient. Needs merging for 3.0.25 final.
Jeremy.
(This used to be commit 8af56dbd00045049ea3c5022822bbaeeecbd9661)

diff --git a/source3/rpc_server/srv_lsa_nt.c b/source3/rpc_server/srv_lsa_nt.c
index 76ec38af6fbfd2af3c97f090fc5bc4f1a803f656..2343e3eedba152c0c7cfa1b7cf3e018532524406 100644 (file)
--- a/source3/rpc_server/srv_lsa_nt.c
+++ b/source3/rpc_server/srv_lsa_nt.c
@@ -1502,7 +1502,7 @@ NTSTATUS _lsa_enum_accounts(pipes_struct *p, LSA_Q_ENUM_ACCOUNTS *q_u, LSA_R_ENU
                (*sids).ptr_sid[j] = 1;
        }
 
-       SAFE_FREE(sid_list);
+       talloc_free(sid_list);
 
        init_lsa_r_enum_accounts(r_u, num_entries);