c - C での文字列の比較 - strcmp

Question

Cで文字列を比較するのに問題があります（私はかなり慣れていません）。このサーバーアプリケーションには、クライアントからのデータの受け入れを待機しているソケットがあります。プログラムのこの特定の部分では、クライアントから受信したデータに基づいて MySQL クエリを実行できるようにしたいと考えています。簡単な登録手順を開始するために、受信したデータに「newuser」の値が含まれていることを知りたいです。strcmp は正の 1 の値を返していますが、値は等しいはずなので、0 を取得する必要があると思います。

ソースコード：

//setup socket
//loop and select structure to handle multiple connections

if ((nbytes = recv(i, buf, sizeof buf, 0)) <= 0) {
// got error or connection closed by client
    if (nbytes == 0) {
        // connection closed
        printf("selectserver: socket %d hung up\n", i);
    } else {
        perror("recv");
    }
    close(i); // bye!
    FD_CLR(i, &master); // remove from master set
} else {

    char check[] = "newuser";
    char fromUser[sizeof check];

    strncpy(fromUser,buf, sizeof check);
    printf("length of fromUser: %d\n", sizeof fromUser);
    printf("length of check: %d\n", sizeof check);
    printf("message from user: %s\n", fromUser);
    printf("check = %s \n", check);
    int diff = strcmp(fromUser, check);
    printf("compare fromUser to check: %d\n", diff);
    if ( strcmp(fromUser, check) == 0) {
        printf("aha! new user");
    }

出力：

length of fromUser: 8
length of check: 8
newuser from user: newuser
check = newuser 
compare fromUser to check:

着信バッファを正しく処理していないか、誤ってバッファをコピーしているように感じます。

score 6 · Accepted Answer

strncpyコピーするのはせいぜい - この場合 -sizeofチェックバイトです。nul バイトがその範囲にない場合、コピーされません。おそらく、「newuser blah blah」のような長い文の一部として「newuser」という単語を取得しているので、その nul を自分で配置する必要があります。

strncpy(fromUser, buf, sizeof check);
fromUser[sizeof check - 1] = '\0';

またはstrlcpy、利用可能な場合はを使用します。

score 3 · Accepted Answer

質問で提供したサンプルコードは次のとおりです（デバッグコードは削除されています）：

//setup socket
//loop and select structure to handle multiple connections

if ((nbytes = recv(i, buf, sizeof buf, 0)) <= 0) {
    [... exception handling here ...]
} else {
    char check[] = "newuser";
    char fromUser[sizeof check];

    strncpy(fromUser,buf, sizeof check);
    if ( strcmp(fromUser, check) == 0) {
        printf("aha! new user");
    }

このコードは間違っています。受信したよりも多くのバイトを buf[] からコピーしている可能性があります。これにより、ガベージと比較することになります（偶然、「newuser」文字列と一致する可能性があります）。そして、他の人が言ったように、文字列の 1 つを NUL で終了しないために、2 つ目のバグがあります。

この場合、memcmp() を使用します。これは strcmp() に似ていますが、NUL で終了する文字列ではなく、長さパラメーターを取ります。

//setup socket
//loop and select structure to handle multiple connections

if ((nbytes = recv(i, buf, sizeof buf, 0)) <= 0) {
    [... exception handling here ...]
} else {
    static const char check[] = "newuser";
    const size_t check_len = sizeof(check) - 1; // exclude the NUL terminator
    if (nbytes >= check_len && memcmp(buf, check, check_len) == 0) {
        printf("aha! new user");
    }

PS 直接関係はありませんが、recv()はを返すこと-1で失敗する可能性がありerrno==EINTRます。これはエラー状態ではありません。再試行する必要があります。通常、これはめったに発生しないため、シグナルを使用する他のコードと統合し、突然コードがランダムに失敗するまで、人々はそれをチェックせずに逃げます。

ベースのアプリでは、select()ソケットをノンブロッキングに設定してからをチェックしerrno==EAGAIN、その場合はに戻る必要がありselect()ます。これは、TCP/IP スタックが破損したパケットを受信した場合に発生する可能性があります。TCP/IP スタックはパケットがあるselect()と認識しているため、読み取り可能であることがわかります。TCP/IP スタックがチェックサム計算を行い、それが必要であることを認識したのは、読み取りを試みたときだけです。データを捨てる。-1その後、ブロックする (悪い) か、非ブロックに設定されている場合はを返しerrno==EAGAINます。

score 2 · Accepted Answer

ここでの問題 (ここでの問題の 1 つ) は、fromUser (作成方法による) が null で終了していないことだと思います。

score 2 · Accepted Answer

fromUser の最後に '\0' 文字がありません:

...
strncpy(fromUser,buf, sizeof check);
fromUser[strlen(check)] = '\0';

score 1 · Accepted Answer

2 つの変更が必要です。

char fromUser[sizeof check] = {'\0'}; //Make all null characters
strncpy(fromUser,buf, sizeof check -1); //Last character is for null character.

score 0 · Accepted Answer

このコードはオフのようです:

if ((nbytes = recv(i, buf, sizeof buf, 0)) <= 0) 
{
 // your stuff
} 
else {
const char *pCheck = "newuser";
char *fromUser = new char[nbytes];
strncpy(fromUser, buff, nbytes);
fromUser[nbytes] = '\0';
if(strcmp(fromUser,check)==0)
 // blah

delete [] fromUser;
}

score -1 · Accepted Answer

-1

と置換する：

char check[] = "newuser\0";

于 2009-06-22T15:29:11.437 に答える

c - C での文字列の比較 - strcmp

7 に答える 7

Related

Reference