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

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

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

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

PHP

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

MariaDB

MariaDBは、MySQL派生のオープンソースなリレーショナルデータベースシステムです。 また、MySQLとほぼ同じデータベースエンジンに対応しています。

Q&A

解決済

1回答

813閲覧

PHPのパスワード認証について

ysk1118

総合スコア22

MySQL

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

PHP

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

MariaDB

MariaDBは、MySQL派生のオープンソースなリレーショナルデータベースシステムです。 また、MySQLとほぼ同じデータベースエンジンに対応しています。

0グッド

0クリップ

投稿2018/11/29 04:59

編集2018/11/29 06:39

ユーザ認証が必要なwebシステムを開発しています。
管理者ユーザはphpよりシステムにログインし、システムを利用するための
ユーザアカウントを作成しています。
作成時にパスワードを設定(通常の文字列)、DB登録時にpassword_hash関数を用いたハッシュ値を
パスワードとして登録、ユーザ認証画面でPassword_verify()にて認証を行う、という仕組みです。

発生している問題・エラーメッセージ

 ・ユーザ新規登録用のPHPで登録されたユーザ・パスワードでは、ログイン認証処理において
パスワード認証(password_verify())でNGとなってしまいます。(ユーザIDあるいはパスワードに誤りがあります。と表示)
・管理者権限を持ったユーザが別途用意してあり、管理者ユーザでwebシステムにログインすれば
パスワード変更画面から他のユーザ情報(パスワード)を変更できますが、
この画面でパスワードを変更してやると、正常に認証されるようになります。
(新規登録時と同じ文字列を指定しても認証OKとなる。)
・パスワードは、password_hash関数で与えた文字列から取得したパスワードのハッシュ値を
データベースに登録するようにしています。ソースを比較してみても、新規登録と変更で
特に違いがあるようには思えません

 上記のことから、新規登録画面と変更画面で何か処理を誤っているのかと思いましたが、
ソースを比較した限りでは、どちらもPassword_hashによって取得したハッシュ値をDBに登録してあり、
どうして新規登録直後はNGとなり、同じパスワードでも変更処理でOKになるのかがわかりません。
下記ソースに何か原因と考えられるミス等があればご教示いただけませんか。
よろしくお願いいたします。

該当のソースコード

1)ユーザ新規登録処理

PHP

1     // 入力したユーザID、パスワードを格納 2 $userid = $_POST["userid"]; 3 $username = $_POST["username"]; 4 $password = $_POST["password"]; 5 $scode = explode(" ",$_POST["sCode"]); 6 7 //useridの登録済み確認 '18.11.29 upd -- SQLインジェクション脆弱性対処 8 //$sql="select * from t_userdata where userid=" . $userid; 9 //$rows=$pdo->query($sql); 10 $res=$pdo->prepare("select * from t_userdata where userid=:userid"); 11 $res->bindValue(':userid',$userid); 12 $res->execute(); 13 14 //if($rows->rowCount() > 0){ 15 if($res->rowCount()> 0){ 16 $errorMessage=sprintf("ユーザーID:%sは既に登録されています。",$userid); 17 }else{ 18 // エラー処理 19 try { 20 $stmt = $pdo->prepare('INSERT INTO t_userdata(userid, name, password, scode) VALUES (:userid, :name, :passwd, :scode)'); 21 $stmt->bindValue(':userid',$userid); 22 $stmt->bindValue(':name',$username); 23 $stmt->bindValue(':passwd',password_hash($passwd,PASSWORD_DEFAULT)); 24 $stmt->bindValue(':scode',$scode[0]); 25 26 $stmt->execute(); 27 28 $signUpMessage = '登録が完了しました。登録IDは '. $userid. ' です。パスワードは '. $password. ' です。'; // ログイン時に使用するIDとパスワード 29 30 $userid=""; 31 $username=""; 32 $password=""; 33 $scode=""; 34 35 } catch (PDOException $e) { 36 $errorMessage=$e->getMessage(); 37 } 38 39 $stmt=null; 40 }

2)ユーザパスワード変更処理

PHP

1 // 入力したユーザID、パスワードを格納 2 $userid = $_GET["SNO"]; 3 $password = $_POST["password"]; 4 $scode = $_POST["section"]; 5 6 // 3. エラー処理 7 try { 8 $stmt = $pdo->prepare("update t_userdata set password=:passwd,sCode=:sCode,passwd_change=:passwd_change,term=:term where userid=:userid"); 9 $stmt->bindValue(':passwd',password_hash($password, PASSWORD_DEFAULT));// パスワードのハッシュ化を行う 10 $stmt->bindParam(':sCode',$scode); 11 $stmt->bindParam(':userid',$userid); 12 $stmt->bindValue(':passwd_change',null,PDO::PARAM_NULL); 13 $stmt->bindParam(':term',$_SESSION["term"]); 14 15 $stmt->execute(); 16 17 $good_msg = 'Update Successful!'; // ログイン時に使用するIDとパスワード 18 } catch (PDOException $e) { 19 $errorMessage = $e->getMessage(); 20 }

3)ユーザ認証処理

PHP

1       //入力されたユーザIDを格納 2 $userid=$_POST["userid"]; 3 4 // 2.ユーザIDとパスワードが入力されていたら認証する 5 $dsn = sprintf('mysql: host=%s; dbname=%s; charset=utf8',$db['host'],$db['dbname']); 6 7 // 3.エエラー処理 8 try { 9 $pdo=new PDO($dsn,$db['user'],$db['pass'],array(PDO::ATTR_ERRMODE=>PDO::ERRMODE_EXCEPTION)); 10 11 $stmt=$pdo->prepare('select * from t_userdata where userid=?'); 12 $stmt->execute(array($userid)); 13 14 $password=$_POST["password"]; 15 16 if ($row = $stmt->fetch(PDO::FETCH_ASSOC)) { 17 if (password_verify($password,$row['password'])){ 18 session_regenerate_id(true); 19 20 // 入力したIDのユーザ名取得 21 $id=$row['id']; 22 $res=$pdo->prepare("select * from t_userdata where id=:id"); 23 $res->bindValue(":id",$id); 24 $res->execute(); 25 $user=$res->fetch(PDO::FETCH_ASSOC); 26 27 $_SESSION["NAME"]=$user['name']; //氏名取得 28 29 $res=null; 30 31 header("Location: main.php"); // メイン画面へ遷移 32 exit(); // 処理終了 33 }else{ 34 // 認証失敗 35 $err_msg='ユーザIDあるいはパスワードに誤りがあります。'; 36 } 37 }else{ 38 // 該当データなし 39 $err_msg='該当データ無し'; 40 } 41 }catch (PDOException $e){ 42 $err_msg='データベースエラー'; 43 }

補足情報(FW/ツールのバージョンなど)

PHP 7.1.6
MariaDB 10.1.24

ユーザ情報格納テーブルの構造
イメージ説明

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

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

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

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

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

otn

2018/11/29 05:35

SQLインジェクション脆弱性がありますね。
ysk1118

2018/11/29 06:11

ご助言ありがとうございます。よろしければ、どの部分が該当箇所か、ご教示いただけませんか。
otn

2018/11/29 06:21

//useridの登録済み確認 の部分
ysk1118

2018/11/29 06:42

ありがとうございます。ソースの該当箇所を修正しました。
guest

回答1

0

ベストアンサー

php

1$password = $_POST["password"]; 2$stmt->bindValue(':passwd',password_hash($passwd,PASSWORD_DEFAULT));

おそらく、変数名の間違いが原因です。

投稿2018/11/29 05:05

maisumakun

総合スコア145123

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

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

退会済みユーザー

退会済みユーザー

2018/11/29 05:26

passwdとpasswordが混在して混乱してるだけなのだから、整理すればいいのに。
ysk1118

2018/11/29 05:55

ご指摘、ありがとうございました。 何度も見直したつもりでしたが、全く気づきませんでした。 本当にお恥ずかしい限りです。 m6u様の助言にも従い、変数名等整理をしたいと思います。 ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問