Czy ten kod jest dobry i optymalny? Opcje

[Balus]

[PHP] pobierz, plaintext 
<?php
$haslo = 'tajnehaslo';
$f_haslo = $_POST['haselko'];
$p_tmp = $_FILES['pliczek']['tmp_name'];
$p_nazwa = $_FILES['pliczek']['name'];
$p_rozmiar = $_FILES['pliczek']['size'];
// $p_typ = $_FILES['pliczek']['type'];
// $p_blad = $_FILES['pliczek']['error'];
 
if($f_haslo != $haslo)
{
	echo 'Niepoprawne haslo!';
}
else
{
	if(is_uploaded_file($f_tmp))
	{
		move_uploaded_file($p_tmp, '/pliczki/'$p_nazwa);
		echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
	}
	else
	{
		echo 'Blad podczas wrzucania pliku!';
	}
}
?>		
[PHP] pobierz, plaintext 

Tak jak w temacie (IMG:style_emoticons/default/winksmiley.jpg)
Czy ten kod jest dobry i optymalny?
Szczególnie chodzi mi o move_uploaded_files

Ten post edytował Balus 30.11.2009, 15:30:25

[darko]

Nie mogę coś edytować, więc napiszę nowego posta. Popracowałbym na tym
1. $f_haslo = $_POST['haselko']; - w ogóle nie filtrujesz danych z formularza
2. przed move_uploaded_file sprawdzałbym za pomocą ini_get czy $p_rozmiar jest mniejszy/równy maksymalnemu dopuszczalnemu i jesli jest, dopiero move_uploaded_file

poprawcie mnie jesli zle mowie

Poprawiam: źle mówisz, sam sprawdź

Ten post edytował darko 30.11.2009, 15:52:22