diff options
| author | Jeremy Allison <jra@samba.org> | 2001-05-10 20:58:38 +0000 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2001-05-10 20:58:38 +0000 |
| commit | 89798ef71c43d176229812066cb5d651c65d1744 (patch) | |
| tree | 59ed224aa5addbe01a1f4a64968ed1bca7d53747 /source3/param | |
| parent | fcc23d40dcb017fe5cff761c4633f95d025eb3f1 (diff) | |
| download | samba-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')
| -rw-r--r-- | source3/param/loadparm.c | 2 |
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) && |