diff options
| author | Volker Lendecke <vlendec@samba.org> | 2003-07-18 09:05:30 +0000 |
|---|---|---|
| committer | Volker Lendecke <vlendec@samba.org> | 2003-07-18 09:05:30 +0000 |
| commit | ffbb6ee3b8ee815e5e4f8132a9225d6a627094f5 (patch) | |
| tree | 89e9e75b52d9cfc234391141c5ae0a5f42534c9c /source3/passdb/pdb_get_set.c | |
| parent | 5e252c86bd97305f3c6578afb29ccd148f985e60 (diff) | |
| download | samba-ffbb6ee3b8ee815e5e4f8132a9225d6a627094f5.tar.gz samba-ffbb6ee3b8ee815e5e4f8132a9225d6a627094f5.tar.bz2 samba-ffbb6ee3b8ee815e5e4f8132a9225d6a627094f5.zip | |
Fix two memleaks in pdb_ldap.c.
Whoever put the private.backend_private_data_free_fn thingy into
SAM_ACCOUNT, could you please revisit my change to pdb_get_set.c and
comment on my comment there?
Thanks,
Volker
(This used to be commit 922ec277d1c80b5532f5cac0ee99ae7cd20f83f1)
Diffstat (limited to 'source3/passdb/pdb_get_set.c')
| -rw-r--r-- | source3/passdb/pdb_get_set.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/source3/passdb/pdb_get_set.c b/source3/passdb/pdb_get_set.c index ba07a4e01c..1505458551 100644 --- a/source3/passdb/pdb_get_set.c +++ b/source3/passdb/pdb_get_set.c @@ -1027,9 +1027,18 @@ BOOL pdb_set_backend_private_data (SAM_ACCOUNT *sampass, void *private_data, if (!sampass) return False; +#if 0 + /* With this check backend_private_data_free_fn is *never* set + as the methods are never set anywhere. What is this + supposed to do ???? + + Volker + */ + /* does this backend 'own' this SAM_ACCOUNT? */ if (my_methods != sampass->private.backend_private_methods) return False; +#endif if (sampass->private.backend_private_data && sampass->private.backend_private_data_free_fn) { sampass->private.backend_private_data_free_fn(&sampass->private.backend_private_data); |