diff options
| author | Jeremy Allison <jra@samba.org> | 2007-11-27 22:22:35 -0800 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2007-11-27 22:22:35 -0800 |
| commit | 1cd1c9db3f3350dd4585d96457e918652557855c (patch) | |
| tree | 6b92d688d8590e2ade1af0720f3a51f8de560a26 /source3 | |
| parent | b11e6f21589a00c1803985f202be9cd360944d43 (diff) | |
| download | samba-1cd1c9db3f3350dd4585d96457e918652557855c.tar.gz samba-1cd1c9db3f3350dd4585d96457e918652557855c.tar.bz2 samba-1cd1c9db3f3350dd4585d96457e918652557855c.zip | |
Remove unused prototype for smbldap_get_single_pstring().
Don't use pstr_sprintf() on an fstring - change to talloc.
Jeremy.
(This used to be commit 6cae4b5fa1bcb848cb2a28daaafeefd6bcd08274)
Diffstat (limited to 'source3')
| -rw-r--r-- | source3/include/smbldap.h | 2 | ||||
| -rw-r--r-- | source3/passdb/pdb_ldap.c | 10 |
2 files changed, 8 insertions, 4 deletions
diff --git a/source3/include/smbldap.h b/source3/include/smbldap.h index 3e2c878dbe..d9d7aca98e 100644 --- a/source3/include/smbldap.h +++ b/source3/include/smbldap.h @@ -141,8 +141,6 @@ void smbldap_make_mod(LDAP *ldap_struct, LDAPMessage *existing, bool smbldap_get_single_attribute (LDAP * ldap_struct, LDAPMessage * entry, const char *attribute, char *value, int max_len); -bool smbldap_get_single_pstring (LDAP * ldap_struct, LDAPMessage * entry, - const char *attribute, pstring value); char *smbldap_get_dn(LDAP *ld, LDAPMessage *entry); int smbldap_modify(struct smbldap_state *ldap_state, const char *dn, diff --git a/source3/passdb/pdb_ldap.c b/source3/passdb/pdb_ldap.c index 707e3f3a30..7a3d55999e 100644 --- a/source3/passdb/pdb_ldap.c +++ b/source3/passdb/pdb_ldap.c @@ -3403,11 +3403,14 @@ static NTSTATUS ldapsam_setsamgrent(struct pdb_methods *my_methods, { struct ldapsam_privates *ldap_state = (struct ldapsam_privates *)my_methods->private_data; - fstring filter; + char *filter = NULL; int rc; const char **attr_list; - pstr_sprintf( filter, "(objectclass=%s)", LDAP_OBJ_GROUPMAP); + filter = talloc_asprintf(NULL, "(objectclass=%s)", LDAP_OBJ_GROUPMAP); + if (!filter) { + return NT_STATUS_NO_MEMORY; + } attr_list = get_attr_list( NULL, groupmap_attr_list ); rc = smbldap_search(ldap_state->smbldap_state, lp_ldap_group_suffix(), LDAP_SCOPE_SUBTREE, filter, @@ -3421,9 +3424,12 @@ static NTSTATUS ldapsam_setsamgrent(struct pdb_methods *my_methods, lp_ldap_group_suffix(), filter)); ldap_msgfree(ldap_state->result); ldap_state->result = NULL; + TALLOC_FREE(filter); return NT_STATUS_UNSUCCESSFUL; } + TALLOC_FREE(filter); + DEBUG(2, ("ldapsam_setsamgrent: %d entries in the base!\n", ldap_count_entries(ldap_state->smbldap_state->ldap_struct, ldap_state->result))); |