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

Timestamp:

2012/03/04 16:34:48 (14 years ago)

Author:

Seasoft

Message:

#1684 (PHPの言語特性に合わせた簡素な実装に改善する)
#1685 (SC_CheckError#FILE_EXT_CHECK ファイル名を送出しないとすり抜けの懸念)

File:

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

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

@@ -719 +719 @@
         $this->createParam($value);
 
-        if ($_FILES[$value[1]]['name'] != '') {
-            $errFlag = 1;
-            $array_ext = explode('.', $_FILES[$value[1]]['name']);
-
-            $strExt = '';
-
-            foreach ($value[2] as $checkExt) {
-                $ext = '';
-
-                // チェック拡張子のピリオドの数を取得('tar.gz'の場合1個、'jpg'のように通常は0個)
-                $count_period = substr_count($checkExt, '.');
-
-                if ($count_period > 0) {
-                    for ($i = max(array_keys($array_ext)) - $count_period; $i < count($array_ext); $i++) {
-                        $ext .= $array_ext[$i] . '.';
-                    }
-                    $ext = preg_replace("/.$/", "" ,$ext);
-                } else {
-                    $ext = $array_ext[ count ($array_ext) - 1 ];
+        $match = false;
+        if (strlen($_FILES[$value[1]]['name']) >= 1) {
+            $filename = $_FILES[$value[1]]['name'];
+
+            foreach ($value[2] as $check_ext) {
+                $match = preg_match('/' . preg_quote('.' . $check_ext) . '$/i', $filename) >= 1;
+                if ($match === true) {
+                    break 1;
                 }
-
-                $ext = strtolower($ext);
-
-                if ($ext == $checkExt) {
-                    $errFlag = 0;
-                }
-
-                if ($strExt == '') {
-                    $strExt.= $checkExt;
-                } else {
-                    $strExt.= "・$checkExt";
-                }
-            }
-        }
-        if ($errFlag == 1) {
-            $this->arrErr[$value[1]] = '※ ' . $value[0] . 'で許可されている形式は、' . $strExt . 'です。<br />';
+            }
+        }
+        if ($match === false) {
+            $str_ext = implode('・', $value[2]);
+            $this->arrErr[$value[1]] = '※ ' . $value[0] . 'で許可されている形式は、' . $str_ext . 'です。<br />';
         }
     }