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

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

ただいまの
回答率

90.84%

  • PHP

    18687questions

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

  • JavaScript

    14849questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML

    8080questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

  • jQuery

    6169questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

  • CSS

    5160questions

    CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

セキュリティ上問題があるかを教えてください。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 344

gomatan1258

score 34

index.html

<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>タイピングサイト</title>
<link rel="stylesheet" href="home.css">
</head>
<body>

<div id="moji">
下記設定を選んでタイピング練習をしましょう!
</div>
<div id="container">
<form action="index.php" method="get">

    <div class="second">
        <p><input type="radio" name="second" value="30">30秒</p>
        <p><input type="radio" name="second" value="60"checked>60秒</p>
        <p><input type="radio" name="second" value="90">90秒</p>
        <p><input type="radio" name="second" value="120">120秒</p>
    </div>
    <div class="method">
        <div><p><input type="radio" name="home" value="homePosition" checked>ホームポジションに戻る練習</p></div>
        <div><p><input type="radio" name="home" value="normal">ノーマルで練習</p></div>
    </div>
    <div class="position">
        <p><input type="radio" name="position" value="tyuudann" checked>中段</p>
        <p><input type="radio" name="position" value="joudann">上段</p>
        <p><input type="radio" name="position" value="gedann">下段</p>
        <p><input type="radio" name="position" value="hitosasiyubi">人差し指</p>
        <p><input type="radio" name="position" value="all">全ポジション</p>
    </div>
        <p><input type="submit" value="送信"></p>

</form>

</div>
</body>
</html>


home.css

.second {
    width: 300px;
    padding-left: 30px;
    background-color: skyblue;
}

.position {
    width: 300px;
    padding-left: 30px;
    background-color: pink;
}

.method {
    width: 300px;
    padding-left: 30px;
    background-color: #ccc;
}

#container {
    width: 300px;
    margin: 0 auto;
}

#moji {
    width: 300px;
    margin: 0 auto;
}

index.phpはjavascriptの部分一部だけ表示します。

if(isset($_GET["home"]) && $_GET["home"]=="homePosition") {
        ?>
        fjs = "fj";//javascriptの関数の外で定義しています。
        <?php 
    }elseif(isset($_GET["home"]) && $_GET["home"]=="normal") {
        ?>
        fjs = "";
        <?php 
            }
        ?>

こちらはホームポジションに戻る練習のためにfjを挿入するかどうか決めています。

time = <?php if(isset($_GET["second"])) {//timeもjavascriptの関数の外で定義しています。
            echo $_GET["second"];
        }

こちらはtime30秒にするか60, 90, 120秒にするかを決めています。

<?php if(isset($_GET["position"]) && $_GET["position"]=="tyuudann") { ?>
            for(var i=0; i<100; i++) {
                words.push(createPart("asdghkl"));//javascriptのコードです。
            }
        <?php } ?>

        <?php if(isset($_GET["position"]) && $_GET["position"]=="joudann") { ?>
            for(var i=0; i<100; i++) {
                words.push(createPart("qwertyuiop"));//javascriptのコードです。
            }
        <?php } ?>
        <?php if(isset($_GET["position"]) && $_GET["position"]=="gedann") { ?>
            for(var i=0; i<100; i++) {
                words.push(createPart("zxcvbnm,."));//javascriptのコードです。
            }
        <?php } ?>

        <?php if(isset($_GET["position"]) && $_GET["position"]=="hitosasiyubi") { ?>
            for(var i=0; i<100; i++) {
                words.push(createPart("rvtgbyhnum"));//javascriptのコードです。
            }
        <?php } ?>

        <?php if(isset($_GET["position"]) && $_GET["position"]=="all") { ?>
            for(var i=0; i<100; i++) {
                words.push(createPart("qwertyuiopasdghklzxcvbnm,."));//javascriptのコードです。
            }
        <?php } ?>

こちらは上中下段人差し指のどれを練習するかを決めています。

タイピングソフトを作っています。
上記のようにhtmlからフォームのラジオボタンのみを使って、index.htmlからindex.phpのscriptタグの中のjavascriptにphpのコードを挿入してフォームの値を渡しています。ちゃんと動くのですが、セキュリティ上問題はないでしょうか?phpはまだ素人です。分かりにくい質問で申し訳ないのですが、よろしくお願いします。

更新しましたが、まだ見にくいかと思います。申し訳ありません。
当方の目的はinput type="text" ではなくinput type="radio"でもセキュリティ上安全どうかを知りたいです。index.php貼ろうとしましたが文字数が超えて入力失敗しました。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • t_obara

    2017/10/03 10:43

    ここでセキュリティの問題がないと言われたとして、安心してサービスを提供されるのでしょうか?セキュリティ上の問題を指摘され、修正して安心できるのでしょうか?

    キャンセル

  • gomatan1258

    2017/10/03 11:01

    自分にはまだphpを使う資格がないということですね。どんな被害に合うとか、逆にどういうことをやって被害を与えられるとかもわからないので、もうちょっと勉強してみます。

    キャンセル

  • 退会済みユーザー

    2017/10/03 11:15

    複数のユーザーから「やってほしいことだけを記載した丸投げの質問」という意見がありました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 1

checkベストアンサー

+4

ソースの全容がつかめないのでコメントしづらいですが、クエリ文字列 second に 1; alert(1) と入力してみてください。アラートが表示されませんか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/03 10:53

    申し訳ございません。クエリ文字列のsecondに1とはURLのsecond=1にするということでしょうか?先程やってみたら、1秒で終わる仕様になりました。alert(1)とはどこに書くのでしょう?

    キャンセル

  • 2017/10/03 11:18

    違います。second=1;alert(1) ですが、セミコロンをパーセントエンコードして、second=1%3balert(1) が確実かと思います

    キャンセル

  • 2017/10/03 11:24

    ご返答ありがとうございます。1がアラートで出力されました。

    キャンセル

  • 2017/10/03 11:31

    ですよね。つまり、セキュリティ上問題があるということです。一種のクロスサイトスクリプティングです

    キャンセル

  • 2017/10/03 11:34

    なるほど!こういうのを全然知りませんでした。勉強になりました。ありがとうございます。

    キャンセル

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

  • ただいまの回答率 90.84%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • PHP

    18687questions

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

  • JavaScript

    14849questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML

    8080questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

  • jQuery

    6169questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

  • CSS

    5160questions

    CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。