UNIX TCP/IP:读取:传输端点未连接读取:传输端点未连接

Question

我正在尝试使用以下程序来显示从端口收到的消息8888.我编译了以下代码,没有任何错误和警告.

运行后,我用broswer打开 127.0.0.1:8888

然后,控制台显示:

read: Transport endpoint is not connected
read: Transport endpoint is not connected

我调试它,但我找不到原因.

平台

Linux内核3.x Ubuntu 64位

码

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

int main(int argc, char *argv[])
{
    int sock;
    char buf[BUFSIZ+1];

    buf[BUFSIZ] = '\0';
    uint16_t port = (uint16_t)atoi("8888");
    struct sockaddr_in ser;
    memset(&ser, 0, sizeof(ser));
    ser.sin_port = htons(port);
    ser.sin_addr.s_addr = htonl(INADDR_ANY);
    ser.sin_family = AF_INET;

    sock = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
    if(sock < 0)
    {
        perror("socket");
        return -7;
    }

    /*Bind*/
    if (bind(sock, (struct sockaddr *)&ser, sizeof(ser)) < 0)
        return -2;

    /*listen*/
    if (listen(sock, 5) < 0)
        return -3;

    /*Accpet*/
    struct sockaddr_in cliAddr;
    socklen_t cliLen = sizeof(cliAddr);
    if (accept(sock, (struct sockaddr*)&cliAddr, &cliLen) < 0)
    {
        perror("accept");
        exit(1);
    }
    int read_len = 0;
    int i = 0;

    /*read and print*/
    while(1)
    {
        read_len = read(sock, buf, BUFSIZ);
        if(read_len < 0)
        {
            perror("read");
            break;
        }
        else
        {
            /*print buf*/
            while(i++ < read_len)
                putchar(buf[i-1]);
            putchar('\n');
        }
        if(read_len != BUFSIZ)
            break;
    }
    return 0;
}

如果您在我的代码中发现任何不良习惯,请告诉我.

Answer 1

你正试图读错插座. accept()返回一个新的套接字,你应该从新的套接字读取数据并将数据写入.

你的代码应该做更像这样的事情:

int readSocket = accept(sock ...);
if (readSocket == -1)
{
    // error
}
else
{
    // set up stuff and while loop
    read_len = read(readSocket....); // << Note which socket is being read

    // other stuff
}