Question

我已经花了很多时间，所以我真的希望有人可以为我提供一些敏锐的洞察力。

我有以下主要功能：

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

int main(int argc, char **argv) {

char *serv_IP;
in_port_t serv_port;
int sock;
struct sockaddr_in serv_addr;

serv_IP = argv[1];
serv_port = atoi(argv[2]);

if ((sock = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP)) < 0) {
    fprintf(stderr, "Failed to create TCP socket\r\n");
    exit(1);
}

memset(&serv_addr, 0, sizeof(serv_addr));
serv_addr.sin_family = AF_INET;

if (inet_pton(AF_INET, serv_IP, &serv_addr.sin_addr.s_addr) == 0) {
    fprintf(stderr, "Invalid IP address\r\n");
    exit(1);
}

serv_addr.sin_port = htons(serv_port);

if (connect(sock, (struct sockaddr *) &serv_addr, sizeof(serv_addr)) < 0) {
    fprintf(stderr, "Failed to connect to serv\r\n");
    exit(1);
}
else {
    printf("You're connected!\n);
}
close(sock)
return 0;
}

现在，这段代码运行得很好。但是，我想要做的是用一个辅助函数调用替换对connect（）的调用，如下所示：

void function(int sock, struct sockaddr_in *serv_addr) {

if (connect(sock, (struct sockaddr *) serv_addr, sizeof(serv_addr)) < 0) {
    printf("Server IP = %s\n", inet_ntoa(serv_addr->sin_addr));
    printf("Server port = %d\n", ntohs(serv_addr->sin_port));
    fprintf(stderr, "Failed to connect to server\r\n");
    exit(1);
}
else {
    // Do other stuff
}
}

我从main（）中删除对connect（）的调用，并将其替换为函数调用：

function(sock, &serv_addr);

一旦调用该函数，就会打印出正确的IP和端口号，但我仍然无法连接到我的服务器。唯一的区别是，在我的main函数（）中，我在连接调用中使用＆amp;前缀serv_addr。 - 即connect（sock，（struct sockaddr *）＆amp; serv_addr ，sizeof（serv_addr）） - 引用它的地址，我不在helper函数中这样做，因为serv_addr的地址已经作为参数传递 - 即connect（sock，（struct sockaddr *） serv_addr ，sizeof（serv_addr））。如果我添加＆amp ;,那就没有区别了，以防你想知道。

所以，将＆amp; serv_addr传递给function（）看似正确，正如我能够打印出正确的IP和端口号验证的那样，为什么我可以连接到main（）但不是当我将serv_addr结构作为参数传递给另一个函数并从那里调用connect（）时？

提前感谢您的帮助！

Answer 1

当sizeof(serv_addr)声明为serv_addr时，

sockaddr_in返回16，但在声明为sockaddr_in*时返回4（32位）或8（64位）。无论哪种方式都太小，AF_INET都需要16.如果errno失败时看了connect()，那么它会告诉您传递了无效的参数值。

您需要直接使用sizeof(sockaddr_in)：

void function(int sock, struct sockaddr_in *serv_addr)
{
    if (connect(sock, (struct sockaddr *) serv_addr, sizeof(sockaddr_in)) < 0)

或间接通过sizeof(*serv_addr)：

void function(int sock, struct sockaddr_in *serv_addr)
{
    if (connect(sock, (struct sockaddr *) serv_addr, sizeof(*serv_addr)) < 0)

Answer 2

那就做到了！非常感谢您的快速回复!!我永远不会想到sizeof（）的返回是一个潜在的问题，它返回实际值的不同大小与指向值的指针。但是，完全有道理。我刚刚阅读了errno.h标题以及如何使用它。

修复它的确切行是：

if (connect(sock, (struct sockaddr *) serv_addr, sizeof(struct sockaddr_in)) < 0)

我不需要'＆amp;'我是对的。在serv_add前面。你还需要sizeof（）中的“struct”，否则它会将sockaddr_in作为未声明的变量返回。

无论如何，再次感谢。

现在我终于可以继续使用我的代码了。

传递sockaddr_in作为参数时，connect（）函数失败

2 个答案: