Nmap Development mailing list archives

[Patch] nsock/tests/tests_main.c


From: Gisle Vanem <gvanem () broadpark no>
Date: Tue, 06 Aug 2013 17:02:46 +0200

I do like Nmap and will try to contribute. Here is a small fix for now:

--- SVN-Latests/nsock/tests/tests_main.c    2013-06-27 15:25:14 +0000
+++ nsock/tests/tests_main.c 2013-07-11 11:55:05 +0000
@@ -64,7 +64,7 @@
    fflush(stdout);
    rc = test_case_run(current);
    if (rc) {
-      printf(TEST_FAILED " (%s)\n", strerror(-rc));
+      printf(TEST_FAILED " (%s)\n", socket_strerror(-rc));
      break;
    }
    printf(TEST_OK "\n");

----------

Using strerror() for Winsock failures is a big no-no.
Besides, this test program uses ANSI codes to print to the console.
Also a no-no.

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


Current thread: