エッジケース共

[hyudai28]

42tokyoのレビューから引用

[hyudai28]

環境変数がない場合にexportを打つと、(null)が表示されるバグが見つかりました。

[hyudai28]

環境変数PATHに"::"が入ったときカレントディレクトリの検索が走らない点

[hyudai28]

SHELLの環境変数セットアップが走っていない点

[hyudai28]

minishellで改行だけをしたときにleakしていました

[hyudai28]

echo -nnn のように複数のオプションを取るケースに対応されていませんでしたが、特殊なオプションのケースだとして処理しました。

[hyudai28]

シングルクオーテーション二つだけの入力時のエラー対応。

[hyudai28]

"export _"をした後にenvをすると無限ループ。

[hyudai28]

環境変数なしで起動した時のリーク。

[hyudai28]

cd: /(ルートディレクトリ)に移動できませんでした。レビュー中は権限の問題かなという話になったので問題なしとしています。レビュー後に検証したところcd ..だと移動できなかったのですが、cd /だと移動できたので怪しげです。

[hyudai28]

実行ユーザーのpwd: PWDを実行した際にbashでは/tmp/minishellと表示されるところが/private/tmp/minishellになっていた、ワカモレ上で/tmpが/private/tmpにシンボリックリンクが貼られていて、getcwd関数だとシンボリックリンクを解釈してしまうみたいでした。

[hyudai28]

リダイレクト時にファイルディスクリプタを渡せませんでしたが、pedageで対応しなくて良いと回答がありました。

[hyudai28]

/dev/urandom cat | head -n 1を実行した際にpermission deniedと表示されました

[hyudai28]

expr $? + $?を実行すると0と2が交互にでましたが、こちらについてはexprコマンドのmanを確認するとexprは評価値が0になる場合はexit statusとして1を返すみたいでした。

[hyudai28]

pipeとredirectionを組み合わせた cat Makefile | grep make > test.txt などのコマンドを実行した際に、既存の test.txt の上部に文字が書き加えられるという動作を行ってしまっていたため、redirection のみ no をつけさせていただきました。

[hyudai28]

"" や "." で微妙に違う挙動でした。

[hyudai28]

cd /tmp/../../ などで、heap use after free や、chdir(NULL) という挙動になっていました。

[hyudai28]

<< "" で static 変数を double free していました。

[hyudai28]

expand する際に、 echo "'$foo'" で 'bar' になるはずのところ 'bar " になっていました。

[hyudai28]

プロンプトに何か入力された状態でシグナルを受け取る時にminishellが終了してしまう。

[hyudai28]

環境変数が渡されずにminishellを起動すると環境変数関連のコマンドが動作しない。

[hyudai28]

ビルトインコマンドのエラー時の終了ステータスが反映されていない（レビューではcdのみ検出）。

[hyudai28]

リダイレクトでfdにマイナスの値を渡すとセグフォしてしまう。

[hyudai28]

fdの最大値 < リダイレクト先のfd < INT_MAXのようなfdを渡す際にopenでのエラーハンドリングが必要。

[hyudai28]

サニタイザをつけると、tokenizerあたりでoverflowを起こしてabortしている。

[hyudai28]

pipe関数がwhile内で使用されいて、fdがcloseされていなかったので、多段パイプを500個くらい繋げたところ、パイプが動かなくなりました。

[hyudai28]

'command |'のあとにスペースだけを入れた入力で、syntax errorの出力の後、free時のNULL詰め処理に不具合がありセグフォしたのでcrashフラグを立てました

[hyudai28]

環境変数の区切り文字がbashと異なり、想定の動作とは異なる動作が行われていました。 export の処理で 環境変数の文字列の追加の処理が行われていませんでした。

[hyudai28]

minishell内でmisnishellを呼んだ後にexitすると常にecho $?が1になってしまいます。

[hyudai28]

unsetのエラーメッセージがexportのものになっていました。

[hyudai28]

catの後のctrl+DでDが表示されてしまいます。

[hyudai28]

./minishell ./minishell exit 1

[hyudai28]

Ctrl+Dを押すとそれまでコマンドラインに入力していた内容がフラッシュされてしまいました。

[hyudai28]

1024byteを超えるコマンドが入力できないようになってしまうので、改善が必要だと思います。

[hyudai28]

" "のようにクオートした空白文字だけをコマンドラインに入力すると、echo $?の出力が258に変わってしまう。

[hyudai28]

echo a " " b の出力はa b(4文字)であるべきだが、a b(3文字)になる。

[hyudai28]

export VARのように値を代入せずに定義した場合、exportで表示されない。

[hyudai28]

export A=1 123のようにexportの2つ目以降の引数がスルーされている。

[hyudai28]

unsetも2つ目以降の引数がスルーされている。

[hyudai28]

Guacamoleにおいて/tmp以下にいる時のpwdの出力が/private/tmp...になる。

[hyudai28]

カレントディレクトリの実行権限(x)がない状態で cd ..がPermission deniedで失敗する。

[hyudai28]

カレントディレクトリにパスが通っていない状態で、カレントディレクトリの実行ファイルを./なしで実行できてしまう。

[hyudai28]

unset PATHの後にexport PATH=/binしてもlsなど/binにいるコマンドが見つからない。

[hyudai28]

子プロセスの実行部分に多数のリークがありました(getpid()とleaksコマンドを組み合わせてデストラクタに設定することで検知可能でした)

[hyudai28]

echo printfのエラーハンドリングが漏れていました

[hyudai28]

stdoutがcloseされていると正常に動作しませんでした。fd 1がopenされていることを期待した実装になっているようです。（原因を特定し忘れました分かったら教えて！）

[hyudai28]

名前付きpipeへのリダイレクトでブロッキングされませんでした（原因を特定し忘れました分かったら教えて！）

[hyudai28]

exit 後ろのスペースがtrimされませんでした

[hyudai28]

signal ハンドラの設定が呼び出し元から引き継がれませんでした。ハンドラが非同期安全ではありませんでした

[hyudai28]

export +=が実装されていませんでした

[hyudai28]

存在しないdirから起動できませんでした