diff options
| author | Jeremy Allison <jra@samba.org> | 2001-05-10 20:58:40 +0000 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2001-05-10 20:58:40 +0000 |
| commit | 237655d1eacfa27d7894217cff5d672f05a3e2d4 (patch) | |
| tree | 04499b2da5ddfc6d750a9d0fa0a78c3386dc573c | |
| parent | 9f8baceaf2f1700e15dbf274a8007a28822240e0 (diff) | |
| download | samba-237655d1eacfa27d7894217cff5d672f05a3e2d4.tar.gz samba-237655d1eacfa27d7894217cff5d672f05a3e2d4.tar.xz samba-237655d1eacfa27d7894217cff5d672f05a3e2d4.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.
| -rw-r--r-- | source/param/loadparm.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/source/param/loadparm.c b/source/param/loadparm.c index bb7413b8e95..45fd4959f23 100644 --- a/source/param/loadparm.c +++ b/source/param/loadparm.c @@ -1716,6 +1716,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) && |