Forum PHP.pl

Witam. Napisałem taki skrypt.
Czy on jest bezpieczny (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[PHP] pobierz, plaintext 
<?php
  $errors = 0;
 
  if (
	empty($aaa) or 
	empty($bbb) or 
	empty($ccc) or 
	empty($ddd)
  {
	  echo 'Wypelnij wszystkie pola<br />';
	  $errors = 1;
  } 
 
  $exist_topsite = mysql_query("SELECT 1 FROM test WHERE Test22 ='".$aaa."'");
 
  if (0 ==  mysql_num_rows($bbb))
  {
		echo 'Topka o wybranej nazwie nie istnieje<br />';
		$errors = 1;
  }
 
	if (!empty($ccc) and !preg_match('@^http://.*.(jpg|gif)$@i', $ccc))
	{
	  echo 'Adres loga musi zaczynać się od http:// i logo musi być rozszerzenia jpg, gif<br />';
 
	  $errors = 1;
 
	}	
 
	if (!empty($ddd) and !preg_match('@^http://.*.(jpg|gif|png)$@i', $ddd))
	{
	  echo 'Adres buttona musi zaczynać się od http:// i logo musi być rozszerzenia jpg, gif lub png<br />';
 
	  $errors = 1;
 
	}
 
 
  if (strpos($eee, "@") == false)
  {
	  echo 'Podaj poprawny email<br />';
	  $errors = 1;
  }
  if (strcmp($fff, $ggg) != 0)
	{
		echo 'Zle wpisane potwierdzenie hasla<br />';
		$errors = 1;
	} 
 
  if (0 == $errors)
  {
	mysql_query("UPDATE ".$aaa."_test SET test4 = '".$iii."'");
 
		if (empty($passwd))
		{
		mysql_query("UPDATE test SET test1 = '".$aaa."', test3 = '".$ccc."'");
		}
		else 
		{
		mysql_query("UPDATE test SET test1 = '".$aaa."', test3 = MD5('".$ggg."'), test2 = '".$ccc."'");
		}
 
	if (1 == mysql_affected_rows())
	{
	  echo "<br /><center><h2><font color=green>OK</font></h2></center><br />";
	  $edited = 1;
	}
	 else
	{
	  echo "<br /><center><h2><font color=red>Error</font></h2><br /></center>";
	  $edited = 0;
	} 
 
  }
  else 
  {
	  $edited = 0;
  }
 
 
?>
[PHP] pobierz, plaintext 

Prosze o pomoc w zabezpieczeniu skryptu. Na serwerze mam włączone magic quote...

Ten post edytował Snap 18.03.2007, 14:21:36

Temat: SQL Injection Insertion

sugerował bym przejrzenie tego tematu gdyż zostało tam powiedziane dużo fajnych rzeczy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ja osobiście filtruje apostrofy z każdej danej wprowadzanej do strony za pomocą skryptu i w zapytaniach zamykam je w cudzysłowy. Myślę że to powinno wystarczyć. Jak ktoś coś niecnego wrzucić jako daną wejściową to wtedy zostanie wygenerowane błędne zapytanie i będę widział tą informację w logach strony.

musisz jeszcze dodać usuwanie TAGów taki jak JS bo wtedy ktoś sobie wpisze <script>...</script> w miejsce ... w pisze skrypt który może ci np pousuwać wszystko na serwie i takie tam