如何使用C ++对Windows计算机上的远程计算机执行ping操作？

Question

I used below code. It works, but in debug mode in Visual Studio if you stop the debug the computer gave blue screen so it is not useful. I did some research and i found this is a common bug for icmpapi. Is there a any way to ping computer c++?

        #include <winsock2.h>
        #include <iphlpapi.h>
        #include <icmpapi.h>

        //Declare and initialize variables
        HANDLE hIcmpFile;
        unsigned long ipaddr = INADDR_NONE;
        DWORD dwRetVal       = 0;
        DWORD dwError        = 0;
        char SendData[]      = "Data Buffer";
        LPVOID ReplyBuffer   = NULL;
        DWORD ReplySize      = 0;

        QByteArray ipArray   = computerIt->GetIP().toLocal8Bit();
        const char *hostIP   = ipArray.data();

        ipaddr = inet_addr(hostIP);
        if ( ipaddr == INADDR_NONE ) 
        {
            EventLogger::LogMessage(true, "<%s> Computer in <%s> Computer Group, IP initialization failed. (inet_addr(hostIP))", computerIt->GetName().toUtf8().constData(), computerGroupIt->GetName().toUtf8().constData());
            break;
        }

        hIcmpFile = IcmpCreateFile();
        if ( hIcmpFile == INVALID_HANDLE_VALUE )
        {
            EventLogger::LogMessage(true, "<%s> Computer in <%s> Computer Group, IcmpCreatefile returned error", computerIt->GetName().toUtf8().constData(), computerGroupIt->GetName().toUtf8().constData());
            break;
        }

        // Allocate space for at a single reply
        ReplySize = sizeof (ICMP_ECHO_REPLY) + sizeof (SendData) + 8;
        ReplyBuffer = (VOID *) malloc(ReplySize);
        if ( ReplyBuffer == NULL )
        {
            EventLogger::LogMessage(true, "<%s> Computer in <%s> Computer Group, unable to allocate memory for reply buffer", computerIt->GetName().toUtf8().constData(), computerGroupIt->GetName().toUtf8().constData());
            break;
        }

        // Starting pinging
        dwRetVal = IcmpSendEcho2(hIcmpFile, NULL, NULL, NULL,
                                ipaddr, SendData, sizeof (SendData), NULL,
                                ReplyBuffer, ReplySize, 1000);
        if ( dwRetVal == 0 )
        {
            computerIt->SetAliveStatus(false);
        }
        else
        {
            computerIt->SetAliveStatus(true);
        }

Answer 1

In the above code there is bug in IcmpSendEcho2 method! It is only seen in debug mode! (See the link) https://msdn.microsoft.com/en-us/library/windows/desktop/aa366050(v=vs.85).aspx

I used raw ping methods for the ping operation. It is much slower but it is safety!

You can check raw ping method: http://tangentsoft.net/wskfaq/examples/rawping.html

Answer 2

be careful. at:

// https://msdn.microsoft.com/en-us/library/windows/desktop/aa366050(v=vs.85).aspx

the code works , BUT is missing an IMPORTANT POINT if you paste/copy this code: MISSING free(ReplyBuffer)