[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
Czy ten kod jest dobry i optymalny?
Szczególnie chodzi mi o move_uploaded_files

Tzn jeżeli jest to do jakiegoś Hostingu czy coś to trochę źle, bo ci ktoś może plik PHP wrzucić na serwer a potem rzeźnia - po stronie ;D Właściwie nawet jeżeli tylko ty z tego korzystasz powinieneś jakoś to zabezpieczyć. No i skąd wiesz że plik zostanie wrzucony?

[PHP] pobierz, plaintext 
@ $a=move_uploaded_file($p_tmp, '/pliczki/'$p_nazwa);
if($a==true) 
echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
else
echo 'Blad';
[PHP] pobierz, plaintext 

O jaką optymalność Ci chodzi? W takim krótkim kodzie ciężko cokolwiek spaprać. A jednak:
- niepotrzebnie tworzysz tyle zmiennych dla $_FILES,
- komunikat błędu nie sprawdza czy upload się powiódł, tylko czy jest plik do uploadu (dodaj analogiczny warunek dla funkcji move_uploaded_file).

[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'];
 
if($f_haslo != $haslo)
{
	echo 'Niepoprawne haslo!';
}
else
{
	if(is_uploaded_file($f_tmp))
	{
		$przenies = move_uploaded_file($p_tmp, $p_nazwa);
		if($przenies == true)
		{
			echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
		}
		else
		{
			echo 'Blad podczas przenoszenia pliku!';
		}
	}
	else
	{
		echo 'Blad podczas wrzucania pliku!';
	}
}
?>
[PHP] pobierz, plaintext 

A teraz co powiecie? :]

[PHP] pobierz, plaintext 
$przenies = move_uploaded_file($p_tmp, $p_nazwa);
if($przenies == true)
{
  echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
}
else
{
  echo 'Blad podczas przenoszenia pliku!';
}
[PHP] pobierz, plaintext 

zamieniłbym na

[PHP] pobierz, plaintext 
if(move_uploaded_file($p_tmp, $p_nazwa)) {
  echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
}
else {
  echo 'Blad podczas przenoszenia pliku!';
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
if(move_uploaded_file($p_tmp, $p_nazwa)) {
  echo 'Plik <strong>'.$p_nazwa.'</strong> ('.$p_rozmiar.') zostal wrzucony na serwer!';
}
else {
  echo 'Blad podczas przenoszenia pliku!';
}
[PHP] pobierz, plaintext 

if(move_uploaded_file($p_tmp, $p_nazwa)) {
pierw chyba trzeba php rozkazac przeniesc ten plik zeby potem sprawdzic czy zostal przeniesiony
poprawcie mnie jesli zle mowie

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


Poprawiam: źle mówisz, sam sprawdź

1.

[PHP] pobierz, plaintext 
$f_haslo = strip_tags(HTMLSpecialChars($_POST['haselko']));
[PHP] pobierz, plaintext 

Tyle wystarczy czy jeszcze jkaieś metody filtrowania polecacie?
2

[PHP] pobierz, plaintext 
if($p_rozmiar >= 666)
{
...
}
[PHP] pobierz, plaintext 

cos takiego?

Z wartości ini_get("upload_max_filesize") usuń literę "M" na końcu, przelicz sobie na bajty tę wartość, bo z tego, co pamiętam $_FILES["jakis_name"]["size"] zwraca rozmiar w bajtach i potem:
sprawdź czy $_FILES["jakis_name"]["error"] == 0 (czyli wszystko OK) i:

if($p_rozmiar <= $przeliczone_na_bajty && $_FILES["jakis_name"]["error"] == 0) {
move_uploaded_file...

maksymalny rozmiar wysyłanego pliku

Ok dla wszystkich pomógł
troszkę pokombinuję i odezwę się
Proszę o nie zamykanie tematu