Question

我是socket编程的新手......我尝试过这个服务器端程序

#define BUFLEN 512
#define MYPORT 3456
void errorp(char* msg)
{
        perror(msg);
        exit(1);
}

int main()
{
        struct sockaddr_in server, client;
        int sock;
        int slen = sizeof(server);
        int clen = sizeof(client);
        char *recvbuf, senbuf[BUFLEN] = {'h','e','l','l','o'};

        if((sock = socket(AF_INET, SOCK_DGRAM, 0) == -1))
                errorp("Socket creation failed");


        printf("To the client: %s, %s", senbuf, " World");
        bzero(&server, sizeof(server));

        server.sin_family = AF_INET;
        server.sin_port = MYPORT;
        server.sin_addr.s_addr = inet_addr("127.0.0.1");
        if(bind(sock, (struct sockaddr*)&server, slen)==-1)
                errorp("Socket Bind Failed");
        if(recvfrom(sock, recvbuf, sizeof(recvbuf), 0, (struct sockaddr*) &client, &clen) == -1)
                errorp("recv from error");
        printf("From the client: %s", recvbuf);

        if(sendto(sock, senbuf, sizeof(senbuf), 0, (struct sockaddr*) &client, sizeof(client)) == -1)
                errorp("Error in sending");

        printf("To the client: %s", senbuf);
        close(sock);
        return 0;
}

没有编译错误，但输出是

Socket Bind Failed: Socket operation on non-socket
To the client: hello,  World

请帮我弄清楚错误在哪里？并帮助摆脱它

Answer 1

错误信息说明了一切：套接字不是（有效）套接字。

这应该让你看看创建套接字的代码：

if((sock = socket(AF_INET, SOCK_DGRAM, 0) == -1))

上面的代码将调用socket()的结果与-1进行比较，然后将比较结果分配给sock。所以它是0或1。调用socket()的结果将丢失。

代码应如下所示：

if ((sock = socket(AF_INET, SOCK_DGRAM, 0)) == -1)

，因为==比=更紧密。

BTW，使用Yoda-Conditition可以避免这种错误＆＃34;

if (-1 == (sock = socket(AF_INET, SOCK_DGRAM, 0)))

同样，至少clen应该是socklen_t类型，因为它的地址被传递，要写入一个值，如果预期的socklen_t的大小将会失败，这将失败与int（显示的代码通过）不同。

Answer 2

if((sock = socket(AF_INET, SOCK_DGRAM, 0) == -1))
//         \__________________________________/

您的括号位于错误的位置。它将sock设置为true / false值，因为==比=“更具约束力”。它应该是：

if ((sock = socket(AF_INET, SOCK_DGRAM, 0)) == -1)
//  \_____________________________________/

将sock设置为socket()的返回值，然后将其与-1进行比较。

你也没有recvbuf的后备存储，这意味着你的recvfrom()一旦开始工作，几乎肯定会做坏事。

C中非套接字错误的套接字操作

2 个答案: