summaryrefslogtreecommitdiff
path: root/source4/nbt_server
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2005-06-27 19:32:52 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 13:18:53 -0500
commit9bf7217aa21e4d11fa40e7bdc6cc69ea82ff83f0 (patch)
tree714fb5766a39e4bf936079eba55dc3dd0f9a7a0b /source4/nbt_server
parentfa8d539e0fbce952266a246364e5e9cd537b29fd (diff)
downloadsamba-9bf7217aa21e4d11fa40e7bdc6cc69ea82ff83f0.tar.gz
samba-9bf7217aa21e4d11fa40e7bdc6cc69ea82ff83f0.tar.bz2
samba-9bf7217aa21e4d11fa40e7bdc6cc69ea82ff83f0.zip
r7955: we should not free a struct which is passed as a parameter, to a function
the caller should free it this fixed a double free bug noticed by Дейтер Александр Валериевич <tiamat@komi.mts.ru>  metze (This used to be commit ee1a5d5419f4d79af5c447a6b397a0f4dc89310a)
Diffstat (limited to 'source4/nbt_server')
-rw-r--r--source4/nbt_server/dgram/request.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/source4/nbt_server/dgram/request.c b/source4/nbt_server/dgram/request.c
index f59193bec5..8ccdeabf71 100644
--- a/source4/nbt_server/dgram/request.c
+++ b/source4/nbt_server/dgram/request.c
@@ -72,7 +72,6 @@ NTSTATUS nbtd_dgram_setup(struct nbtd_interface *iface, const char *bind_address
if (!NT_STATUS_IS_OK(status)) {
DEBUG(0,("Failed to bind to %s:%d - %s\n",
iface->bcast_address, lp_dgram_port(), nt_errstr(status)));
- talloc_free(iface);
return status;
}
@@ -87,7 +86,6 @@ NTSTATUS nbtd_dgram_setup(struct nbtd_interface *iface, const char *bind_address
if (!NT_STATUS_IS_OK(status)) {
DEBUG(0,("Failed to bind to %s:%d - %s\n",
bind_address, lp_dgram_port(), nt_errstr(status)));
- talloc_free(iface);
return status;
}
dgram_set_incoming_handler(iface->dgmsock, dgram_request_handler, iface);