-1

ポート 457 でクライアントからサーバーに文字列を送信できるプロトコル (TCP の上に構築) を作成しようとしています。

サーバー.c

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

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

int main(int argc, char *argv[])
{
    int sockfd, newsockfd, portno;
    socklen_t clilen;
    char buffer[256];
    struct sockaddr_in serv_addr, cli_addr;
    int n;
    if (argc < 2) {
        fprintf(stderr,"ERROR, no port provided\n");
        exit(1);
    }
    sockfd = socket(AF_INET, SOCK_STREAM, 0);
    if (sockfd < 0) 
        error("ERROR opening socket");
    bzero((char *) &serv_addr, sizeof(serv_addr));
    portno = atoi(argv[1]);
    serv_addr.sin_family = AF_INET;
    serv_addr.sin_addr.s_addr = INADDR_ANY;
    serv_addr.sin_port = htons(portno);
    if (bind(sockfd, (struct sockaddr *) &serv_addr,
             sizeof(serv_addr)) < 0) 
        error("ERROR on binding");
    listen(sockfd,5);
    clilen = sizeof(cli_addr);
    newsockfd = accept(sockfd, 
                       (struct sockaddr *) &cli_addr, 
                       &clilen);
    if (newsockfd < 0) 
        error("ERROR on accept");
    bzero(buffer,256);
    n = read(newsockfd,buffer,255);
    if (n < 0) error("ERROR reading from socket");
    printf("Here is the message: %s\n",buffer);
    n = write(newsockfd,"U got your messaze",18);
    if (n < 0) error("ERROR writing to socket");
    close(newsockfd);
    close(sockfd);
    return 0; 
}

Client.c

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

void error(const 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]);
        exit(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,(struct sockaddr *) &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 writing to socket");
    bzero(buffer,256);
    n = read(sockfd,buffer,255);
    if (n < 0) 
        error("ERROR reading from socket");
    printf("%s\n",buffer);
    close(sockfd);
    return 0;
}

プロトコルを正しく設定していないように感じます。私ですか?

4

2 に答える 2

0

間違ったタイプがあります:


htonsuint16_tパラメータとして期待

htons((uint16_t)portno);

read()write()期待するssize_t

ssize_t n;

bzero非推奨です。使用してください

memset(buffer, 0, sizeof(buffer));

bcopy非推奨です。使用してください

memmove(server->h_addr, &serv_addr.sin_addr.s_addr, server->h_length);

使用buffer時にNUL 終了read()

/* bzero(buffer,256); Not needed */
n = read(newsockfd,buffer,255);
if (n < 0) error("ERROR reading from socket");
buffer[n]= '\0'; /* here */
printf("Here is the message: %s\n",buffer);

send()また、最近のプログラムではandrecv()の代わりにread()andを使用していることに注意してください。write()


最後に、マジック ナンバーを使用しないでください

fgets(buffer, 255, stdin);

代わりは:

fgets(buffer, sizeof(buffer), stdin); /* 256 */

なぜ255?fgets() 関数は、n-1 バイトが読み取られるまで、stream から s が指す配列にバイトを読み取るため、256 が正しいです。

于 2013-07-25T16:55:22.893 に答える
0

最初にプロトコルの仕様を作成する必要があります。そして、それをレビューした後、仕様を実装します。あなたは基本的に、あなたのプロトコルをリバースエンジニアリングするよう私たちに求めています. これは、プロトコルを作成する逆方向のアプローチです。実装を開始する前に、プログラムが何をする必要があるかについて明確な意図を持っている必要があります。

サーバー プログラムは、クライアントからの入力が 255 バイトを超えて受け入れられないという点で、限られた種類の ECHO サーバーとして機能します。サーバーが読み取ることができたものはすべてコンソールに記録され、メッセージU got your messazeが応答としてクライアントに送信されます。

あなたが対処したいかもしれないいくつかの問題:

  • 可能性は低いread()ですが、クライアントが 256 バイト未満のデータを送信したとしても、クライアントが送信したデータよりも少ないデータで呼び出しが返される可能性があります。たとえば、クライアントaaaaaaaaaaが一度に 1 バイトを送信する場合、サーバーは最初の のみを見て、aそれが完全なメッセージであると想定する可能性があります。

  • すでに閉じられている接続への書き込みに対して予防策を講じません。これにより が生成されSIGPIPE、プログラムが予期せず終了する可能性があります。

  • 通常、シグナルはシステム コールを中断することがあります。この状態を検出し、それが発生した場合はシステム コールを再起動する必要があります。

于 2013-07-25T16:45:30.430 に答える