Code 101

ここで、あなたが gtk-gnutella のコードを触れるだけ C に馴染んでいるかどうかを確認するため、少しばかりのテストを行ってみたいと思います。私達がプロジェクトで経験したコード実装の落し穴からの出題となります。

テスト 1

作成者: Richard Eckart
難易度: 普通

問題

以下のコードはアップロードのヘッダからユーザ・エージェントを取り出します。GUI はユーザ・エージェントの文字列を表示するか、u->user_agent が NULL なら "..." を表示します。残念ながら、ある場所にバグがあるのでテストするユーザの GUI には常に "..."が表示されます。どこが誤っているか解かりますか?

 1: /*
 2:  * User-Agent を取り出します.
 3:  *
 4:  * X-Token: GTKG トークン
 5:  * User-Agent: 何でも
 6:  * Server: 何でも (User-Agent が無い場合でも)
 7:  */
 8:
 9: token = header_get(header, "X-Token");
10: user_agent = header_get(header, "User-Agent");
11:
12: /* Maybe they sent a Server: line, thinking they're a server? */
13: if (user_agent != NULL)
14:     user_agent = header_get(header, "Server");
15:
16: if (user_agent != NULL)
17:     faked = !version_check(user_agent, token, u->ip);
18:
19: if (u->user_agent == NULL && user_agent != NULL) {
20:     if (faked) {
21:         gchar *name = g_strdup_printf("!%s", user_agent);
22:
23:         u->user_agent = atom_str_get(name);
24:         g_free(name);
25:     } else
26:         u->user_agent = atom_str_get(user_agent);
27: }

テスト 2

作成者: Richard Eckart
難易度: 普通

問題

GTK1 サポートでコンパイルされた gtk-gnutella を実行中に以下のコードがメモリリークを含んでいるのに気付きました。コード中には新しいメモリを割り当てる場所が二箇所あります、18 および 19 行目です。どちらにメモリリークがあるか解かりますか?

 1: /*
 2: * on_popup_downloads_push_activate
 3: *
 4: * All selected downloads fallback to push
 5: */
 6: void on_popup_downloads_push_activate(
 7:    GtkMenuItem * menuitem, gpointer user_data)
 8: {
 9:     struct download *d;
10:     GList *node_list, *data_list = NULL;
11:     GtkCTree *ctree_downloads = GTK_CTREE
12:         (lookup_widget(main_window, "ctree_downloads"));
13:     GtkCTree *ctree_downloads_queue = GTK_CTREE
14:         (lookup_widget(main_window, "ctree_downloads_queue"));
15:
16:     gtk_clist_freeze(GTK_CLIST(ctree_downloads_queue));
17:     gtk_clist_freeze(GTK_CLIST(ctree_downloads));
18:     node_list = g_list_copy(GTK_CLIST(ctree_downloads)->selection);
19:     data_list = downloads_gui_collect_ctree_data(ctree_downloads,
20:         node_list, TRUE, TRUE);
21:
23:     for (; NULL != data_list; data_list = g_list_next(data_list)) {
24:         d = (struct download *) data_list->data;
25:
26:         if (!d) {
27:             g_warning(
28:                 "on_popup_downloads_push_activate(): row has NULL data\n");
29:             continue;
30:         }
31:         download_fallback_to_push(d, FALSE, TRUE);
32:     }
33:
34:     gtk_clist_thaw(GTK_CLIST(ctree_downloads_queue));
35:     gtk_clist_thaw(GTK_CLIST(ctree_downloads));
36:     g_list_free(data_list);
37:     g_list_free(node_list);
38: }

解答

解答は別のページにあります。テストに挑戦する前に解答を読むような事はしないで下さいね。