From 4a4e67bd2c7bb0022ae760ca3919aa2620be5943 Mon Sep 17 00:00:00 2001 From: david Date: Thu, 23 Sep 2010 16:32:11 +0000 Subject: [PATCH] Set the close_handlers member in worker Thread objects. The lack of this caused them to use the close_handlers table of the parent thread that spawned them instead. See http://seclists.org/nmap-dev/2010/q3/804 for more details. --- CHANGELOG | 4 ++++ nse_main.lua | 1 + 2 files changed, 5 insertions(+) diff --git a/CHANGELOG b/CHANGELOG index 106195d4a..a56bdac6c 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,5 +1,9 @@ # Nmap Changelog ($Id$); -*-text-*- +o [NSE] Fixed a bug with worker threads calling the wrong destructors. + Fixing this allows better parallelism in http-brute.nse. The problem + was reported by Patrik Karlsson. [David, Patrick] + o [Zenmap] Made the topology node radiuses grow logarithmically instead of linearly, so that hosts with thousands of open ports don't overwhelm the diagram. Also only open ports (not diff --git a/nse_main.lua b/nse_main.lua index e1b01acdf..f2adddd45 100644 --- a/nse_main.lua +++ b/nse_main.lua @@ -692,6 +692,7 @@ local function run (threads, scantype) type = current.type, parent = current.parent, info = format("'%s' worker (%s)", current.short_basename, tostring(co)); + close_handlers = {}, -- d = function(...) end, -- output no debug information }; local thread_mt = {