summaryrefslogtreecommitdiffstats
path: root/source/nmbd/nmbd_winsproxy.c
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>1998-08-30 04:27:26 +0000
committerAndrew Tridgell <tridge@samba.org>1998-08-30 04:27:26 +0000
commitc5e5c25c854e54f59291057ba47c4701b5910ebe (patch)
treedb5e5fda202a062e8f7be1791e748d266588ec1d /source/nmbd/nmbd_winsproxy.c
parenta564e4662711d384069757ce3ee5adcadc1b061d (diff)
downloadsamba-c5e5c25c854e54f59291057ba47c4701b5910ebe.tar.gz
samba-c5e5c25c854e54f59291057ba47c4701b5910ebe.tar.xz
samba-c5e5c25c854e54f59291057ba47c4701b5910ebe.zip
added some defensive programming to nmbd. This mostly means zeroing
areas of memory before freeing them. While doing this I also found a couple of real bugs. In two places we were freeing some memory that came from the stack, which leads to a certain core dump on many sytems.
Diffstat (limited to 'source/nmbd/nmbd_winsproxy.c')
-rw-r--r--source/nmbd/nmbd_winsproxy.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/source/nmbd/nmbd_winsproxy.c b/source/nmbd/nmbd_winsproxy.c
index 5635124bcda..2084d3915ab 100644
--- a/source/nmbd/nmbd_winsproxy.c
+++ b/source/nmbd/nmbd_winsproxy.c
@@ -191,6 +191,7 @@ static void wins_proxy_userdata_free_fn(struct userdata_struct *userdata)
p->locked = False;
free_packet(p);
+ ZERO_STRUCTP(userdata);
free((char *)userdata);
}