ls -l はなぜ事故を起こすのか

はじめに この記事は以下の記事で事故がなぜ起きたかを技術的に理解し筆者のtipsを共有しようと思い書きました。あくまで筆者のtipsですので、これが正しいとは限りません。 やはり、下記のコマンドが原因でした。 [root@foo script]# ls -l | sh 事故そのものは ls コマンドに -l を付けて実行してしまった事で、ls コマンドが出力したシンボリックリンクが以下の様になってしまったという話です。

[mattn]

書いた。

[naaa145]

引用元を読まずにブコメを書くせっかちさんが、ぶっつけ本番環境やらかすんやろな

[uunfo]

| shをやめろ／ああ、ls | grep shしたかったのか。普通はls *.shにすると思うけど。せめて¥.shか"sh"にしておけばなあ／コピペも事故になりそうで怖いなあ。レビューまでするなら常にシェルスクリプトにすればいいのに。

[rryu]

ファイル名を抽出するならfind一択で、lsの結果をgrepするという発想はなかった。

[tsutsumi154]

−lスイッチが悪いわけじゃあるまい

[door-s-dev]

lsの結果を直接shに渡すこと自体に怖さを感じるなぁ/ブコメ見て完全に誤読していたことに気づいた。なんかすんません

[shikiarai]

| shとか怖すぎて逆に笑った。世界を滅ぼしに来たのか？ / grep打ち忘れにしてもそもそもgrepでshを探すのはやめよーという話だな……

[RySa]

最後の一言に尽きる。ls -lは友達、怖くない。

[camellow]

そもそも | shとかやるのが微妙って話だし、| grep sh とするつもりでgrepを書き忘れたというミスの話なら-lを付けないって対策もどうせミスで台無しになるだろうしaliasの設定をしてないサーバで同じことをやるだろうよ。

[fa11enprince]

パイプに渡すときは常にls -1を使ってる。xargsと組み合わせることが個人的には多いです。

[trashtoy]

ls の実行結果を直接コマンドとして実行する発想はなかった. (感覚的におっかなく感じる) ls のオプション云々以前に, 安全性に関する感性の問題を感じる / と思ったら早計だったっぽい

[estragon]

本番環境でぶっつけ本番のコマンド叩くなってのは当然のことではあるけど、インシデント対応で何が起こってるのか把握するためとか、焦ってる時はあるから難しい / “本番環境はカジュアルに触ってはならぬ。”

[takehikom]

どうやって防ぐか『事前に試験環境で、想定通りのコマンドが実行される様なスクリプトを書いておき、レビューを受け、それを本番に持っていって実行します』

[stp7]

正直 “| grep sh” の時点で怖い。

[mather314]

find 使いなされ

[at_yasu]

あー…なるほど、そういう事故かぁ…

[ku__ra__ge]

「ls | sh がやりたかった事ではなく ls | grep sh しようとしたら grep を忘れたというシナリオ」　元記事見てなんじゃこりゃと思ってたが、こういうことだったのか。

[sirobu]

そもそもls -laのaliasをllにしておけばいいんじゃないの

[ohaan]

「ls -l」はファイルの詳細を表示するコマンドとだけ教わる事が多いらしく、割とカジュアルに初学者が本番環境で業務を任されると「ls -l」やってたな。

[knok]

インタラクティブかどうかで挙動が変わるコマンドは怖い

[fashi]

なぜかrm等の危険な作業を行う場合の話になってるけど本番環境でlsしたりgrepしたりするだけでも同様の入念なコードレビューを行っているのだろうか

[saiyu99sp]

sl しゅっぽぽぽぽぽ…

[t-tanaka]

「余計なファイルを生成」違う。シンボリックリンクのリンク先には，まず間違いなく既存のファイルが存在している。そのファイルを書き潰してしまうのが問題。

[hamamuratakuo]

どうやって防ぐか　よくやるのがスクリプトを書くことです。事前に試験環境で、想定通りのコマンドが実行される様なスクリプトを書いておき、レビューを受け、それを本番に持っていって実行します。

[manmo999]

xargsで組み立ててshつけることはよくあるけど、ls -l|shはやらないなぁ

[richard_raw]

なに？その用途ならfind ... -print0でxargsに渡すのではないのか！（めんどくさい人）

[jojo800]

複数の条件で探したいときはgrepで探す。findで-oとかつなげるのだるすぎない？

[UDONCHAN]

それな

[yetch]

検索文字列をコマンドと同じにしないようにしよう

[hagane]

lsをパイプに渡すときは意識してls -1にしてるかなあ

[pribetch]

MEL使い「とりあえずstring $hoge[] = 'ls -sl';」

[honeybe]

slの話ではなかった(ぇ

[tettekete37564]

ls -l に grep 使おうとするのは不用意すぎるな。まあタイムスタンプとか見たかったんだろうから ls -l *.sh でよくね。後半の while read にこだわってるのもちょっと。別に xargs でよくね？ ls -1 *.sh | xargs echo で事足りような

[wataken44]

参照先の本来やりたかったことをやるなら ls *.sh でよくね（本番環境でレビューされてないコマンドをうつなというのはそれはそう）

[mas-higa]

ls 関係ないし | 後の grep の引数が実行可能なファイル名であり grep をタイプし忘れたのなら、どうやったって事故る。

[asuka0801]

本番環境で気軽にrootでコマンドを打つなというのがまず一つあると思うけどもls -lを手癖で打っちゃう人はそれなりに多そう。

[su_zu_ki_1010]

本番環境で作業する時は事前に実行したいコマンドを用意して、テスト環境でテストしておくのが大事。そして、本番環境は事前用意したコマンドをコピペで実行していくだけ、これは徹底しないとダメですね。

[m50747]

findだけどな。

[d0i]

shell scriptでコマンド列を生成した結果を `./nanika.sh |sh -x` ってやることはある。この場合、loopとか変数置換を展開して所望の操作になってるか、 `|sh -x` つけないで実行して確認できるのがうれしい(そういう話じゃない

[punychan]

「ですので ls | としていればシンボリックリンクに起因した事故は起きなかった」いや、ls | sh とかして、そこに万一PATHが通ってたりしたら、大事故だぞ。

[tobigitsune]

トップブコメ群、全文読んでる人がほとんどいないのでは…？