Re: Ncat problems and fixes

[David Fifield <david () bamsoftware com>]

On Wed, Feb 13, 2013 at 05:49:08PM +0400, OlkhinAG () nk rn-inform ru wrote:
> 1. Ncat Portable building for windows:
>                 OpenSSL paths should be changed to ..\mswin32\OpenSSL\include 

Can you explain more what you mean? mswin32/OpenSSL no longer exists in
the source tree. There is a precompiled OpenSSL in /nmap-mswin32-aux;
see http://nmap.org/book/inst-windows.html#inst-win-source.

>                 All references to openssl/applink.c should be removed - not relevant for modern openSSL (also from 
> https://secwiki.org/w/Nmap/Ncat_Portable)

What is the error you get when using applink? Can you give us a tested
patch that removes it?

> 2. Odd behavior when running several instances of ncat using -e or -c switch. 
>                 Problem caused by non-unique named pipe names in run_command_redirected in ncat_exec_win.c
>                 Fix:
> int myPid = GetCurrentProcessId();
> Snprintf(pipe_name, sizeof(pipe_name), "\\\\.\\pipe\\ncat-%d-%d", myPid, pipe_serial_no);

Thanks, this is committed in r30641.

> 3. Ncat with -e or -c switch not ends when connection dies.
>                 There is error in subprocess_thread_func:
> DWORD n
> ...
> n = ncat_recv(
> if (n <= 0)
>                                n is unsigned and if connection dies recv returns -1
>                 Fix:
>                 int n;
> ...
> if (GetOverlappedResult(info->child_out_r, &overlap, (LPDWORD) &n, FALSE)) {

Thanks, fixed in r30642.

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