Changeset 18462

Timestamp:

2010/01/01 23:54:47 (14 years ago)

Author:

Seasoft

Message:

• merge r18409
  • 取得元: version-2_4

【取得元のログメッセージ】

• #566 CSVアップロードで無限ループが発生する可能性があるソースを修正しました

File:

• branches/comu-ver2/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSVCategory.php (modified) (2 diffs)

branches/comu-ver2/data/class/pages/admin/products/LC_Page_Admin_Products_UploadCSVCategory.php

@@ -109 +109 @@
                     // レコード数を得る
                     $rec_count = $this->lfCSVRecordCount($enc_filepath);
+                    if ($rec_count === false) {
+                        $err = false;
+                        $arrErr['bad_file_pointer'] = "※ 不正なファイルポインタが検出されました";
+                    }
 
                     $fp = fopen($enc_filepath, "r");
@@ -362 +366 @@
      *
      * @param string $file_name ファイルパス
-     * @return integer CSV のカウント数
+     * @return mixed CSV のカウント数; $file_name が無効な場合は false
      */
     function lfCSVRecordCount($file_name) {
         $count = 0;
         $fp = fopen($file_name, "r");
-        while(!feof($fp)) {
-            $arrCSV = fgetcsv($fp, CSV_LINE_MAX);
-            $count++;
-        }
-
+        if ($fp !== false) {
+            while(!feof($fp)) {
+                $arrCSV = fgetcsv($fp, CSV_LINE_MAX);
+                $count++;
+            }
+        } else {
+            return false;
+        }
         return $count-1;
     }