C套接字编程：connect（）上的无效参数错误

Question

我正在将客户端编写为TCP客户端服务器程序的一部分。

我的代码到达连接部分并引发了无效的参数错误，我已经遍历了几次代码，但找不到问题。

该代码接收3个参数，第一个是IP地址或主机名，第二个是端口，第三个是要发送的消息的最大长度。

我的代码使用getaddrinfo来转换ip地址或主机名，创建所需的变量，启动连接，从文件读取，发送数据和接收数据。

我使用以下代码运行代码：

gcc -std=gnu99 -O3 -Wall -o pcc_client pcc_client.c
./pcc_client 127.0.0.1 2233 4

输出为：

sockaddr_in初始化
错误，启动连接：无效的参数

#include <stdlib.h>
#include <stdio.h>
#include <stdio.h>
#include <unistd.h>
#include <assert.h>
#include <errno.h>
#include <string.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <netdb.h>
#include <dirent.h>

#define FILE_ADDR   "/dev/urandom"

int main(int argc, char *argv[]) {
    if (argc != 4) {
        printf("should receive 3 arguments Received %d args\n", argc);
        exit(1);
    }

    //Get command line arguments
    unsigned int port = atoi(argv[2]);
    int length = atoi(argv[3]); //Number of bytes to read
    char* buffer = malloc(length * sizeof(char)); //Buffer to hold data read from file
    char* recvBuf = malloc(10 * sizeof(char)); // Buffer to hold response from server

    struct addrinfo hints, *servinfo, *p;
    struct sockaddr_in *serv_addr;
    int rv;
    char ip[100];

    memset(&hints, 0, sizeof hints);
    hints.ai_family = AF_INET;
    hints.ai_socktype = SOCK_STREAM;

    if ((rv = getaddrinfo(argv[1], argv[2], &hints, &servinfo)) != 0) {
        perror("getaddrinfo error\n");
        return 1;
    }
    for (p = servinfo; p != NULL; p = p->ai_next) {
        serv_addr = (struct sockaddr_in *) p->ai_addr;
        strcpy(ip, inet_ntoa(serv_addr->sin_addr));
    }
    //   inet_aton(ip, &h.sin_addr);
    freeaddrinfo(servinfo);

    //Initialize socket
    int sockfd;
    sockfd = socket(AF_INET, SOCK_STREAM, 0);
    if (sockfd < 0) //Error creating socket
            {
        perror("Error creating socket \n");
        exit(1);
    }

    printf("socket created\n");

    //Initialize sockaddr_in structure
    memset((void*)serv_addr, 0,(size_t) sizeof(*serv_addr));
    serv_addr->sin_family = AF_INET;
    serv_addr->sin_port = htons(port);
    serv_addr->sin_addr.s_addr = inet_addr("127.0.0.1"); //change?


    //Initialize connection
    if (connect(sockfd, (struct sockaddr *) &serv_addr, sizeof(serv_addr)) < 0) { //Error connecting
        perror("Error starting connection \n");
        exit(1);
    }
    printf("connect succesful\n");

    exit(0);

}

Answer 1

你定义 serv_addr

struct sockaddr_in *serv_addr;

然后你用它

memset((void*)serv_addr, 0,(size_t) sizeof(*serv_addr));
serv_addr->sin_family = AF_INET;
serv_addr->sin_port = htons(port);
serv_addr->sin_addr.s_addr = inet_addr("127.0.0.1"); //change?

但是在代码中的这两个位置之间没有任何地方初始化指针！这意味着serv_addr在未初始化，它的值是不确定的，并会指向一些看似随机位置。取消引用指针将导致未定义的行为。

简单、自然和事实上的标准解决方案serv_addr 不是创建一个指针，而是一个结构对象：

struct sockaddr_in serv_addr;

然后，当您需要一个指针时，您可以使用 address-of 运算符&。

上述问题还由你居然复杂使用的&呼叫时，操作者connect。用serv_addr作为一个指针，则&serv_addr是一个指针的指针。它将是 类型struct sockaddr_in **。正是这个带有指向指针的指针的问题导致了错误消息，因为您发送的指针不是指向sockaddr_in结构对象的指针。

通过使用如上所示的结构对象也可以解决这个问题。

Answer 2

你用serv_addr错了。

您已声明serv_addr为sockaddr_in*指针。getaddrinfo()成功退出后，您将遍历输出列表，将指针分配serv_addr给列表中的每个指针，ai_addr然后释放列表，将serv_addr指针指向无效的内存。然后，当您尝试填充serv_addr数据时，您将浪费内存。然后，你最终甚至没有通过一个有效的指针sockaddr_in到connect()可言，你实际上是在传递一个指针的指针的sockaddr_in，这就是为什么它报告说是“无效的参数”。

实际上，您正在处理这种情况通常都是错误的。使用时 getaddrinfo()，由于它返回可能有多个套接字地址的链接列表，因此您需要遍历该列表，并尝试connect()访问每个地址，直到其中一个成功为止。如果您想升级代码以支持IPv4和IPv6（通过设置hints.ai_family = AF_UNSPEC;），则这一点尤其重要。

尝试类似这样的方法：

int main(int argc, char *argv[])
{
    if (argc != 4)
    {
        printf("should receive 3 arguments Received %d args\n", argc);
        exit(1);
    }

    struct addrinfo hints, *servinfo, *p;
    int sockfd = -1;

    memset(&hints, 0, sizeof hints);
    hints.ai_family = AF_INET; // or AF_UNSPEC
    hints.ai_socktype = SOCK_STREAM;
    hints.ai_protocol = IPPROTO_TCP;

    int rv = getaddrinfo(argv[1], argv[2], &hints, &servinfo);
    if (rv != 0)
    {
        perror("getaddrinfo error\n");
        return 1;
    }

    for (p = servinfo; p != NULL; p = p->ai_next) {
        //Initialize socket
        sockfd = socket(p->ai_family, p->ai_socktype, p->ai_protocol);
        if (sockfd < 0) continue;
        //Initialize connection
        rv = connect(sockfd, p->ai_addr, (socklen_t) p->ai_addrlen);
        if (rv == 0) break;
        close(sockfd);
        sockfd = -1;
    }

    freeaddrinfo(servinfo);

    if (sockfd < 0) //Error creating/connecting socket
    {
        perror("Error creating/connecting socket \n");
        exit(1);
    }

    printf("connect successful\n");

    ...

    close(sockfd);
    exit(0);
}