[php] Upload plików - problem ze skryptem Opcje

[hausner15]

Witam - mam problem z skryptem uploadu plików - nie działa i nie wime dlaczego - niemoge się doszukać błędu. Jeżeli dobrze pamiętam pisało że jest od w 17 lini.

[HTML] pobierz, plaintext 
<form name="upload" method="post" enctype="multipart/form-data" action="upload.php"> 
<input type="file" name="plik" /> 
<input type="hidden" name="MAX_FILE_SIZE" value="512000" /> 
<input type="submit" value="Dodaj plik" /> 
</form>
[HTML] pobierz, plaintext 

I kod upa:

[PHP] pobierz, plaintext 
<?php 
//Sprawdzamy czy plik został wysłany 
if(!is_uploaded_file($_FILES[plik][tmp_name])){ 
echo "Musisz wybrać jakiś plik Cwaniaczku! 
"; 
exit;} 
//Sprawdzamy czy nie ma błędów 
if($_FILES[plik][error] != 0){ 
echo "Wystąpił błąd podczas przesyłania pliku! 
Sprawdź czy plik posiada odpowiednią wagę i spróbuj ponownie! 
"; 
exit;} 
//Sprawdzamy MIME TYPE pliku 
if($_FILES[plik][type] == "image/pjpeg" || 
$_FILES[plik][type] == "image/png" || 
$_FILES[plik][type] == "image/gif"){ 
//Dotarliśmy tutaj to wszystko jest OK = UPLOAD 
if(move_uploaded_file($_FILES[plik][tmp_name], "./upload/".$_FILES[plik][name])){ 
echo "Plik wrzucono pomyślnie! 
"; 
exit; 
}else{ 
echo "Wystąpił nieoczekiwany błąd podczas kopiowania! 
"; 
exit; 
} 
}else{ 
echo "Możesz uploadować tylko obrazki! 
"; 
exit; 
} 
?>
[PHP] pobierz, plaintext 

Proszę o pomoc.

[Cienki1980]

Zakładam, że sprawdzałeś to na pliku jpg ... w moim przypadku $_FILES['plik']['typ'] miało wartość image/jpg a nie tak jak masz w kodzie image/pjpg. Po poprawieniu tego upload działa poprawnie ( przynajmniej u mnie na lokalu )

[cornholio666]

[PHP] pobierz, plaintext 
<?php
$_FILES[plik][type] == "image/pjpeg"  ...
?>
[PHP] pobierz, plaintext 

Takie sprawdzanie jest niepoprawne. Powinieneś sprawdzić co się znajduje po ostatniej kropce czy to jest jpg, czy gif...

[hausner15]

Działa. Dzięki a niezauważyłem...

//dodano

Dobra mam jeszcze jedną sprawe - poniewarz ten skrypt jest troszkę kiepski: wystarczy że ktoś na swoim dysku zrobi plik:

[PHP] pobierz, plaintext 
<?php
<form name="upload" method="post" enctype="multipart/form-data" action="http://mojastrona.pl/upload.php">
<input type="file" name="plik" />
<input type="hidden" name="MAX_FILE_SIZE" value="999999999999999999999999" />
<input type="submit" value="Dodaj plik" />
</form>
?>
[PHP] pobierz, plaintext 

I może upać obrazki o jakiej chce wielkości!

Jak mogę się przed tym zabezpieczyć??

Proszę o pomoc

Ten post edytował hausner15 16.05.2007, 16:18:40

[k@zio]

Działa. Dzięki a niezauważyłem...

//dodano

Dobra mam jeszcze jedną sprawe - poniewarz ten skrypt jest troszkę kiepski: wystarczy że ktoś na swoim dysku zrobi plik:

[PHP] pobierz, plaintext 
<?php
<form name="upload" method="post" enctype="multipart/form-data" action="http://mojastrona.pl/upload.php">
<input type="file" name="plik" />
<input type="hidden" name="MAX_FILE_SIZE" value="999999999999999999999999" />
<input type="submit" value="Dodaj plik" />
</form>
?>
[PHP] pobierz, plaintext 

I może upać obrazki o jakiej chce wielkości!
Jak mogę się przed tym zabezpieczyć??
Proszę o pomoc

Może

[PHP] pobierz, plaintext 
<?php
if ($_SERVER['HTTP_REFERER'] == "http://mojastrona.pl/upload.php"){
  // Sprawdzamy czy plik został wysłany 
	if(!is_uploaded_file($_FILES[plik][tmp_name])){
	  ................ 
	 }
} else echo "Spadaj";
?>
[PHP] pobierz, plaintext 

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[hausner15]

Nie chce mi to działać (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) może macie jakiś inny sposób??

[in5ane]

Ojejku zrob sobie zwyklego IF'a:

$_FILES[plik][size] <--- i tutaj juz bedziesz mial rozmiar pliku w bajtach

Tworzysz tego IF'a, aby ta zmienna (wyzej) gdy bedzie wieksz na np. od 1,5MB to wyswietla komunikat a jezeli mniejsze to puszcza skrypt dalej.

Ale i tak watpie zeby komus sie chcialo akurat na twoj darmowy serwer (50 MB) wrzucac cos wielkiego.

Powiem Ci tez jedna rzecz, ze ten skrypt nie ma sensu, gdyz:

np. ktos uploaduje plik o nazwie "asd.jpg" a po nim ktos kolejny uploaduje plik o nazwie "asd.jpg" to juz lipton... nie wgra poprawnie pliku drugiej osobie, dlatego propnuje dopisac aby skrypt losowal nazwe samemu (np. jakies byle jakie znaki od 1 do 9 i od A do Z) (ilosc znakow dowolna)

[hausner15]

Znaczy to jest dla testu - chce to skleić ze skryptem txtbb a tam jako nazwa użytkownika jest <!-- USER NAME -->. Jak to wkleje to jako avatarek o nazwie profilu bedzie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . A to z ifem to wpadłem właśnie przed chwilką zanim to przeczytałem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) .