Re: [Patch] main.cc

[Fyodor <fyodor () insecure org>]

On Mon, Mar 24, 2008 at 05:10:43PM +0100, Gisle Vanem wrote:
> main.cc uses SIGSEGV, SIGINT etc., but <signal.h> isn't
> included (for MingW). An easy fix:
>
> --- SVN-Latest/main.cc  Sun Mar 23 16:53:29 2008
> +++ main.cc     Sun Mar 23 17:13:49 2008
> @@ -102,6 +102,7 @@
>  #include "nmap.h"
>  #include "NmapOps.h"
>  #include "utils.h"
> +#include "tcpip.h"

Thanks Gisle.  But why not just include signal.h?  I suppose it may
depend on another header file or two, but I'd rather include those
than include all of tcpip.h just for that.  Be sure to look at how
other headers handle these files, because some need various guards to
keep them out of Windows compilation, etc.

Cheers,
-F


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