質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

解決済

3回答

3093閲覧

SQL文に変数を書くとセキュリティ上よくないのでしょうか?

TODS

総合スコア13

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

1グッド

1クリップ

投稿2018/10/02 04:42

テーブル名の指定に変数を使用

テーブル名の指定に変数を使いましたが、SQLインジェクション攻撃というものがあるのを知り(理解はできていません)、質問させて頂きました。
引数が数値の場合だけ問題になるのでしょうか?

PHP

1 $tbl_name = $_POST['select_tbl']; 2 $sql = 'SELECT * FROM '.$tbl_name.''; 3 $stmt=$pdo->query($sql);

試したこと

バインド機構というものを利用してみようと思いましたが、動きませんでした。
カラムの値に使用するものと知り、違う書き方があれば教えて下さい。
よろしくお願い致します。

PHP

1 $tbl_name = $_POST['select_tbl']; 2 $sql = 'SELECT * FROM :tbl_name'; 3 $stmt=$pdo->prepare($sql); 4 $stmt->bindParam(':tbl_name', $tbl_name, PDO::PARAM_STR); 5 $stmt->execute();
digitalhimiko👍を押しています

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

この手の処理がどうしても必要な場合はホワイトリストをつくっておけば
クライアントから送られてくる攻撃的なデータも回避できます
カラム名やテーブル名はprepareで渡せません

PHP

1$table_list=["select_tbl1","select_tbl2","select_tbl3"]; 2$tbl_name = filter_input(INPUT_POST,'select_tbl'); 3if(in_array($tbl_name,$table_list)!==false){ 4 $sql = "SELECT * FROM ${tbl_name}"; 5 print $sql; 6 // $stmt=$pdo->query($sql); 7} 8?> 9<form method="post"> 10<label><input type="radio" name="select_tbl" value="dummy">dummy</label> 11<label><input type="radio" name="select_tbl" value="select_tbl1">1</label> 12<label><input type="radio" name="select_tbl" value="select_tbl2">2</label> 13<label><input type="radio" name="select_tbl" value="select_tbl3">3</label><br> 14<input type="submit" value="go"> 15</form>

投稿2018/10/02 05:01

yambejp

総合スコア114572

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

TODS

2018/10/02 07:14

具体的な方法についてもお教えて頂きありがとうございます。勉強します。
guest

0

仮に、その$select_tblに、ユーザーのアカウント情報が格納されているuserなどが送られてきた場合を考えて下さい。

IDlabelpassword
1kunaikunai123

これをselect *しているわけですから、画面には全ユーザーのログイン情報が表示されてしまうわけです。
悪意のある第三者がこの画面を開いたら、エライことになるのは想像に難くないでしょう。

投稿2018/10/02 04:57

kunai

総合スコア5405

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

TODS

2018/10/02 07:13

ご説明頂きありがとうございます。他の方法を考えなければならいですね。ありがとうございます。
guest

0

はい。セキュリティ上はよろしくありません。
なぜなら、攻撃者からは $_POST['select_tbl'] にブラウザで任意の文字列を指定できるからです。

バインド機構は認識の通りです。テーブル名等には通常使えません。
どうしてもテーブル名に外部から取得する変数を使いたいのであれば、危険な文字列がないかどうかをチェックし、危険な文字列があれば拒否する、もしくは文字列をエスケープして使用するといった方法になります。

いくつか方法がありますので、PHP, SQL, エスケープなどで調べてみると良いと思います。

投稿2018/10/02 04:50

ssasaki

総合スコア1167

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

TODS

2018/10/02 07:10

ありがとうございます。教えて頂いた内容調べてみます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問