のmemcpy（）修正クライアントソケットファイルディスクリプタクライアントとの間でserver.I構造を送信しようと

Question

ので、イムは、両側に構造を定義した：

struct msg { 
    char name[50]; 
    char time[50]; 
    int len; 
    char buf[200]; 
} 

問題がserver.cで発生します。私はすでにこの行で問題を見つけた：

struct msg s1; 
char buffer_input[1024]={0};// the buffer stream for sending 
...// omitted all sorts of initializations cuz they all passed my debug and tests 
memset(buffer_input,0,sizeof(buffer_input)); 
memcpy(buffer_input,&s1,sizeof(s1)); 
/* Originally, the send() call read: */ 
/* ssize_t size=send(client_sock,(struct sockaddr*)&client_sock_address,&addrlen); */ 
ssize_t size=send(client_sock,buffer_input,sizeof(buffer_input),0); 
if(size<0) perror("send()"); 

は、標準出力に、私が持っている：

send() : Socket operations on non-socket object 

をので、私はmemcpy(buffer_input,&s1,sizeof(s1))が原因である可能性がありますことを予感しています。だから、コードを次のように変更しました。

memset(buffer_input,0,sizeof(buffer_input)); 
strcpy(buffer_input,"example"); 

send（）は完全に機能し、正しい形式でメッセージを受け取りました。

一連のデバッグ後、memcpy(...)の後に、クライアントソケットのファイル記述子が0から8に変わったことに気付きました。

ライン

ssize_t size=send(client_sock,(struct sockaddr*)&client_sock_address,&addrlen); 

が

ssize_t size=send(client_sock, buffer_input, 1024); 

それともbuffer_inputを取り除くと持っを読む必要がありますので、私は..

Answer 1

をソケットのファイルディスクリプタを変更memcpyどのケースでできた、不思議

ssize_t size=send(client_sock, &s1, sizeof(s1)); 

（！あなたも、同様のmemsetのものを削除することができます）

EDIT

いくつかのより多くのコーヒーの後は、ここではより良いソリューションです：

/* 50 + 100 + 100 + 4 */ 
#define BUFFER_LENGTH 254 

... 

unsigned char buffer[BUFFER_LENGTH]; 

... 

memcpy(buffer, s1.name, 50); 
memcpy(buffer + 50, s1.time, 50); 
uint32_t net_len = htonl(s1.len); 
memcpy(buffer + 100, &net_len, 4); 
memcpy(buffer + 104, ss1.buf, 100); 

... 

ssize_t size=send(client_sock, buffer, BUFFER_LEN); 

Answer 2

memcpyはでは、いけません任意の状況、ファイル記述子を変更します。あなたはそれを我々が持っている情報（s1あなたはbufferを可能に1000年よりもはるかに少ない約300バイト、でなければなりません）与えられた可能性は低いと思われる不正なパラメータを与える場合

何それこと行う上書きメモリであることがあります。

あなたのバッファがあなたの構造にとって小さすぎる可能性は、そのコードが本当にあなたが持っているものであれば（文字列とシーケンスの両方で、関数の入力や終了を含めて）低いと思われます。

私が示唆できるのは、デバッグコードです。

あなたがmemcpy、出力は次の値を呼び出す前に、行う必要があります。

• sizeof(buffer)。
• &buffer。
• sizeof (s1)。
• &s1。
• client_sock。
• &client_sock。コールの後に再びそれらを

次に出力。それに基づいて、我々はmemcpy呼び出しによって破損を検出する（または破棄する可能性がある）ことができるはずです。