summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2008-01-09 22:09:55 +0100
committerMichael Adam <obnox@samba.org>2008-01-09 22:09:55 +0100
commit7c6d77dd883df958f9231f28d0080573e08e8add (patch)
tree72d1fb7b9839fe2d5cf3ce66d286612a710527e3
parentfbd9a15996ba4beb48c12a5632ca812e862e984c (diff)
downloadsamba-7c6d77dd883df958f9231f28d0080573e08e8add.tar.gz
samba-7c6d77dd883df958f9231f28d0080573e08e8add.tar.bz2
samba-7c6d77dd883df958f9231f28d0080573e08e8add.zip
Fix memleak in ldapsam_rename_sam_account() found by IBM checker.
The check for out of memory was the wrong way round. Michael (This used to be commit d7a7b793203b986823859ac5171d2d4c30e52415)
-rw-r--r--source3/passdb/pdb_ldap.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/passdb/pdb_ldap.c b/source3/passdb/pdb_ldap.c
index 28df56bdb1..90a6ff011b 100644
--- a/source3/passdb/pdb_ldap.c
+++ b/source3/passdb/pdb_ldap.c
@@ -1973,7 +1973,7 @@ static NTSTATUS ldapsam_rename_sam_account(struct pdb_methods *my_methods,
/* rename the posix user */
rename_script = SMB_STRDUP(lp_renameuser_script());
- if (rename_script) {
+ if (rename_script == NULL) {
return NT_STATUS_NO_MEMORY;
}