Nmap Development mailing list archives

Re: afp-serverinfo.nse script -- new AFP library


From: Patrik Karlsson <patrik () cqure net>
Date: Tue, 30 Mar 2010 00:31:48 +0200


On 29 mar 2010, at 23.12, David Fifield wrote:

Yes, I have that same reference. The even-byte padding got fixed by you
and Andrew. The other thing that looked like an alignment issue but
really wasn't was the two-byte length for the UTF-8 server name.

The only other problem I saw was the reversed byte order in the server
flags, which is trivial to fix. Again see
http://seclists.org/nmap-dev/2010/q1/665.

We had some off-list mail correspondence and I don't know if the last
version made it back to the list.
Anyway, I'm happy to bring the patch back in to my latest release of
the library and help out ironing out the last few problems.

If you've got it in a convenient format then please send it to the list
so we can work it in.

I'm attaching a patch to afp.lua and a patch to the script so that it uses the new library.
The script currently calls the fp_get_server_info method of the Proto class directly rather than going through the 
Helper class.
As it does not need to have a session established before being called adding a function to the helper class would 
simply act as a wrapper.



//Patrik


David Fifield
_______________________________________________
Sent through the nmap-dev mailing list
http://cgi.insecure.org/mailman/listinfo/nmap-dev
Archived at http://seclists.org/nmap-dev/

--
Patrik Karlsson
http://www.cqure.net
http://www.twitter.com/nevdull77





_______________________________________________
Sent through the nmap-dev mailing list
http://cgi.insecure.org/mailman/listinfo/nmap-dev
Archived at http://seclists.org/nmap-dev/


Current thread: