Commit Fest:May

OSS/Linux

かねてより提案していた SE-PostgreSQL のパッチに関して、Commit Fest:May の開始に伴い、開発チームリーダの１人である Tom Lane から物凄い勢いでレビューが返ってきた。
http://wiki.postgresql.org/wiki/CommitFest:May
http://archives.postgresql.org/message-id/3275.1210019965@sss.pgh.pa.us

それに合わせて、かなり大掛かりにコードをいじっている。
主なものでも
・PGACEのフックを static inline に変更
・initdb時に pg_security の代用ファイルを使わないように修正（On Memoryにキャッシュ）
・TRUNCATE→無条件DELETEへの書き換えをやめる（トリガ関数非互換）
・EXECUTEパラメータリストへのチェックを追加
・oidシステム列も書き込み可能にした
・WHERE句への条件追加をやめ、テーブルのスキャン中にフックを呼ぶようにした
・セキュリティポリシーを修正

それに加えて、パッチを投稿するためには、少なくとも regression test をパスしないといけないらしい。
最初に投稿した際のバージョンだと、大小様々に30個程度が FAIL になっていたようだ（死）が、上記のような改善を加えることで、現バージョンでは3個にまで減らすことができた。
このうち2個は機能追加に伴う真っ当なもので、あと1個は security_label 型の演算子の実装が必要であるためのもの。

[kaigai@masu sepgsql]$ make check
    ：
〜（略）〜
    ：
========================
 3 of 115 tests failed.
========================
security_label型に配列としての利用を許すオペレータが必要
*** ./expected/type_sanity.out  Mon Apr 21 13:14:24 2008
--- ./results/type_sanity.out   Thu May 15 22:35:00 2008
***************
*** 65,74 ****
       WHERE p2.typname = ('_' || p1.typname)::name AND
             p2.typelem = p1.oid and p1.typarray = p2.oid);
   oid | typname
! -----+---------
   210 | smgr
   705 | unknown
! (2 rows)

  -- Make sure typarray points to a varlena array type of our own base
  SELECT p1.oid, p1.typname as basetype, p2.typname as arraytype,
--- 65,75 ----
       WHERE p2.typname = ('_' || p1.typname)::name AND
             p2.typelem = p1.oid and p1.typarray = p2.oid);
   oid  |    typname
! ------+----------------
    210 | smgr
    705 | unknown
!  3403 | security_label
! (3 rows)

  -- Make sure typarray points to a varlena array type of our own base
  SELECT p1.oid, p1.typname as basetype, p2.typname as arraytype,

PostgreSQL本体のチェックより先に、SE-PostgreSQLがエラーを検出している

*** ./expected/create_function_1.out    Thu May 15 22:34:44 2008
--- ./results/create_function_1.out Thu May 15 22:35:01 2008
***************
*** 72,78 ****
  ERROR:  only one AS item needed for language "sql"
  CREATE FUNCTION test1 (int) RETURNS int LANGUAGE C
      AS 'nosuchfile';
! ERROR:  could not access file "nosuchfile": No such file or directory
  CREATE FUNCTION test1 (int) RETURNS int LANGUAGE C
      AS '(略)/regress.so', 'nosuchsymbol';
  ERROR:  could not find function "nosuchsymbol" in file "(略)regress.so"
--- 72,78 ----
  ERROR:  only one AS item needed for language "sql"
  CREATE FUNCTION test1 (int) RETURNS int LANGUAGE C
      AS 'nosuchfile';
! ERROR:  SELinux: could not obtain security context of nosuchfile
  CREATE FUNCTION test1 (int) RETURNS int LANGUAGE C
      AS '(略)/regress.so', 'nosuchsymbol';
  ERROR:  could not find function "nosuchsymbol" in file "(略)regress.so"

新しいシステムカタログpg_securityが追加されたので

*** ./expected/sanity_check.out Sun Nov 25 12:55:19 2007
--- ./results/sanity_check.out  Thu May 15 22:35:06 2008
***************
*** 111,116 ****
--- 111,117 ----
   pg_pltemplate           | t
   pg_proc                 | t
   pg_rewrite              | t
+  pg_security             | t
   pg_shdepend             | t
   pg_shdescription        | t
   pg_statistic            | t
***************
*** 149,155 ****
   timetz_tbl              | f
   tinterval_tbl           | f
   varchar_tbl             | f
! (138 rows)

  --
  -- another sanity check: every system catalog that has OIDs should have
--- 150,156 ----
   timetz_tbl              | f
   tinterval_tbl           | f
   varchar_tbl             | f
! (139 rows)

  --
  -- another sanity check: every system catalog that has OIDs should have

しかしこの regression test、作業ディレクトリの下に代替のルートディレクトリを作ってそこに PostgreSQL をインストールするという作りになっているので、正しいセキュリティコンテキストが付かずに難儀する。
仕方ないので、sepgsql_regression_test_mode という boolean を追加して、regression test に必要なパーミッションを与えるようにした。
user_home_tの共有ライブラリをロードできたりするので、通常運用では絶対に on にしてはならないが、まぁ、これはこれでアリだろう。