C言語 文字列配列の初期化

Question

現在、C言語でソケットライブラリやselectシステムコールを用いて複数人チャットのシステムを作っています。

チャットに参加するクライアントは以下のような構造体で管理しており、ソケットディスクリプタ・名前・チャットに送信するメッセージを保存しています。

typedef struct{
    int  sock;
    char name[NAMELENGTH];
    char msg[MESSAGEMAXLENGTH];
} client_info;

そして以下のようにsendシステムコールやrecvシステムコール（自作ラッパー関数を使っています）を用いて、メッセージの送受信を行っています。

// チャットサーバのメインループ
void chat_loop()
{
    int msg_sender_client; // メッセージを送信したclient_idを保存する
    int client_id;
    static char msgprompt[]="Input your message ↓: \n";


    for(;;){
        for(client_id=0; client_id<N_client; client_id++){
            Send(Client[client_id].sock, msgprompt, strlen(msgprompt), 0);
        }
        // メッセージを送信したclient_id受け取る
        msg_sender_client = receive_message();

        // 送信されたメッセージとユーザ名を表示
        send_message(msg_sender_client);
    }
}

// クライアントからのメッセージを受信して、そのclient_idを返す
static int receive_message()
{
    fd_set mask, readfds;
    int client_id, val;
    int strsize;

    /* ビットマスクの準備 */
    // クライアントのソケット番号 Client[client_id].sock を監視するように設定
    FD_ZERO(&mask);
    for(client_id=0; client_id<N_client; client_id++){
        FD_SET(Client[client_id].sock, &mask);
    }

    /* 受信データの有無をチェック */
    readfds = mask;

    // データを送ってきたクライアントを知らべる
    select( Max_sd+1, &readfds, NULL, NULL, NULL );

    for( client_id=0; client_id<N_client; client_id++ ){
        if( FD_ISSET(Client[client_id].sock, &readfds) ){
            // ユーザのメッセージをBufで受け取る
            strsize = Recv(Client[client_id].sock , Buf, BUFLEN-1,0);
            Buf[strsize]='\0';

            // Bufで受信したメッセージをユーザ構造体に保存する
            if(strsize <= MESSAGEMAXLENGTH){
                strncpy(Client[client_id].msg, Buf, strsize);
            }else{
                strncpy(Client[client_id].msg, Buf, MESSAGEMAXLENGTH);
            }
            val = client_id;
            break;
        }
    }
    return val;
}

// あるクライアントから送信されたメッセージをクライアント全員に送信する
static void send_message( int msg_sender_client )
{
    int client_id;
    char *msg = chop_nl(Client[msg_sender_client].msg);
    int len;

    // ユーザごとにメッセージの色を変える
    switch (msg_sender_client) {
        case 0:
            len = snprintf(Buf, BUFLEN, "\x1b[34m <message from Mr.%s> %s \033[m\n",
                           Client[msg_sender_client].name, msg);
            break;
        case 1:
            len = snprintf(Buf, BUFLEN, "\x1b[35m <message from Mr.%s> %s \033[m\x1b[0m\n",
                           Client[msg_sender_client].name, msg);
            break;
        case 2:
            len = snprintf(Buf, BUFLEN, "\x1b[36m <message from Mr.%s> %s \033[m\x1b[0m\n",
                           Client[msg_sender_client].name, msg);
            break;
        default:
            len = snprintf(Buf, BUFLEN, "\x1b[36m <message from Mr.%s> %s \033[m\x1b[0m\n",
                           Client[msg_sender_client].name, msg);
            break;
    }

    // メッセージを送信
    for(client_id=0; client_id<N_client; client_id++){
        Send(Client[client_id].sock, Buf, len,0);
    }

    // 今回以前のユーザ構造体のメッセージは初期化しておく
    // TODO:初期化できてないみたい
    memset(Client[msg_sender_client].msg, '\0', sizeof(Client[msg_sender_client].msg));
//    Client[msg_sender_client].msg = "";
}

// 文字列の改行を取り除く
static char *chop_nl(char *s)
{
    int len;
    len = strlen(s);
    if( s[len-1] == '\n' ){
        s[len-1] = '\0';
    }
    return(s);
}

static void send_message( int msg_sender_client )内でメッセージの送信を行った後は、 memset(Client[msg_sender_client].msg, '\0', sizeof(Client[msg_sender_client].msg));としてユーザのメッセージを初期化しているのですが、どうやらそれがうまく行っておらず、実行時に長めのメッセージの送信を行うとその後にそのメッセージのキャッシュ？？が残ってしまいます。

どうすればメッセージを初期化することができるのかわからないため、教えていただきたいです。
よろしくお願い致します。

Comments

[SHOMI]

sendで送った内容が一度のrecvですべて受信されるとは限りません。
分割されて受信されたのではありませんか？
送った長めのメッセージと、一度目に受信されたメッセージの文字数はあっていますか？

[SHOMI]

一度目と二度目でmsg_sender_clientの値は同じなのでしょうか。

[tedmosby]

一度目と二度目で同じユーザ（同じmsg_sender_client）で、長いメッセージを送信した後ほかのユーザでメッセージを送信すると上記のようにはならず、キーボード入力した内容のみが送信されます。

メッセージの長さは#define MESSAGEMAXLENGTH 140として決定していて、static int receive_message()内でその長さまでのメッセージならそのまま受信して、それより長いメッセージならMESSAGEMAXLENGTHまで受信するようにしています。

// Bufで受信したメッセージをユーザ構造体に保存する
if(strsize <= MESSAGEMAXLENGTH){
strncpy(Client[client_id].msg, Buf, strsize);
}else{
strncpy(Client[client_id].msg, Buf, MESSAGEMAXLENGTH);
}
val = client_id;
break;

[tedmosby]

メッセージの送信では、static void send_message( int msg_sender_client )内で、
// メッセージを送信
for(client_id=0; client_id<N_client; client_id++){
Send(Client[client_id].sock, Buf, len,0);
}

としています。
BUFは
#define BUFLEN 500 /* 通信バッファサイズ */
static char Buf[BUFLEN]; /* 通信用バッファ */
となっています。

BUFLENとMESSAGEMAXLENGTHが異なることが問題なのでしょうか？？

[SHOMI]

受信長がMESSAGEMAXLENGTH以上の場合msgに終端の'\0'が付きませんよ。
それだけが問題かはわかりませんが。

Answer 1

既にコメントが付いているみたいですが、、、

char msg[MESSAGEMAXLENGTH];

に対し、

strncpy(Client[client_id].msg, Buf, MESSAGEMAXLENGTH);

は、文字列の最後に '\0'が付かないので、ダメです。(もう一つも strsize = MESSAGEMAXLENGTH の時、同様にダメ)
char msg[MESSAGEMAXLENGTH + 1]; のようにして、msg[MESSAGEMAXLENGTH] ='\0' としましょう。 strncpy()は、'\0'を付けない時があります。

Answer 2

Clientという配列が定義されていないようですが、そこらへんはどうなってるんでしょうか。

Comments

[tedmosby]

省略させていただきましたが、static client_info *Client; /* クライアントの情報 */
で定義しています。

[y_waiwai]

肝心なところを省略されてはどもなりませんねー

[y_waiwai]

Clientがどういう初期化されてるのか不明ですが、msg_sender_clientがClientの範囲内である、というのをしっかり確認しよーね。

＃おそらく範囲外アクセスでおかしくなってるものとえすぱー