アップロードファイルリスト表示時に必要以上にメモリを使用している

[imamura623]

■現象

アップロードファイルリスト(admin_page_list_c_file)にて、アップロード可能なファイルサイズの制限を上げていた場合などに必要以上のメモリを使用してしまう可能性がある。

■原因

ファイルをリストアップする関数(db_file_c_file_list)でファイルサイズを調べるために取得するファイルのバイナリデータが、ファイルサイズ取得後も保持されるため。

■修正内容

ファイルサイズ取得後にバイナリデータを格納した変数を破棄し、メモリを開放するように変更。

■関連情報

元のDescription

​http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=19977より転記

webapp/lib/db/file.phpのdb_file_c_file_list関数では各ファイルのファイルサイズを調べるためにそれぞれの行をセレクトしていますが
ファイルサイズを調べ終えた後もファイルのバイナリデータを保持しているため
アップロード可能なファイルサイズの制限を上げていた場合などには高確率でメモリを食いつぶします。

管理画面のファイル管理でエラーが出る場合は、ループ末尾で
unset($c_file_list[$i]['bin']);
などとしてメモリを開放する必要があります 

[kiwa]

動作上の問題ではなさそうなので、そのまま修正待ちにまわします。開発者の方は検討をお願いします。

[ebihara]

修正方針が明確かつ対応も容易なので、対応します。

[kiwa]

対応します。

[kiwa]

以下のリビジョンで対応しました。ご確認ください。

• 2.12 - r10747, r10749
• trunk - r10748, r10750

指摘されたdb_file_c_file_list以外に複数のファイルのバイナリデータを保持するような流れを見つけられなかったので、db_file_c_file_listのみ修正しています。他の修正箇所が存在している場合は差し戻していただければ対応します。

[kiwa]

確認しました。問題ありません。