2

tcpサーバーからtcpクライアントにデータを送信しようとすると、「不正なファイル記述子」エラーが発生し続けます。これはソケットに関してどういう意味ですか?私はしばらくこれに取り組んでいますが、コードに何が問題があるのか​​わかりません。基本的に2日前に使用していたのと同じコードで、そのコードは正常に機能しました。ソケットを介して送信しようとしたときに、ファイル記述子が正しくない一般的な原因と、それらをチェック/修正する方法を誰かに教えてもらいたいと思っていました。どんな助けでも大歓迎です。役立つ場合に備えて、以下にいくつかのコードを投稿します。

/*Waits to connect a client. Returns true if successful*/
bool TcpServer::launchServer() {
int status;

struct addrinfo hints;
struct addrinfo *servinfo;  //will point to the results

//store the connecting address and size
struct sockaddr_storage their_addr;
socklen_t their_addr_size;


memset(&hints, 0, sizeof hints); //make sure the struct is empty
hints.ai_family = AF_INET;  //ipv4
hints.ai_socktype = SOCK_STREAM; //tcp

//get server info, put into servinfo
if ((status = getaddrinfo("192.168.2.3", port, &hints, &servinfo)) != 0) {
    printf("\ngetaddrinfo error: %m", errno);
    return false;
}

//make socket
fd = socket(servinfo->ai_family, servinfo->ai_socktype, servinfo->ai_protocol);
if (fd < 0) {
    printf("\nserver socket failure %m", errno);
    return false;
}

//allow reuse of port
int yes=1;
if (setsockopt(fd,SOL_SOCKET,SO_REUSEADDR,(char*) &yes,sizeof(int)) == -1) {
    perror("setsockopt");
    return false;
}

//bind
if(bind (fd, servinfo->ai_addr, servinfo->ai_addrlen) < 0) {
    printf("\nBind error %m", errno);
    return false;
}

//free up space
freeaddrinfo(servinfo);

//listen
if(listen(fd, 5) < 0) {
    printf("\nListen error %m", errno);
    return false;
}
their_addr_size = sizeof(their_addr);


//accept
comm_fd = accept(fd, (struct sockaddr*)&their_addr, &their_addr_size);
if( comm_fd < 0) {
    printf("\nAccept error %m", errno);
    return false;
}

return true;
}   //END LAUNCHSERVER






void TcpServer::communicate() {


fd_set read_flags,write_flags; // the flag sets to be used
struct timeval waitd = {10, 0};          // the max wait time for an event
int sel;        // holds return value for select();
int numRead;    //holds return value for read()
int numSent;    //holds return value for send()
char in[255];   //in buffer
char out[255];  //out buffer

//clear buffersz
memset(&in, 0, 255);
memset(&out, 0, 255);


while(!done) {
    FD_ZERO(&read_flags);
    FD_ZERO(&write_flags);
    FD_SET(comm_fd, &read_flags);
    FD_SET(comm_fd, &write_flags);
    FD_SET(STDIN_FILENO, &read_flags);
    FD_SET(STDIN_FILENO, &write_flags);

    //call select
    sel = select(comm_fd+1, &read_flags, &write_flags, (fd_set*)0, &waitd);

    //if an error with select
    if(sel < 0)
        continue;

    //if socket ready for reading
    if(FD_ISSET(comm_fd, &read_flags)) {

        //clear set
        FD_CLR(comm_fd, &read_flags);

        memset(&in, 0, 255);

        numRead = recv(comm_fd, in, 255, 0);
        //if an error, exit
        if(numRead < 0) {
            printf("\nError reading %m", errno);
            myAgent->getRobot()->pauseSensorStream();
            done = true;
        }   //end if error
        //if connection closed, exit
        else if(numRead == 0) {
            printf("\nClosing socket");
            close(comm_fd);
            done = true;
        }   //end if connection closed
        //if message, call getsendback
        else if(in[0] != '\0') {
            //std::cout<<"\nClient: "<<in;
            getSendBack(in);
        }   //end if message
    }   //end if ready for read


    //if stdin is ready for reading
    if(FD_ISSET(STDIN_FILENO, &read_flags))
        fgets(out, 255, stdin);


    //if socket ready for writing
    if(FD_ISSET(comm_fd, &write_flags)) {

        //printf("\nSocket ready for write");
        FD_CLR(comm_fd, &write_flags);

        //check validity by checking for a digit
        if(isdigit(out[0])) {

            //create message to send
            std::stringstream tosend;
            tosend<<"@ "<<out;
            //std::cout<<"\ntosend: "<<tosend.str();

            //send
            //********ERROR HAPPENS HERE PRINTS OUT MESSAGE BELOW******
            numSent = send(comm_fd, tosend.str().c_str(), tosend.str().length(), 0);
        }   //end if valid message
        //if error, exit
        if(numSent < 0) {
            printf("\nError sending %m", errno);
            done = true;
        }   //end if error
        //wait for message to get there, then clear
        usleep(5000);
        memset(&out, 0, 255);
    }   //end if
}   //end while
}   //END COMMUNICATE

クライアントコードは基本的に同じです。

4

2 に答える 2

3

errno が EBADF の場合、プログラムは「Bad file descriptor」を出力します。send の man ページから:

EBADF = 無効な記述子が指定されました。

send() 呼び出しの前にソケットが閉じられていることは確かです。プログラムは「接続が閉じられた」ブランチの後に「書き込み準備完了」ブランチに進むことができるため、これが発生する可能性があります。

次のことを試してください。

else if(numRead == 0) {
    printf("\nClosing socket");
    close(comm_fd);
    break;
} 

それ以外の:

    else if(numRead == 0) {
        printf("\nClosing socket");
        close(comm_fd);
        done = true;
    }
于 2011-08-19T13:41:14.547 に答える
0

あなたはあなた自身の質問に答えました。numSent と numRead を明示的に初期化しないと、ガベージが発生します。これは、numSent の負の数になる可能性があり、out[] 配列に数字がない場合はエラーになります。

于 2011-08-25T17:26:48.463 に答える