Undefined indexエラーを解決したい

Question

過去に作ったCMSでエラーが出るようになってしまいました。
パスワードを変更する画面なのですが、
下記ファイルでUndefined indexエラーが表示されてしまいます。
（Notice: Undefined index: pw_id in /home/users/0/●●●●●●●/update.php on line 33）

13行目のmysql_real_escape_string($_POST['pw_id']));
のところのようなのですが、具体的にどのように記述すれば良いか
ご教示いただけないでしょうか。
どうぞよろしくお願いいたします。

以前はエラーが出ていなかったのに、
出るようになってしまうというのは、
サーバーの仕様変更なのでしょうか？

	<?php

	if(!empty($_POST)){
		if($_POST['pw'] and $_POST['pw_new'] and $_POST['pw_new2']){
			if($_POST['pw_new']==$_POST['pw_new2']){
				$sql=sprintf('SELECT * FROM password WHERE pw_id=0');
				$OLD_PASS=sha1(mysql_real_escape_string($_POST['pw']));
				$record=mysql_query($sql) or die(mysql_error());
				$PASS=mysql_fetch_assoc($record);
				$SQL_PASS=htmlspecialchars($PASS['pw']);

				if($OLD_PASS==$SQL_PASS){
					$sql_up=sprintf('UPDATE password SET pw="%s" WHERE pw_id=0',
						sha1(mysql_real_escape_string($_POST['pw_new'])),
						mysql_real_escape_string($_POST['pw_id'])
					);
					mysql_query($sql_up) or die(mysql_error());
					echo '<p>パスワードを変更しました。</p>';
				}
				else{echo '<p>現在のパスワードが違います。</p><input type="submit" class="SEND" onClick="history.back()" value="戻る" />';}
			}
			else{echo '<p>新しいパスワードを確認してください。</p><input type="submit" class="SEND" onClick="history.back()" value="戻る" />';}
		}
		else{echo '<p>入力されていない項目があります。</p><input type="submit" class="SEND" onClick="history.back()" value="戻る" />';}
	}
	?>

Comments

[K_3578]

古すぎるものを直すのって1から作り直すより時間かかる気がする

[退会済みユーザー]

何をきっかけに発生するようになったのか、サーバーの仕様変更を疑うなら変更前と後の情報がないと判断できないよ、サーバーの仕様を確認してね。

Answer 1

そもそもPOST送信するデータの仕様が示されていないので他人にはわからん。

$_POST['pw']
$_POST['pw_new']
$_POST['pw_new2']
に加えて出現する
$_POST['pw_id']も含めて、それぞれの違いや機能や目的がわからん。

mysql_real_escape_string()やmysql_query()とか
sha1()とか
今どき滅んだ古いやり方なところはツッコミ禁止でしょうか。

Answer 2

過去に作ったCMSで

でしたら作った人に確認されたほうが良いと思います。
エラーは、エラーに書いてあるとおり、配列に指定のインデックス名（キー名）がないことによるものですが、じゃあ、「そのキーが存在したときだけ処理」のように単純に変更するだけで良いとは限りません。

リクエストを生成するフォームなどあるなら、そちらを見直したり、仕様変更に対応できてるかどうかを調査する必要があるかと思います。

Comments

[m.ts10806]

>mysql_query

よほど古い実装なんでしょうね。
PHP7から削除された機能で実装されています。

Answer 3

ここんとこ

                    $sql_up = sprintf('UPDATE password SET pw="%s" WHERE pw_id=0',
                        sha1(mysql_real_escape_string($_POST['pw_new'])),
                        mysql_real_escape_string($_POST['pw_id'])
                    );

sprintf のフォーマットでは置換対象がひとつしかないけど、値はふたつ指定してますよね。
なんかソース修正ミスってんじゃないの?
$_POST['pw_id'] も実際になさそげだし。