php не нравится del

[ilushka2306]

в 59 строке не нравится if($_GET['del']) delete_users();

PHP код:

<title></title>
<?php
function show_users()
{
?>
<SCRIPT language='JavaScript1.1' type='text/javascript'>
<!--
function Delete(N)
{
if(confirm("Удалить пользователя?"))
{
parent.location='?del='+N;
}
else
{
return false;
}
}
-->
</SCRIPT>
<?php 
require './connect.php';

echo '<table align="center"><tr><td width="70" align="left"><font color="FF0000"> ID </font></td><td width="130" align="left"><font color="FF0000"> Имя </font></td> <td width="130"><font color="FF0000"> Пароль </font></td><td width="130"><font color="FF0000"> Доступ </font></td></tr>';
$result = mysql_query("SELECT * FROM test ORDER BY id;");
while($row = mysql_fetch_array($result))
{
echo '
<tr>
<td>
'.stripslashes($row['id']).'
</td>
<td>
'.stripslashes($row['name']).'
</td> 
<td>
'.stripslashes($row['password']).'
</td>
<td>
'.stripslashes($row['access']).'
</td> 
<td align="center">
<a href="#" OnClick="Delete('.$row['id'].')"> удалить </a> 
</td>
</tr>';
}
echo '</table>';
}
function delete_users()
{
require './connect.php';

// Подключение к СУБД
$link = mysql_connect(HOST,USER,PASS) or die (mysql_error());  
        
 $query = "DELETE FROM test WHERE id = '".$_GET['del']."';";
 mysql_query($query);
}

if($_GET['del']) delete_users();
show_users();
?>

[Johnatan]

Замените на

PHP код:

if(array_key_exists('del', $_GET)) 


Кстати, что будет если в качестве аргумента $_GET['del'] передать ' OR '1'='1

З.Ы. Интересно сколько человек прочитав вопрос в первую очередь подумали о подавлении вывода ошибок?

[ADSoft]

ошибка говотрит о том.. .что в $_GET нет индекса del ... то есть не передали его через этот метод
советовал бы использовать intval()

[graymaster]

Почините

PHP код:

if ($_GET['del']) delete_users(); 


на что-то вроде

PHP код:

if (isset($_GET['del']) && intval($_GET['del']) delete_users(); 


позаботьтесь о том, что бы у вас не было пользователя с айди=0

И Никогда, вы слышите, никогда (!!!) не делайте валидацию данных, тогда ваша база данных будет обязательно периодически обновляться, кроме того, вас и ваших пользователей в будущем будет ждать множество интереснейших сюрпризов от любителей SQL injection