Как хранить IP-адреса клиентов в массиве на C (программирование сокетов)

Question

У меня есть простая программа клиент / сервер в сокете на C. Я использую inet_ntoa, который возвращает ip клиентов, подключенных к серверам. Я запускаю цикл 2 раза, чтобы соединить 2 клиента и сохранить int в массиве char.

Проблема в том, что когда я печатаю массив, он всегда дает последний ip, добавленный в массив. Например:

x.x.x.x connected 
y.y.y.y connected 

Массив печатает y.y.y.y два раза

#include <netdb.h>
#include <netinet/in.h>
#include <stdlib.h>
#include <string.h>
#include <sys/socket.h>
#include <sys/types.h>
#include <arpa/inet.h>

#define MAX 80
#define PORT 8080
#define SA struct sockaddr

// Driver function 
int main() {

  int sockfd, connfd, len;
  struct sockaddr_in servaddr, cli;
  struct sockaddr_in addr_remote;
  char * ips[2];
  // socket create and verification 
  sockfd = socket(AF_INET, SOCK_STREAM, 0);
  if (sockfd == -1) {
    printf("socket creation failed...\n");
    exit(0);
  } 
  else
    printf("Socket successfully created..\n");
  bzero( & servaddr, sizeof(servaddr));

  // assign IP, PORT 
  servaddr.sin_family = AF_INET;
  servaddr.sin_addr.s_addr = htonl(INADDR_ANY);
  servaddr.sin_port = htons(PORT);

  // Binding newly created socket to given IP and verification 
  if ((bind(sockfd, (SA * ) & servaddr, sizeof(servaddr))) != 0) {
    printf("socket bind failed...\n");
    exit(0);
  } 
  else
    printf("Socket successfully binded..\n");

  // Now server is ready to listen and verification 
  int i = 0;
  for (i = 0; i < 2; i++) {
    if ((listen(sockfd, 5)) != 0) {
      printf("Listen failed...\n");
      exit(0);
    } 
    else
      printf("Server listening..\n");
    len = sizeof(cli);

    // Accept the data packet from client and verification 
    connfd = accept(sockfd, (SA * ) & addr_remote, & len);
    if (connfd < 0) {
      printf("server acccept failed...\n");
      exit(0);
    }
    else
      printf("server acccept the client...\n");

    // Function for chatting between client and server 
    // func(connfd); 
    //printf( " Welcome %s " , inet_ntoa(addr_remote.sin_addr));
    ips[i] = inet_ntoa(addr_remote.sin_addr);

  }
  for (i = 0; i < 2; i++) {

    printf("%s", ips[i]);

  }
  // After chatting close the socket 
  close(sockfd);
}

Answer 1

Вы должны выделить свой собственный массив символов для IP и скопировать его из статического буфера, возвращаемого inet_ntoa(). Простой пример:

char ips[2][20];
...
strcpy(ips[i], inet_ntoa(...))

EDIT: Дело в том, что функция inet_ntoa() сохраняет свой результат во внутреннем статически распределенном буфере и возвращает только указатель на него, который является постоянным. Таким образом, ваши ip[0] и ip[1] содержат одинаковый указатель, который указывает на последний IP-адрес, полученный из inet_ntoa().