Question

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

void error(char *msg)
{
    perror(msg);
    exit(0);
}

int main(int argc, char *argv[])
{
    int sockfd, portno, n;
    struct sockaddr_in serv_addr;
    struct hostent *server;

    char buffer[256];

    if (argc < 3)
    {
        fprintf(stderr, "usage %s hostname port\n", argv[0]);
    }
    portno = atoi(argv[2]);
    sockfd = socket(AF_INET, SOCK_STREAM, 0);

    if (sockfd < 0);
    {
        error("ERROR opening socket");
    }
    server = gethostbyname(argv[1]);
    if (server == NULL)
    {
        fprintf(stderr, "ERROR, no such host\n");
        exit(0);
    }
    bzero((char *)&serv_addr, sizeof(serv_addr));
    serv_addr.sin_family = AF_INET;
    bcopy((char *)server->h_addr,(char *)&serv_addr.sin_addr.s_addr,server->h_length);

    serv_addr.sin_port = htons(portno);
    if(connect(sockfd, &serv_addr, sizeof(serv_addr)) < 0)
    {
        error("ERROR connecting");
    }
    printf("Please enter the message:");
    bzero(buffer, 256);
    fgets(buffer, 255, stdin);
    n = write(sockfd, buffer, strlen(buffer));
    if (n <0)
    {
       error("ERROR reading from socket");
    }
    printf("%s\n", buffer);
    return 0;
}

当我运行此程序时，我收到了错误

ERROR opening socket: Success

然后程序什么都不做。另外，我收到了警告

 warning: passing argument 2 of ‘connect’ from incompatible pointer type [enabled by default]
 if(connect(sockfd,&serv_addr, sizeof(serv_addr)) < 0)
 ^
In file included from Clientown.c:3:0:
/usr/include/sys/socket.h:138:12: note: expected ‘const struct sockaddr *’ but argument is of type ‘struct sockaddr_in *’
extern int connect (int __fd, __CONST_SOCKADDR_ARG __addr, socklen_t __len);

这是怎么回事？我是c的新手，我真的坚持使用这段代码。连接有问题吗？警告是否重要？有人可以帮我解决吗？感谢。

Answer 1

你有一个分号，其中不应该有分号：

if (sockfd < 0);
{
    error("ERROR opening socket");
}

因此，即使if条件失败，也始终会调用error()函数。你想要：

if (sockfd < 0)
{
    error("ERROR opening socket");
}

Answer 2

Penguin Brian already answered所述问题，但我想指出，由于提问者正在使用POSIX / BSD套接字，他们也应该使用正确的接口，即getaddrinfo()。它处理名称和端口/服务转换，并支持IPv4和IPv6（当然，如果需要，允许调用者限制为任何一个）。

man 3 getaddrinfo手册页确实包含两个示例程序（客户端和服务器类型），但这里是我的，为了说明，有足够的错误检查：

#define  _POSIX_C_SOURCE 200809L
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netdb.h>
#include <string.h>
#include <stdio.h>
#include <errno.h>

int main(int argc, char *argv[])
{
    struct addrinfo hints, *list, *curr;
    const char     *host, *port;
    int             sockfd, result;

    if (argc != 3 || !strcmp(argv[1], "-h") || !strcmp(argv[1], "--help")) {
        printf("\n");
        printf("Usage: %s [ -h | --help ]\n", argv[0]);
        printf("       %s host port\n", argv[0]);
        printf("\n");
        return EXIT_FAILURE;
    }

    host = argv[1];
    port = argv[2];

    /* "" or "-" host refers to loopback. */
    if (!host[0] || (host[0] == '-' && !host[1])) 
        host = NULL;

    memset(&hints, 0, sizeof hints);
    hints.ai_family = AF_UNSPEC;        /* IPv4 or IPv6 */
    hints.ai_socktype = SOCK_STREAM;    /* TCP */
    hints.ai_protocol = 0;
    hints.ai_flags = AI_ADDRCONFIG;

    list = NULL;
    result = getaddrinfo(host, port, &hints, &list);
    if (result) {
        fprintf(stderr, "%s.\n", gai_strerror(result));
        return EXIT_FAILURE;
    }

    for (curr = list; curr; curr = curr->ai_next) {

        sockfd = socket(curr->ai_family, curr->ai_socktype, curr->ai_protocol);
        if (sockfd == -1)
            continue;

        if (connect(sockfd, curr->ai_addr, curr->ai_addrlen) != -1)
            break;

        close(sockfd);
        sockfd = -1;
    }

    if (sockfd == -1) {
        freeaddrinfo(list);
        fprintf(stderr, "Cannot connect.\n");
        return EXIT_FAILURE;
    }

    printf("Descriptor %d is now connected. Disconnecting.\n", sockfd);
    fflush(stdout);

    if (close(sockfd)) {
        fprintf(stderr, "Error closing socket: %s.\n", strerror(errno));
        return EXIT_FAILURE;
    }

    printf("Done.\n");
    return EXIT_SUCCESS;
}

请注意，上述代码不会记录或报告来自connect()调用的错误消息，因为getaddrinfo()界面提供了多种替代方案，而且我不确定哪个在典型案例中产生的错误是适当的报告。（请记住，即使在成功的连接情况下，循环中的一个或多个connect()调用可能已失败，但没有发生任何实际的“错误”。）getaddrinfo()调用本身将失败，如果主机（节点）或端口（服务）部分无效。

在我自己的应用程序中，我倾向于使用错误cookie（每个错误一位）来记录connect()调用中来自getaddrinfo()调用的一个结果列表的所有错误。我测试的错误是EADDRINUSE（本地端口已在使用中），ECONNREFUSED（连接被拒绝），ENETUNREACH（网络无法访问），EACCES（权限被拒绝），{ {1}}（所有临时端口已在使用中），EADDRNOTAVAIL（不支持地址系列（IPv4 / IPv6）），以及所有其他错误的全能端口。

如果EAFNOSUPPORT个调用都没有成功，我会检查cookie是否发生了错误 - 作为无序集;即，忽略它们发生的订单，并将第一个匹配报告给上面的列表。任何措施都不是万无一失的，更像是尝试有用的启发式方法，而且我怀疑一般的错误信息（“无法连接”或类似 - 毕竟这是浏览器所做的）通常同样有用大多数用户。

Answer 3

关于第二个问题：您传递指向特定 Internet 套接字地址结构的指针，但connect需要通用套接字地址结构，因此只需将一种类型的指针强制转换为另一种：

connect(sockfd, (const struct sockaddr *)&serv_addr, sizeof(serv_addr)

这个警告实际上几乎无关紧要，因为现代架构上的所有指针类型都具有相同的大小。

ERROR开口插座：成功

3 个答案: