From ddaf29c3e49dcf55a7d4803353725d0e944de9c7 Mon Sep 17 00:00:00 2001 From: david Date: Tue, 22 Feb 2011 20:16:34 +0000 Subject: [PATCH] Fix two bugs in the new shortport.ssl portrule, reported by Tom Sellers. The first is that "sctp" was being treated as a list of port states, when it should have been put in the same table as "tcp". The second is the same bug that we've seen before, which is that it was returning a function value instead of the result of calling the function. --- nselib/shortport.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/nselib/shortport.lua b/nselib/shortport.lua index 4a64c7439..acdee7597 100644 --- a/nselib/shortport.lua +++ b/nselib/shortport.lua @@ -184,5 +184,5 @@ local LIKELY_SSL_SERVICES = { -- portrule = shortport.ssl function ssl(host, port) return port.version.service_tunnel == "ssl" or - port_or_service(LIKELY_SSL_PORTS, LIKELY_SSL_SERVICES, "tcp", "sctp") + port_or_service(LIKELY_SSL_PORTS, LIKELY_SSL_SERVICES, {"tcp", "sctp"})(host, port) end