Почему неожиданно меняется значение? inet_ntoa

Я пытаюсь создать функцию, которая собирает все IP-адреса, присутствующие на машине, и сможет определить, присутствует ли IP-параметр в параметре.

Проблема в том, что значение переменной, содержащей мой IP-адрес, изменяется после выполнения inet_ntoa на другом IN_ADDR.

Я безуспешно пытался использовать std :: string. Я знаю, что inet_ntoa устарел, но я безуспешно пробовал что-то еще. Также хотелось бы понять, что я делаю не так.

void CIPUtilities::AwaitIPAddress(struct sockaddr_in WFIAddr) {
// also tried with AwaitIPAddress(char * IPName)
char* IPName = inet_ntoa(WFIAddr.sin_addr);
printf("\n Waiting for the IP:  %s to be ready.\n", IPName);
int i;
bool ctn = TRUE;

/* Variables used by GetIpAddrTable */
PMIB_IPADDRTABLE pIPAddrTable;
DWORD dwSize = 0;
DWORD dwRetVal = 0;
IN_ADDR IPAddr;
ULONG outBufLen = 0;
ULONG Iterations = 0;

/* Variables used to return error message */
LPVOID lpMsgBuf;

// Allocate a 15 KB buffer to start with.
outBufLen = WORKING_BUFFER_SIZE;
do {

    do {
        //printf("inside do\n");
        Sleep(300); //to be removed ? Only there to avaoid to loop too much.
        pIPAddrTable = (MIB_IPADDRTABLE*)MALLOC(outBufLen);

        if (pIPAddrTable == NULL) {
            printf("Memory allocation failed for GetIpAddrTable\n");
            //exit(1); // no need to exit we need debug
        }

        dwRetVal = GetIpAddrTable(pIPAddrTable, &outBufLen, 0);

        if (dwRetVal == ERROR_BUFFER_OVERFLOW) {
            FREE(pIPAddrTable);
            pIPAddrTable = NULL;
        }
        else {
            break;
        }
    } while ((dwRetVal == ERROR_BUFFER_OVERFLOW) && (Iterations < MAX_TRIES));

    if (dwRetVal == NO_ERROR) {
        // If successful, search the IP from the data we retrived
        for (i = 0; i < (int)pIPAddrTable->dwNumEntries; i++) {
            IPAddr.S_un.S_addr = (u_long)pIPAddrTable->table[i].dwAddr;
            printf("1- The value of IPName is %s\n", IPName);
            printf("\tIP Address[%d]:     \t%s\n", i, inet_ntoa(IPAddr));
            printf("2- The value of IPName is %s\n", IPName);
            //if (strcmp(IPName, inet_ntoa(IPAddr)) == 0) {
            //    printf("IP adress[%s] is found in AddrTable\n", inet_ntoa(IPAddr));
            //    ctn = FALSE; // We exit the loop because the adress is created.
            //    break; // no need to parse more addresses.
            //}
        }
    }
    else
    {
        printf("GetIpAddrTable failed with error %d\n", dwRetVal);
        printf("Required size should be %d\n", dwSize);
        if (FormatMessage(FORMAT_MESSAGE_ALLOCATE_BUFFER | FORMAT_MESSAGE_FROM_SYSTEM | FORMAT_MESSAGE_IGNORE_INSERTS, NULL, dwRetVal, MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),       // Default language
            (LPTSTR)&lpMsgBuf, 0, NULL)) {
            printf("\tError: %s", (char*)lpMsgBuf);
            LocalFree(lpMsgBuf);
        }
        //exit(1);  // no need to exit we need debug
    }

    if (pIPAddrTable) {
        FREE(pIPAddrTable);
        pIPAddrTable = NULL;
    }

} while (ctn);}

В основном вывод будет:

Waiting for the IP:  10.0.4.3 to be ready.
1- The value of IPName is 10.0.4.3
        IP Address[0]:          160.223.17.135
2- The value of IPName is 160.223.17.135
1- The value of IPName is 160.223.17.135
        IP Address[1]:          169.254.165.50
2- The value of IPName is 169.254.165.50
1- The value of IPName is 169.254.165.50
        IP Address[2]:          10.0.4.3
2- The value of IPName is 10.0.4.3
1- The value of IPName is 10.0.4.3
        IP Address[3]:          10.0.12.44
2- The value of IPName is 10.0.12.44
1- The value of IPName is 10.0.12.44
        IP Address[4]:          192.168.0.17
2- The value of IPName is 192.168.0.17
1- The value of IPName is 192.168.0.17
        IP Address[5]:          127.0.0.1
2- The value of IPName is 127.0.0.1
1- The value of IPName is 127.0.0.1

Я удалил некоторые части функции, чтобы сделать ее более читаемой.


c++ sockets pointers winsock2
person Gcha83    schedule 18.04.2021    source источник
comment
inet_ntoa возвращает указатель на внутренний статический буфер, который будет перезаписываться при каждом вызове.   -  person G.M.    schedule 18.04.2021

Ответы (1)


arrow_upward
1
arrow_downward

Вкратце: это происходит потому, что вы используете возвращаемый указатель напрямую, а не копируете строку.

char* IPName = inet_ntoa(WFIAddr.sin_addr);

Как указано в документации inet_ntoa (см. здесь для Windows или здесь для Linux) результатом является указатель на статический буфер:

Возвращенная строка гарантированно действительна только до тех пор, пока следующий вызов функции Windows Sockets не будет выполнен в том же потоке. Следовательно, данные должны быть скопированы до того, как будет выполнен другой вызов Windows Sockets.

Все, что вам нужно сделать, это скопировать содержимое буфера в строковую переменную.

char IPName[18];
char* temp = inet_ntoa(WFIAddr.sin_addr); 
if (temp != NULL)
{
  strcpy(IPName, temp);
}

(Подробнее о размере вашего буфера см. здесь)

И вы, конечно же, должны обработать случай ошибки (если temp равен NULL)

person Roxxorfreak    schedule 18.04.2021