[FDclone-users:00966] Re: LOG_DIR from within archive browser causes SEGV

["Akinori MUSHA" <knu@iDaemons.org>]

遅くなりましたが、試す時間が取れましたので報告します。

At Tue, 03 Apr 2012 21:28:07 +0900,
Takashi SHIRAI wrote:
> In Message-Id <86wr5yklz5.knu@iDaemons.org>
>         "Akinori MUSHA" <knu@iDaemons.org>さんwrites:
> > > 　もう 4 年前のことなんて忘却の彼方で、コード見ても何やって
> > > んだかさっぱりな状態ではありますが、下記 patch で修正される
> > > んじゃないかと思います。試して下さい。
> > 
> > 　対応ありがとうございます。
> > 
> > 　試しましたが、まだSEGVしてしまいます。
> 
> 　落ちるか否かが表示ファイル数に依存していたようですね。その
> 依存性を取り除いてみました。先の patch は破棄してこちらをお
> 試し下さい。

これで直ったようです。

> > 　これはバグというほどではないですが、アーカイブブラウザ内で H から cd
> > するとアーカイブのパス名が新しいカレントディレクトリ+アーカイブファイル
> > 名へと更新され、たとえそのファイルが存在しなくてもブラウザが継続します。
> 
> 　あれは shell を呼んでますからね。やろうと思えば何でもアリ
> なので、何をされてもいい耐性は難しいですが、影響の大きそうな
> ところだけなら対処可能だと思います。
> 　取り敢えず、archive browser で使用禁止の internal command
> は command line 実行も禁止して、他の command も終了後に必ず
> CWD を元に戻すようにしてみました。

あまりいろいろは試していませんが、 cd については

        cd / && ls

が意図通り実行され、復帰するとCWDが戻っていることを確認しました。

> > これはブラウザを抜けてくれた方がうれしそうです。
> 
> 　EXECUTE_SH で Enter のみ入力すると対話モードになりますが、
> こうなると見ため shell そのものなんで一時的に cd したくなる
> ことはあると思います。
> 　それで archive browser を抜けちゃうと一瞬意味不明になると
> 思うので、元の CWD に戻しておいた方が無難だと思います。cd し
> たか否かを見張ってるような実装は却って面倒ですし。
> 
> 　そもそも EXECUTE_SH 自体を archive browser 内で使用禁止に
> した方が安全なのかも知れませんけど、それだと全く危険性のない
> command まで使えなくなって利便性が落ちますからね。
> 　そこそこ安全でそこそこ便利な落としどころとなると、こんな感
> じになるんじゃないでしょうか。

そうですね。いいと思います。ありがとうございました。

-- 
Akinori MUSHA / http://akinori.org/