Nmap Development mailing list archives

Re: Upgrading NSE to Lua 5.3


From: Patrick Donnelly <batrick () batbytes com>
Date: Mon, 9 Feb 2015 12:20:25 -0500

On Mon, Feb 9, 2015 at 12:11 PM, Paulino Calderon Pale
<paulino () calderonpale com> wrote:
Hi Patrick,

I see now. I realized the library bin was broken but didn’t think about checking the specific tag you told us to test 
hehe. The branch at that revision seems to work as expected. I got an error related to bit.bor() but it looks like it 
is related to the changes you are working on:

sh-3.2# ./nmap -p443 --script ssl* scanme.nmap.org

Starting Nmap 6.47SVN ( http://nmap.org ) at 2015-02-09 11:03 CST
NSE: Failed to load /Users/cldrn/Code/nse-lua53/./scripts/ssl-ccs-injection.nse:
/Users/cldrn/Code/nse-lua53/./nselib/bit.lua:36: attempt to call a nil value (global 'select')
stack traceback:
        /Users/cldrn/Code/nse-lua53/./nselib/bit.lua:36: in function 'bit.bor'
        /Users/cldrn/Code/nse-lua53/./nselib/vulns.lua:393: in main chunk
        [C]: in function 'require'
        /Users/cldrn/Code/nse-lua53/./scripts/ssl-ccs-injection.nse:9: in function 
</Users/cldrn/Code/nse-lua53/./scripts/ssl-ccs-injection.nse:1>


Anyway, the new branch worked on my side =).

Oops, that's a bug. The select function needs to be saved in a local.
I've committed a fix but unfortunately it will be on top of the recent
changes to the bin library. If you backport 33991, it should work.

-- 
Patrick Donnelly
_______________________________________________
Sent through the dev mailing list
https://nmap.org/mailman/listinfo/dev
Archived at http://seclists.org/nmap-dev/

Current thread: