Toshiharu Harada
harad****@nttda*****
2008年 11月 6日 (木) 17:39:57 JST
原田です。こんにちは。 メインライン化ですが、その後もしぶとく続いています。 細かな経緯と状況は、2chのスレで読んでいただくとして、 11/4に-mm treeに第12回目の提案を投稿しました。 すると、早速Andrew Mortonからコメントがつきました。 以下は、熊猫先生による要約です。 （１）task_struct に in_execve を追加するパッチが 当たらない （２）list_head を使わずに list1 を追加する理由を 説明せよ （３）予め procfs の super_block を記憶しておき、 d_realpath で比較する方が良い （４）round_up ではなく PTR_ALIGN は使えないのか （５）list1_head を hlist_head で代用できないか （６）fmb_list から解放されないのはバグではないか （７）temporal ではなく temporary だ（少なくとも ２箇所ある） （８）dynamic_memory_size カウンタは役に立つのか （９）tmy_alloc でゼロクリアする必要は無いのでは？ （１０）tmy_print_ascii の説明をせよ （１１）CONFIG_SYSCTL=n でコンパイルできるかどうか確認せよ （１２）tmy_read_memory_counter の返す値の単位は （１３）ユーザスペースとのインタフェースに関してドキュメント化せよ （１４）tmy_read_memory_counter は６４ビットにすべきで はないか （１５）なぜ is_byte_range が必要なのか （１６）is_decimal や is_hexadecimal や is_alphabet_char は ctype.h で定義済み （１７）str_starts や normalize_line はグローバル にすべきではないか （１８）isprint を使えないか （１９）文字列操作が多すぎないか （２０）128〜255 の範囲の文字を含むファイル名を正 しく扱えるのか （２１）tmy_find_domain でロックは不要なのか （２２）strchr(pathname, '\0') よりも pathname + strlen(pathname) が良い （２３）tmy_io_printf の success の意味は （２４）executable mapping in the mapping list が正しい ことの保証は （２５）プログラムが削除されても大丈夫か （２６）tmy_check_flags の説明が無意味 （２７）tmy_check_flags で WARN_ON または WARN_ON_ONCE が良い （２８）sbin_init_started の役割を説明せよ （２９）複数の /sbin/init が動作する cgroups 環境 では何が起こるのか （３０）tmy_check_domain_quota よりも tmy_domain_quota_exceeded が良い （３１）tmy_find_or_assign_new_profile のロック取得が早 すぎる （３２）write_profile と read_profile の先を説明せよ （３３）ユーザスペースとのインタフェースに関してレビューが必要 （３４）削除されたエントリは解放されないのか （３５）manage_by_non_root はユーザスペースとのインタ フェースか （３６）containerised environment ではどう動作するのか （３７）一般ユーザがログフラッディング攻撃を仕掛けられないか （３８）PID を用いたインタフェースは良くない （３９）strict_strtoul の戻り値をチェックしないのか （４０）/sbin/tomoyo-init というハードコーディングを避 けられないか （４１）policy_loader_exists は sys_access で十分 ではないか （４２）call_usermodehelper の前に policy_loader_exists を呼ぶ理由は （４３）read_updates_counter は何をしているのか （４４）access_ok は copy_to_user の中で行われる ので不要では （４５）false と true は定義済み （４６）acl_info を packed にする理由を説明せよ （４７）__packed__ ではなく __packed を使え （４８）KERNEL_DOMAIN が大文字なのは何故か （４９）TOMOYO の関数であることが判るような関数名を付けよ 大変細かくコードを読んでいるようで、これらの内容に 答える（対応する）ことができれば、mmツリーに入れてもらえること は ほぼ確実と言って良いと思います。 12回目を含め、TOMOYOの過去の提案内容については、下記より 参照いただけます。 http://elinux.org/TomoyoLinux#Mainline なかなか気持ちよくマージしてもらえませんが（笑）、やるべきことを やり続けて着実に進んでいると思います。乞うご期待 (^-^) -- 原田 季栄 (Toshiharu Harada) harad****@nttda*****
TOMOYO
Fork