Nmap Development mailing list archives
Re: parsing of script-args is broken
From: Patrick Donnelly <batrick () batbytes com>
Date: Mon, 27 Apr 2009 16:38:20 -0600
Hi Jah, On Mon, Apr 27, 2009 at 7:53 AM, jah <jah () zadkiel plus com> wrote:
Hi Patrick, This patch looks good to me. Do you think we should allow lists too: vhost={domain.co.uk,domain.com}? Your patch will pass that example to loadstring() as: ["vhost"]={domain.co.uk,domain.com} whereas it is intented as: ["vhost"]={"domain.co.uk", "domain.com"} supplying the arguments with quoted list-members works fine by the way: --script-args vhost={'domain.co.uk','domain.com'} => ["vhost"]={'domain.co.uk','domain.com'} A list seems to me to be a reasonable thing to be able to supply to a script and with this patch, the user will have to quote the list items themselves which isn't very intuitive given that we can quote key/value pairs on their behalf.
I don't think array were originally intended but I have updated the patch to allow this. Correctly implementing this with Lua's limited regular expressions can be tricky to get correct so if you find any more bugs (or have suggestions) let us know. Thanks, -- -Patrick Donnelly "One of the lessons of history is that nothing is often a good thing to do and always a clever thing to say." -Will Durant
Attachment:
sargs2.patch
Description:
_______________________________________________ Sent through the nmap-dev mailing list http://cgi.insecure.org/mailman/listinfo/nmap-dev Archived at http://SecLists.Org
Current thread:
- parsing of script-args is broken jah (Apr 26)
- Re: parsing of script-args is broken Patrick Donnelly (Apr 27)
- Re: parsing of script-args is broken jah (Apr 27)
- Re: parsing of script-args is broken Patrick Donnelly (Apr 27)
- patch looks good jah (Apr 27)
- Re: parsing of script-args is broken David Fifield (Apr 30)
- Re: parsing of script-args is broken Patrick Donnelly (May 01)
- Re: parsing of script-args is broken Patrick Donnelly (May 10)
- Re: parsing of script-args is broken Patrick Donnelly (May 15)
- Re: parsing of script-args is broken Ron (May 17)
- Re: parsing of script-args is broken jah (May 17)
- Re: parsing of script-args is broken Patrick Donnelly (May 17)
- Re: parsing of script-args is broken jah (May 18)
- Re: parsing of script-args is broken jah (Apr 27)
- Re: parsing of script-args is broken Patrick Donnelly (Apr 27)