BugTrack/655
の編集
Top
/
BugTrack
/
655
[
トップ
] [
編集
|
差分
|
履歴
|
添付
|
リロード
] [
新規
|
一覧
|
検索
|
最終更新
|
ヘルプ
|
ログイン
]
*attach.inc.php プラグインで、ATTACH_PASSWORD_REQUIREを有効にした時に、&br;添付したファイルが削除できなくなる [#m5960262] -ページ: BugTrack -投稿者: [[jjyun]] -優先順位: 普通 -状態: 完了 -カテゴリー: プラグイン -投稿日: 2004-08-13 (金) 14:21:00 -バージョン: 1.4.3, 1.4.4rc1 **メッセージ [#w0acca7e] attach.inc.php プラグインの設定で、 ATTACH_PASSWORD_REQUIRE を有効にしてファイルを添付した場合、 入力したパスワードはそのまま、logファイルに書き出されています。 一方、添付ファイルの削除の際に入力するパスワードは、 md5()を用いてハッシュ値に変換されてから比較されているため、 添付時に入力したパスワードと同じ文字列を入力しても、 「パスワードが違うため削除できない」という状況になってしまいます。 attach_upload() の部分を修正すればよいと思います。 : $obj->getstatus(); - $obj->status['pass'] = ($pass !== TRUE and $pass !== NULL) ? $pass : ''; + $obj->status['pass'] = ($pass !== TRUE and $pass !== NULL) ? md5($pass) : ''; $obj->putstatus(); return array('result'=>TRUE,'msg'=>$_attach_messages['msg_uploaded']); } ---- -こんにちは。この機能はさっぱり使ったことがないのですが、こういう時はログに書き出すパスワードの方を md5ハッシュにすべきではないでしょうか？ :) ログを見ることで、パスワードが漏れちゃいますからね。どう思われますか？ -- [[henoheno]] &new{2004-08-13 (金) 21:54:16}; -あ、この差分はまさにそういうものかな(^^; どう直すべきかについて勝手に妄想してしまいました・・・(^^ -- [[henoheno]] &new{2004-08-13 (金) 21:57:04}; -今回のそれは、まさしくログへの出力内容になる部分だと理解しています。修正よろしくお願いします。 -- [[jjyun]] &new{2004-08-14 (土) 00:12:52}; -ATTACH_PASSWORD_REQUIRE の利用者がどの程度いるのかわからないのですが、あるいは移行用のスクリプトが必要かも -- [[にぶんのに]] &new{2004-08-15 (日) 00:54:58}; -ここで説明されていることが正しいならば、動作確認まできちんとやった利用者は [[jjyun]] さんが初めてでしょうね。 -- [[henoheno]] &new{2004-08-15 (日) 07:36:24}; -確かにご指摘の通りでした。ありがとうございました :) -- [[henoheno]] &new{2004-08-15 (日) 09:16:05}; -- [[cvs:plugin/attach.inc.php]] (v1.4:1.60) -復旧する場合、手作業で対応するならば、ログファイル(attach/*.log)の３行目をそれぞれmd5ハッシュと置き換えると良さそうです。 -- [[henoheno]] &new{2004-08-15 (日) 09:21:38}; -- ※例えば3行目に「fsfs」とある場合: echo -n "fsfs" | md5sum の出力結果と入れ替える (-nオプションは必須) //#comment
タイムスタンプを変更しない
*attach.inc.php プラグインで、ATTACH_PASSWORD_REQUIREを有効にした時に、&br;添付したファイルが削除できなくなる [#m5960262] -ページ: BugTrack -投稿者: [[jjyun]] -優先順位: 普通 -状態: 完了 -カテゴリー: プラグイン -投稿日: 2004-08-13 (金) 14:21:00 -バージョン: 1.4.3, 1.4.4rc1 **メッセージ [#w0acca7e] attach.inc.php プラグインの設定で、 ATTACH_PASSWORD_REQUIRE を有効にしてファイルを添付した場合、 入力したパスワードはそのまま、logファイルに書き出されています。 一方、添付ファイルの削除の際に入力するパスワードは、 md5()を用いてハッシュ値に変換されてから比較されているため、 添付時に入力したパスワードと同じ文字列を入力しても、 「パスワードが違うため削除できない」という状況になってしまいます。 attach_upload() の部分を修正すればよいと思います。 : $obj->getstatus(); - $obj->status['pass'] = ($pass !== TRUE and $pass !== NULL) ? $pass : ''; + $obj->status['pass'] = ($pass !== TRUE and $pass !== NULL) ? md5($pass) : ''; $obj->putstatus(); return array('result'=>TRUE,'msg'=>$_attach_messages['msg_uploaded']); } ---- -こんにちは。この機能はさっぱり使ったことがないのですが、こういう時はログに書き出すパスワードの方を md5ハッシュにすべきではないでしょうか？ :) ログを見ることで、パスワードが漏れちゃいますからね。どう思われますか？ -- [[henoheno]] &new{2004-08-13 (金) 21:54:16}; -あ、この差分はまさにそういうものかな(^^; どう直すべきかについて勝手に妄想してしまいました・・・(^^ -- [[henoheno]] &new{2004-08-13 (金) 21:57:04}; -今回のそれは、まさしくログへの出力内容になる部分だと理解しています。修正よろしくお願いします。 -- [[jjyun]] &new{2004-08-14 (土) 00:12:52}; -ATTACH_PASSWORD_REQUIRE の利用者がどの程度いるのかわからないのですが、あるいは移行用のスクリプトが必要かも -- [[にぶんのに]] &new{2004-08-15 (日) 00:54:58}; -ここで説明されていることが正しいならば、動作確認まできちんとやった利用者は [[jjyun]] さんが初めてでしょうね。 -- [[henoheno]] &new{2004-08-15 (日) 07:36:24}; -確かにご指摘の通りでした。ありがとうございました :) -- [[henoheno]] &new{2004-08-15 (日) 09:16:05}; -- [[cvs:plugin/attach.inc.php]] (v1.4:1.60) -復旧する場合、手作業で対応するならば、ログファイル(attach/*.log)の３行目をそれぞれmd5ハッシュと置き換えると良さそうです。 -- [[henoheno]] &new{2004-08-15 (日) 09:21:38}; -- ※例えば3行目に「fsfs」とある場合: echo -n "fsfs" | md5sum の出力結果と入れ替える (-nオプションは必須) //#comment
テキスト整形のルールを表示する
![[PukiWiki]](image/pukiwiki-dev.png "[PukiWiki]")
