Changeset 17489

Timestamp:

2008/08/03 14:57:49 (16 years ago)

Author:

nanasess

Message:

無効なファイルポインタが渡された場合に無限ループするバグ修正(#311)

File:

• branches/comu-ver2/data/class/util/SC_Utils.php (modified) (2 diffs)

branches/comu-ver2/data/class/util/SC_Utils.php

@@ -1021 +1021 @@
     }
 
+    /**
+     * テキストファイルの文字エンコーディングを変換する.
+     *
+     * $filepath に存在するテキストファイルの文字エンコーディングを変換する.
+     * 変換前の文字エンコーディングは, mb_detect_order で設定した順序で自動検出する.
+     * 変換後は, 変換前のファイル名に「enc_」というプレフィクスを付与し,
+     * $out_dir で指定したディレクトリへ出力する
+     *
+     * TODO $filepath のファイルがバイナリだった場合の扱い
+     * TODO fwrite などでのエラーハンドリング
+     *
+     * @access public
+     * @param string $filepath 変換するテキストファイルのパス
+     * @param string $enc_type 変換後のファイルエンコーディングの種類を表す文字列
+     * @param string $out_dir 変換後のファイルを出力するディレクトリを表す文字列
+     * @return string 変換後のテキストファイルのパス
+     */
     function sfEncodeFile($filepath, $enc_type, $out_dir) {
         $ifp = fopen($filepath, "r");
 
-        $basename = basename($filepath);
-        $outpath = $out_dir . "enc_" . $basename;
-
-        $ofp = fopen($outpath, "w+");
-
-        while(!feof($ifp)) {
-            $line = fgets($ifp);
-            $line = mb_convert_encoding($line, $enc_type, "auto");
-            fwrite($ofp,  $line);
-        }
-
-        fclose($ofp);
-        fclose($ifp);
-
+        // 正常にファイルオープンした場合
+        if ($ifp !== false) {
+
+            $basename = basename($filepath);
+            $outpath = $out_dir . "enc_" . $basename;
+
+            $ofp = fopen($outpath, "w+");
+
+            while(!feof($ifp)) {
+                $line = fgets($ifp);
+                $line = mb_convert_encoding($line, $enc_type, "auto");
+                fwrite($ofp,  $line);
+            }
+
+            fclose($ofp);
+            fclose($ifp);
+        }
+        // ファイルが開けなかった場合はエラーページを表示
+          else {
+              SC_Utils::sfDispError('');
+              exit;
+        }
         return     $outpath;
     }
@@ -1431 +1456 @@
                     break;
                 }
-                
+
                 mb_ereg("^(.*[\/])(.*)",$data_, $matches);
                 $data=$matches[2];