| SQLインジェクション対策はおすみですか? 開発開始時点からのコンサルティングから、公開済みWebサイトの脆弱性検査、 脆弱性発見後の適切な対策まで |
2007-09-05 そろそろ入力値検証に関して一言いっとくか
● Webアプリケーション脆弱性対策としての入力値検証について
Webアプリケーションのセキュリティ対策としての「入力値検証」について色々言われている。セキュアコーディングの基本は入力値検証だといわれたり、さほど重要でないと言われたりしている。本当のところはどうなのだろうか。以下、バイナリデータを扱う場合の多いミドルウェア(Webサーバーなど)と対比しながら、この問題を掘り下げたい。
バイナリデータの場合(≒ミドルウェアの場合)
バイナリデータでは、入力検証が重要である。少し前にmod_imagefightを取り上げた(画像版サニタイズ言うな(2))ので、ビットマップ画像を例に説明しよう。その際に使用したBMP形式の説明を再掲する。
0000:MARK(2) ='BM' 0002:ファイルサイズ(4) * 0006:予約1(2) =0 0008:予約2(2) =0 000A:ビットマップ開始位置(4) *(間接的にサイズに関連) 000E:ヘッダサイズ(4) =0x28 * 0012:イメージ横幅(4) * 0016:イメージ高さ(4) * 001A:プレーン数(2) =1 001C:ピクセルあたりのビット数(2) 1,4,8,24 * 001E:圧縮形式(4) =0,1 0022:圧縮後のイメージサイズ * 0026:水平解像度(4) *(間接的にサイズに関連) 002A:垂直解像度(4) *(間接的にサイズに関連) 002E:使用色数(4) * 0032:重要な色数(4) 0036:カラーパレット ZZZZ:ビットマップヘッダの中には、サイズに関連する値が多いことに注意されたい。
例えば、実際に受け取ったファイルのサイズと、BMPヘッダ中の「ファイルサイズ欄」が矛盾したら処理が継続できない。また、イメージの横幅と高さを鵜呑みにしてメモリを確保していたら、これらに巨大な値を入れるだけで(画像の中身は送らなくても)、アプリケーションは巨大なメモリをアロケートしようと試みるので、簡単にDoS攻撃が成立してしまう。また「ピクセルあたりビット数」が「1,4,8,24」のいずれかになっていることもチェックする必要があるだろう。
BMPの例に限らず、一般にバイナリデータの形式は、「データ長」に続いて「データ本体」が続く場合が多い。このデータ長に間違ったあるいは巨大なデータが入っていることをチェックしなければ脆弱性の原因となる。「入力データのチェック」が必要となる所以である。
すなわち、バイナリ形式のデータを扱う上では、
・データの整合性を保証することにより動作を確実にする
・リソース割り当てを適当に制限してDoS耐性を持たせる
などのために、入力の検証は必須なのである。
また、ここで注目して欲しいことは、BMPの画像イメージの内容まではチェックしないことだ(圧縮されている場合は別)。
すなわち、入力のチェックといっても、データの中身をチェックするのではなく、いわばデータの構造や枠組みをチェックしていることになる。そりゃそうだろう。TCP/IPでやりとりできるデータに制限があったり(例えば「'」があればエラーになる)、勝手に書き換えられたり(たとえば「;」が「_」に置換される)、勝手に削除される(例えば「<」や「>」があれば削除される)としたら、TCP/IPは使い物にならなくなる。
Webアプリケーションの場合
Webアプリケーションの場合で考えると、入力データの構造のチェックとしては以下が考えられる。・HTTPヘッダ(Cookieなど)の形式チェック
・クエリ・ストリングの形式チェック
・POSTデータの形式チェック(application/x-www-form-urlencodedなどの)
これらは確かに必要だろう。しかし、これらチェックはアプリケーションの役目というよりは、ミドルウェアやライブラリの役目だ。ミドルウェア類からアプリケーションに渡されるデータは、データの中身であって、上記の形式チェックや構文解析は既に終わっている。
ミドルウェア側の構文解析などがすんでしまえば、入力データがいかなる値であっても、データを受け取った時点で直ちにセキュリティ上問題が起こるわけではない。問題が発生するのは、その値を使う時である(当たり前だ)。
その例として、クロスサイト・スクリプティング(XSS)やSQLインジェクションなどのインジェクション系脆弱性があり、値を使う時に(出力時に)エスケープなどを行う必要があるのだ。データを使う時にどの文字が問題になるかは、データの使い方(HTML、SQL、・・・)によって変わる。そのため、データを使う直前まで、エスケープなどの処理はできないことになる。
アプリケーションの入力チェックですべきこと
Webアプリケーションでも入力値のチェックはするべきであるが、それはセキュア・コーディングという意味においてではない。ユーザが間違ったデータを入力した場合にできるだけ早期に誤りを発見して、アプリケーションの信頼性を高めるという、ビジネス要件の意味合いからである。したがって、電話番号とか、価格とか、メールアドレスのように文字種や桁数などが限定される場合には、ぜひその形式になっているかをチェックすべきである。
しかし、住所欄や掲示板への投稿本文欄など、実質的に文字種を制限できない場合も多い。そのような場合には、入力チェックとしてできることは、せいぜい桁数(文字数)のチェックくらいである。
すなわち、そもそも入力チェック(文字種チェック)は出来る場合と出来ない場合があり、文字種チェックなどが脆弱性対策に(結果として)役立つ場合もあれば、役立たない場合もある。したがって、Webアプリケーションの場合、入力チェックは根本的な脆弱性対策にはなりえず、「やっておいた方がよい」という保険的な対応にすぎない。
まとめ
古典的なセキュアコーディングの原則の一つに入力値検証があることから、Webアプリケーションにおいても入力値検証が脆弱性対策の基本であるかのような説明を見かけることが多い。しかしバイナリデータを扱うミドルウェア等の場合、入力データの検証とはデータの枠組みの検査であって、入力データそのものの検査ではない場合が多い。一方、Webアプリケーションにおいて、入力値の検査は入力データそのものの検査を論じる場合が多い。ここにギャップがある。(Web)アプリケーションの場合、通常データの枠組みは解析済みであり、データそのものを受け取る。したがって、ミドルウェアなどの場合に比べて入力値検証の重要性は低い。一方、アプリケーションの場合、ビジネス要件に従った入力値検証は重要である。下表にこれらをまとめた。
| ミドルウェア | アプリケーション | |
| データの枠組みの検査 | 重要 | 重要でない |
| データそのものの検査 | 不要な場合が多い | 重要(*1) |
このように整理すると分かりやすい(上記は大まかな議論であって、もちろん例外はありえる)。
従来混乱しがちだった「Webアプリケーションのセキュリティ対策における入力値検証」の意味を考える出発点として、新しい視点を提案するものである。
とてもよくまとまってると思います。では、現状はどうなのか、というのを過去〜最近の経験から。 「住所欄や掲示板への投稿本文欄など、実質的に文字種を制限できない」値に対してはエスケープするなど正しく実装されてることが多いです。それが要件として明記されている..
徳丸浩の日記 - そろそろ入力値検証に関して一言いっとくか - Webアプリケーション脆弱性対策としての入力値検証についてを表示したところ、以下のようになりました。 入力値検証の話題から、細菌対策の広告を出すとは・・・
XSS対策:JavaScriptのエスケープ(その3) - ockeghem(徳丸浩)の日記にて、JavaScriptのリテラルを動的生成する場合のエスケープ方法について検討したが、id:hoshikuzuさんから、考慮がもれているという指摘を受けた(2007-10-11 - hoshikuzu | star_dust の書斎 - JavaScript
- http://www.tokumaru.org/ ×122
- http://d.hatena.ne.jp/teracc/20070908 ×64
- http://d.hatena.ne.jp/ockeghem/20071021/1192986523... ×59
- http://www.devnull.jp/tdiary/20070907.html ×23
- http://www.devnull.jp/tdiary/ ×18
- http://bakera.jp/ebi/topic/2986 ×17
- http://www.hash-c.co.jp/ ×16
- http://www.saiyasuweb.com/item/Gストリング ×15
- http://b.hatena.ne.jp/entry/http://www.tokumaru.or... ×12
- http://d.hatena.ne.jp/teracc/ ×11
- http://d.hatena.ne.jp/ikepyon/ ×11
- http://d.hatena.ne.jp/ockeghem/ ×11
- http://hsj.jp/ ×10
- http://d.hatena.ne.jp/ockeghem/20071010/1191985572... ×10
- http://d.hatena.ne.jp/ikepyon/20070905 ×8
- http://r.hatena.ne.jp/pero1/セキュリティ/ ×8
- http://b.hatena.ne.jp/entrylist?sort=hot&of=100&th... ×8
- http://tokumaru.org/ ×7
- http://www.tokumaru.org/was/ ×6
- http://b.hatena.ne.jp/HiromitsuTakagi/ ×6
- http://b.hatena.ne.jp/keyword/サニタイズ ×6
- http://www.about-reference.com/php/pear_manual/pac... ×5
- http://a.hatena.ne.jp/ikepyon/ ×5
- http://b.hatena.ne.jp/ockeghem/ ×5
- http://a.hatena.ne.jp/yamagata21/ ×5
- http://erokey.ddo.jp/diary2/一般/一般常識問題/20070901.htm... ×5
- http://b.hatena.ne.jp/t/SQLインジェクション ×5
- http://d.hatena.ne.jp/into_the_blue/20070906 ×4
- http://blog.chew.jp/result/sql データベース名/sql データベース名... ×4
- http://labs.ceek.jp/hbnews/list.cgi ×4
- http://d.hatena.ne.jp/teracc/searchdiary?word=*[セキ... ×4
- http://bakera.jp/ebi ×4
- http://b.hatena.ne.jp/keyword/SQLインジェクション ×3
- http://www.hash-c.co.jp/index.html ×3
- http://www.saiyasuweb.com/3/c/テーブル ×3
- http://d.hatena.ne.jp/ockeghem/20071021 ×3
- http://b.hatena.ne.jp/naoya/favorite ×3
- http://www.devnull.jp/tdiary/200709.html ×3
- http://reader.livedoor.com/subscribe/http://www.to... ×3
- http://b.hatena.ne.jp/entrylist?url=http://&sort=h... ×3
- http://b.hatena.ne.jp/entrylist?sort=hot&of=50&thr... ×3
- http://d.hatena.ne.jp/teracc/searchdiary?of=10&wor... ×3
- http://a.hatena.ne.jp/mao140/ ×2
- http://twitter.com/ockeghem ×2
- http://b.hatena.ne.jp/TAKESAKO/20070905 ×2
- http://b.hatena.ne.jp/keyword/TCP/IP ×2
- http://b.hatena.ne.jp/t/security ×2
- http://b.hatena.ne.jp/entrylist?sort=hot&of=100&th... ×2
- http://www.tokumaru.org ×2
- http://b.hatena.ne.jp/n2s/ ×2
- http://d.hatena.ne.jp/tessy/ ×2
- http://labs.ceek.jp/hbnews/ ×2
- http://d.hatena.ne.jp/ockeghem/?of=5 ×2
- http://b.hatena.ne.jp/entrylist?sort=hot&of=50&thr... ×2
- http://a.hatena.ne.jp/harupu/ ×2
- http://b.hatena.ne.jp/entry/5788968 ×2
- http://labs.ceek.jp/hbnews/list.cgi?p=1&c=0 ×2
- http://b.hatena.ne.jp/TAKESAKO/ ×2
- http://www.pluto.dti.ne.jp/~rinou/ ×2
- http://hsj.jp/junknews/ ×2
- http://d.hatena.ne.jp/ockeghem/?of=25 ×2
- http://d.hatena.ne.jp/ockeghem/searchdiary?of=5&wo... ×2
- http://www.tokumaru.org/index.htm ×1
- http://d.hatena.ne.jp/into_the_blue/ ×1
- http://r.hatena.ne.jp/nkeen/A07-IT-Hacker/ ×1
- http://momegoto.g.hatena.ne.jp/ ×1
- http://d.hatena.ne.jp/ikepyon/searchdiary?word=*[セ... ×1
- http://d.hatena.ne.jp/ockeghem/searchdiary?word=ブラ... ×1
- http://fastladder.com/subscribe/http://www.tokumar... ×1
- http://del.icio.us/otsune?page=16 ×1
- http://r.hatena.ne.jp/yoiIT/受信箱/http://r.hatena.ne... ×1
- http://hsj.jp/index.html ×1
- http://d.hatena.ne.jp/keyword/XSS脆弱性 ×1
- http://d.hatena.ne.jp/ockeghem/searchdiary?word=XS... ×1
- http://sideblue.net/archives/category/it ×1
- http://d.hatena.ne.jp/ockeghem/searchdiary?word=文字... ×1
- http://d.hatena.ne.jp/ockeghem/20070727/1185506417... ×1
- http://d.hatena.ne.jp/ikepyon/200709 ×1
- http://d.hatena.ne.jp/ockeghem/20070519/1179592129... ×1
- http://d.hatena.ne.jp/ockeghem/20070524/1179979230... ×1
- http://www.devnull.jp/tdiary/index.rb ×1
- http://cglove.com/837283628367837D8362837689E6919C... ×1
- http://tokumaru.org/was/ ×1
- http://d.hatena.ne.jp/ockeghem/?of=20 ×1
- http://d.hatena.ne.jp/ockeghem/?of=30 ×1
- http://sideblue.net/ ×1
- http://d.hatena.ne.jp/ockeghem/20071010 ×1
- http://tinyurl.com/preview.php?num=25ebpl ×1
- http://d.hatena.ne.jp/otsune/ ×1
- http://d.hatena.ne.jp/tessy/20070827/1188221438 ×1
- http://r.hatena.ne.jp/NSR250R-SP/受信箱/ ×1
- http://d.hatena.ne.jp/ikepyon/comment?date=2007090... ×1
- http://d.hatena.ne.jp/ockeghem/20070528 ×1
- http://d.hatena.ne.jp/tessy/20070827 ×1
- http://www.about-reference.com/php/manual/language... ×1
- http://labs.ceek.jp/hbnews/list.cgi?k=5&p=3&c=0 ×1
- http://tokumaru.org/d/20070909.html ×1
- http://d.hatena.ne.jp/ockeghem/?date=20071010&guid... ×1
- http://d.hatena.ne.jp/tessy/20070902 ×1
- http://del.icio.us/network/lomo ×1
- 入力値検証 ×4 / PHP 入力値エスケープ ×3 / 入力値チェック PHP SQL ×3 / JavaScript 入力値検証 ×3 / Webアプリケーション 入力 チェック ×3 / php POSTデータ検査 ×3 / 桁数 検証 ×2 / ウェブアプリケーションセキュリティ 金床 ×2 / web 入力値検証 ×2 / 徳丸 脆弱性 ×2 / そろそろ 一言いっとくか ×2 / 検証 SQL 入力 ×2 / DbCommandBuilder ×2 / html 入力値制限 ×2 / 入力の検証 ×2 / 電話番号 桁数 チェック php ×2 / XSS select 改ざん ×2 / XSS 対策 入力チェック サニタイズ ×2 / 入力検証 ×2 / MDB2 quote sqlインジェクション ×2 / Webアプリケーション PRG ×2 / SQL バイナリデータ 中身 ×1 / SQL 入力検証 ×1 / php 3の倍数 ×1 / SQL文 web 脆弱性 ×1 / +が_に 入力 ×1 / aspx 画像 バイナリ ×1 / javascript データのチェックライブラリ ×1 / quote mdb2 エスケープ ×1 / パラメータの改ざん 対策 ×1 / 入力値|外部からの値 検証項目 ×1 / 一般的なWEBアプリケーションのセキュリティ対策 ×1 / javascript escape 脆弱性 ×1 / web 入力チェック ×1 / 値 検証 ×1 / DOS パラメータ 入力チェック ×1 / IEEE754 バイナリ ×1 / 暗黙の型変換 実行速度 ×1 / 強制ブラウジング 図 ×1 / javascript 入力値の検査 数値 ×1 / WEB 入力チェック ×1 / 文字種チェック ×1 / 徳丸 意味 ×1 / webアプリ 入力チェック ×1 / セキュアコーディング ×1 / 検証の重要性 ×1 / php 入力文字数のチェック機能 ×1 / URL置換 脆弱性対策 ×1 / javascript 入力値チェック 分岐 ×1 / SQl 文字種チェック ×1 / 徳丸 web ×1 / バイナリ形式 桁数 ×1 / 入力値チェック webアプリケーション ×1 / FORM入力の検証 ×1 / 画像 XSS ×1 / SQL 入力 チェック ×1 / break ×1 / www.go-pear.org ×1 / ウェブアプリケーション セキュリティ 根本的 ×1 / web 脆弱性検証 ×1 / 入力検証 \\d* ×1 / javascript 動的生成 脆弱性 ×1 / SQLインジェクション 入力値 チェック 重要性 ×1 / pear 入力値検証 ×1 / ミドルウェアとアプリケーション ×1 / イメージ検証入力 ×1 / WEBアプリケーション セキュリティ対策 ×1 / パラメータの改ざん 表示画像 ×1 / xss ×1 / POSTデータ 改ざん バリデート ×1 / SQLインジェクション対策 PHP サンプル ×1 / webアプリ 入力値チェック ×1 / javascript 入力値 検査 ×1 / WEBアプリケーション 入力性 ×1 / バインドメカニズム ruby ×1 / javascript 入力データ検証 ×1 / hidden値 escape 必要 ×1 / php 桁数チェック ×1 / 入力値 ×1 / ruby 値 受け取る ×1 / SQLインジェクション webアプリケーション ミドルウェア ×1 / WAF CSRF ×1 / web アプリケーション 入力値 チェック ×1 / HTML 入力値 セキュリティ ×1 / api パラメタ改ざん ×1 / SQL文の作り方 ×1 / mdb2 バイナリーデータ ×1 / POST 検査 入力値 ×1 / Webアプリケーション 脆弱性検査 文字列 エスケープ ×1 / Webアプリ 入力値チェック ×1 / javascript 入力制限 文字種 ×1 / パラメータの改ざん 例 ×1 / web xss sql ×1 / カラム名に予約語 ×1 / バインドメカニズム サニタイズ ×1 / php 文字種チェック ×1 / javascript 入力チェック 桁数 ×1 / javascript 入力 桁数チェック ×1 / hidden値 検証 ×1 / php 文字種 チェック ×1
| SQLインジェクション対策はおすみですか? 開発開始時点からのコンサルティングから、公開済みWebサイトの脆弱性検査、 脆弱性発見後の適切な対策まで |
例えば、バインドメカニズムがまだ広まっていなかった頃、数値型の SQLインジェクションを防ぐためには、入力値検証を行うのが一般的(*1)でしたから、“セキュリティの要請”として入力値検証が重要な面もあったかも知れませんね〜。
*1) 型変換も一般的でしたけどw
s/上手/上図/ ですね :-)
私も基本はエスケープという考え方を広めていきたいと思っています。ただし、少し複雑なアプリケーションを作ろうとすると、「例外」という問題が発生する可能性が非常に高いのでは?と感じることが多くなってきました。最も単純と思われるSQLInjectionの対策にしても、バインドメカニズムが使用出来ないケースは存在します。
エスケープだけでは防げないセキュリティの問題は存在していて、それらは入力値検証によって対策しなければならないという現実もあります。その部分のフォローが必要かなと感じる今日この頃です。
また、セキュリティの要請ではなく業務要件という考え方は、いまいち理解出来ません。例えば、パラメータの改ざんにより本来アクセス出来てはならないデータにアクセス出来てしまう問題があったとしたら、それは一般的なセキュリティの問題ではないということでしょうか?
>エスケープだけでは防げないセキュリティの問題は存在していて、
>それらは入力値検証によって対策しなければならない
そんなケースはないと思いますよ。
>パラメータの改ざんにより本来アクセス出来てはならないデータにアクセス出来てしまう問題
業務要件としてのセキュリティ要件であって、セキュアコーディングの話ではないですね。
突っ込みありがとうございます>みなさま
ミドルウェアとアプリケーションの役割は、すぱっと分かれるわけでもないので、ここで書いた話が絶対不変の真理などというつもりはありません。こう考えたら、おおむねうまく整理できるのではないかという提案です。
あと、入力値検証というのはあくまで入力値を受け取った時点での検証であって、その値を使う時にチェックしなければならないケースはあり得ると思います。例えば、リダイレクタに指定するURLに改行が入っていないかとかです。しかしこれとて、リダイレクタのAPI側で本来チェックすべきものだと思います。
>そんなケースはない・・・
お、そうなんですか。私が知らない間に、言語は進化しているようですね。
>業務要件としてのセキュリティ要件・・・
なるほど、セキュリティ要件を書く人は大変ですね。
masaさん
意見を述べるなら根拠を出して表明するようにしませんか。
「俺は知ってるけどお前ら知らないから言えるんじゃ」
と言われているようにしか聞こえません。
masaさんの方が間違っているかもしれないのに、
手の内を明かさないのでは誰も否定できません。
続けて。
masaさん曰く
「パラメータの改ざんにより本来アクセス出来てはならないデータにアクセス出来てしまう問題があったとしたら、それは一般的なセキュリティの問題ではないということでしょうか?」
意味がわからないです。「パラメータの改ざん」と言ったらなんでもそれに該当しますし、「本来アクセスできてはならないデータにアクセスできてしまう」と言ったら、セキュリティ上の脅威のCIAのCの全部がそれなのですから、何のことを指しておっしゃているのか特定できません。
そんな質問をしておきながら、「それは一般的なセキュリティの問題ではないということでしょうか?」なんて「そんなわけねーだろ」と言わんばかりの質問を投げかけるのは失礼です。
> 別の通りすがりさん
すいません、例をあげます。
SQLInjectionでしたら、
"select * from usertable order by " + columName + asc_or_desc;
といったコードでSQLInjectionの問題が発生した場合です。クォートされない個所への文字列挿入全般の問題なので、他にも例はたくさんあるかと思います。
このような問題にエスケープで対策出来ますか?分岐ロジックを書いていますか?実際は、入力値検証で対応するのではないですか?
と、いうことです。
パラメータの改ざん・・・のくだりは、アクセス認可と置き換えればよいでしょうか。
http://www.ipa.go.jp/security/awareness/vendor/programmingv2/contents/102.html
ここの図5の三段階目の入力値検証をイメージしています。
カラム名なら「"」で括るのが正しいですよ。
カラム名に予約語があったら困るので当然そういう文法が言語に用意されているわけです。http://www.postgresql.jp/document/pg653doc/j/user/syntax.htm
「asc_or_desc」のところは、そういうSQL文の作り方をしちゃいけないってだけです。入力のところでvalidationしなければならないってものではありません。
図5のものは、今ここで言われている validation のことではありませんね。脆弱性対策というより元々仕様で要求されているセキュリティ機能の一つでしょう。それが見えてはいけないことが仕様で定義されていなければこれが欠陥なのかどうか機械的判別はできないわけで、検査者の視点で言えば、検査者はそのとき「見えてはいけないもの」だということを常識で判断するのでしょう。それは仕様に欠陥があるという話で、ここで言われている脆弱性対策としての validation とは関係がないでしょう。
別の通りすがりさん
> カラム名なら「"」で括るのが正しいですよ。
恥ずかしながら知りませんでした、ありがとうございます。この指定については少し調査してみます。
asc_or_descの問題は「そういうSQL文の作り方をしちゃいけない」と言われると、「検証済ならよいのでは?」と思ってしまいます。ただ、入力値検証で対応しなければならない訳ではないですね。すいません。
>・・・それは仕様に欠陥があるという話で、・・・
常識で判断出来ることを仕様に書かなければならないとしたら、「セキュリティ要件を書く人は大変ですね。」という感想に変わりはありません。
私は、hidden値やselectboxで選択された値など、ユーザに悪意がない限り通常は改ざんされることがないデータが改ざんされ、その結果システムに悪影響が及ぶならば、業務要件がどうであれセキュリティの欠陥だと考えていますので。
フレームを発生させてしまい申し訳ありません。
カラム名のクォートについて調査した結果だけ報告して最後とさせて頂きます。
PostgreSQL8.2.4で検証したところ、「"」によって「"」がエスケープされますので、SQLInjection対策として有効であることが確認できました。テーブル名やカラム名に含まれる特殊文字のエスケープに(データベース名にも)適用が可能な、エスケープ関数が用意されている言語も多いようです。
http://www.go-pear.org/manual/ja/package.database.mdb2.intro-quote.php
http://msdn2.microsoft.com/ja-jp/library/system.data.common.dbcommandbuilder.quoteidentifier(VS.80).aspx
asc_or_descの部分は、「動的な入力値をそのまま使用して構文を組み立ててはいけない」と、理解することが出来ました。
不快な思いをさせてしまった方には深くお詫び致します。