summaryrefslogtreecommitdiff
path: root/source3/param/loadparm.c
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2001-05-10 20:58:38 +0000
committerJeremy Allison <jra@samba.org>2001-05-10 20:58:38 +0000
commit89798ef71c43d176229812066cb5d651c65d1744 (patch)
tree59ed224aa5addbe01a1f4a64968ed1bca7d53747 /source3/param/loadparm.c
parentfcc23d40dcb017fe5cff761c4633f95d025eb3f1 (diff)
downloadsamba-89798ef71c43d176229812066cb5d651c65d1744.tar.gz
samba-89798ef71c43d176229812066cb5d651c65d1744.tar.bz2
samba-89798ef71c43d176229812066cb5d651c65d1744.zip
free_service() was never setting the valid field to False. This is a bug
that's been here since 2.0.x and before.... This would cause a memleak. Jeremy. (This used to be commit 3412950dbd5b70f28ffdbe1a79488a409ca1a051)
Diffstat (limited to 'source3/param/loadparm.c')
-rw-r--r--source3/param/loadparm.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index c843489596..8888c45933 100644
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -1740,6 +1740,8 @@ static void free_service(service * pservice)
pservice->copymap = NULL;
}
+ pservice->valid = False;
+
for (i = 0; parm_table[i].label; i++)
if ((parm_table[i].type == P_STRING ||
parm_table[i].type == P_USTRING) &&