永遠にルーピング？

Question

ちょっと、私の構造体のポインタとして "次"を反復しようとしている間に、構造体への偽のポインタを作成しようとしても、それは何の理由もなく永遠にループします。

私のコード：

void 
command_login(conn,msg) 
    void *conn; 
    const char *msg; 
{ 
    const char *u,*p; 
    char *home; 
    ConnectionQueue* q; 
    DBResult *result; 
    DBResult *tmp; 
    bool got_pass,got_user,admin; 
    if (!conn) 
    return; 

    /* initalize */ 
    q = (ConnectionQueue *)conn; 
    got_pass = false; got_user = false; admin = false; 
    result = NULL; 
    tmp = NULL; 
    if (msg == NULL || *msg == '\0'){ 
    send_socket(q->conn,"LOGIN: Invalid user and password\n"); 
    return; 
    } 

    (void)strtok((char *)msg," "); 
    u = strtok((char *)NULL, " "); 
    p = strtok((char *)NULL, " "); 
    if (!u || !p){ 
    send_socket(q->conn,"LOGIN: Invalid user or password\n"); 
    return; 
    } 

    printf("command_login(): u = %s, p = %s\n",u,p); 
    tmp = store_query("SELECT * FROM `users`"); 
    if (!tmp){ 
    /* should never happen */ 
    send_socket(q->conn,"LOGIN: Failed to fetch results\n"); 
    return; 
    } 

    printf("command_login(): Checking user and password...\n"); 
    result = tmp; 
    while (result != NULL){ 
    if (!got_user){ 
     if (strcmp(result->field_name,"user") == 0 && strcmp(result->field_value,u) == 0) 
     got_user = true; 
    }else if (!got_pass){ 
     if (strcmp(result->field_name,"pass") == 0){ 
      if (strcmp(result->field_value,transform_password(p)) == 0) 
      got_pass = true; 
     } 
    }else if (!admin){ 
     if (strcmp(result->field_name,"admin") == 0){ 
     admin = boolean_string(result->field_value); 
     } 
    }else{ 
     break; 
    } 
    result = result->next; 
    } 

    free_result(result); 
    printf("command_login(): Checking got user and got password\n"); 
    if (!got_user || !got_pass){ 
    send_socket(q->conn,"LOGIN: Invalid user or password\n"); 
    return; 
    } 

    printf("command_login(): Login successfully\n"); 
    send_socket(q->conn,"LOGIN: Success\n"); 
    q->conn->state &= ~S_NEED_LOGIN; 
    q->admin = admin; 
#ifndef _WIN32 
    home = getenv("HOME"); 
#else 
    home = getenv("HOMEPATH"); 
#endif 
    if (!home) 
    return; 

    if (!chdir(home)){ 
    send_socket(q->conn,"LOGIN: Failed to change to user home directory %s: %s\n",home,strerror(errno)); 
    close_connection(q->conn); 
    return; 
    } 
    send_socket(q->conn,"CWD: %s\n",home); 
} 

それはここに永遠にループ：

while (result != NULL){ 
    if (!got_user){ 
     if (strcmp(result->field_name,"user") == 0 && strcmp(result->field_value,u) == 0) 
     got_user = true; 
    }else if (!got_pass){ 
     if (strcmp(result->field_name,"pass") == 0){ 
      if (strcmp(result->field_value,transform_password(p)) == 0) 
      got_pass = true; 
     } 
    }else if (!admin){ 
     if (strcmp(result->field_name,"admin") == 0){ 
     admin = boolean_string(result->field_value); 
     } 
    }else{ 
     break; 
    } 
    result = result->next; 
    } 

任意のアイデア？ありがとう

Answer 1

新しいDBResult構造体を作成するときは、nextポインタをNULLに設定しますか？ mallocで構造を作成すると、resultのテストは終了しません。

DBResultの定義とstore_queryという定義を知らなくても、もっと知ることは本当に難しいです。 store_queryは何を返しますか？

リストを作成しているとします。 store_queryでは、nextはret〜tmpに設定されていません。私はそれがあなたがしたいことだと思います。それが正しくないのであなたのリスト構築を注意深く見てください。

DBResultの画像を描画するのに役立つかもしれません。

Answer 2

わかりましたので、何をやっていると、あなたは一つの要素を持つ円形のリンクリストを作っている

ret->next = tmp; 
ret = tmp; 
ret->next = tmp; // = ret 

です。必要なのは、追加するたびに新しい要素を割り当てることです。最後にすべてを解放することを忘れないでください。

編集

はパターンが `addto_dbresult`何をするのか、この

DBResult* head = NULL; 
DBResult* row = NULL; 

while(have_rows()) { 
    row = malloc(sizeof(DBResult)); 
    // load row into row 

    row->next = head; 
    head = row; 
} 

return head; 

// later when you're all done 
while(head) { 
    DBResult* element = head; 
    head = head->next; 
    free(element); 
}