Changeset 21578

Timestamp:

2012/03/03 02:54:09 (12 years ago)

Author:

Seasoft

Message:

#1680 (SC_CheckError#evalCheck 'false' 以外で、値が false の定数名を渡すとエラーと誤評価する)

File:

• branches/version-2_12-dev/data/class/SC_CheckError.php (modified) (2 diffs)

branches/version-2_12-dev/data/class/SC_CheckError.php

@@ -1156 +1156 @@
 
     /**
-     * PHPコードとして評価可能かチェックする.
+     * パラメーターとして適切な文字列かチェックする.
      *
      * @access private
@@ -1173 +1173 @@
 
     /**
-     * $value が PHPコードとして評価可能かチェックする.
-     *
+     * パラメーターとして適切な文字列かチェックする.(サブルーチン)
+     *
+     * 下記を満たす場合を真とする。
+     * ・PHPコードとして評価可能であること。
+     * ・評価した結果がスカラデータ(定数に指定できる値)であること。
+     * 本メソッドの利用や改訂にあたっては、eval 関数の危険性を意識する必要がある。
      * @access private
-     * @param mixed PHPコードとして評価する文字列
-     * @return mixed PHPコードとして評価できない場合 false,
-     *               評価可能な場合は評価した値
+     * @param string 評価する文字列
+     * @return bool パラメーターとして適切な文字列か
      */
     function evalCheck($value) {
-        // falseは、正当な式と評価する。
-        if ($value === 'false') {
-            return true;
-        }
-        return @eval('return ' . $value . ';');
+        return @eval('return is_scalar(' . $value . ');');
     }