Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [php] Upload plików - problem ze skryptem
Forum PHP.pl > Forum > Przedszkole
hausner15
15.05.2007, 20:05:50
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 
  1. <form name="upload" method="post" enctype="multipart/form-data" action="upload.php"> 
  2. <input type="file" name="plik" /> 
  3. <input type="hidden" name="MAX_FILE_SIZE" value="512000" /> 
  4. <input type="submit" value="Dodaj plik" /> 
  5. </form>
[HTML] pobierz, plaintext


I kod upa:

[PHP] pobierz, plaintext 
  1. <?php 
  2. //Sprawdzamy czy plik został wysłany 
  3. if(!is_uploaded_file($_FILES[plik][tmp_name])){ 
  4. echo "Musisz wybrać jakiś plik Cwaniaczku! 
  5. "; 
  6. exit;} 
  7. //Sprawdzamy czy nie ma błędów 
  8. if($_FILES[plik][error] != 0){ 
  9. echo "Wystąpił błąd podczas przesyłania pliku! 
  10. Sprawdź czy plik posiada odpowiednią wagę i spróbuj ponownie! 
  11. "; 
  12. exit;} 
  13. //Sprawdzamy MIME TYPE pliku 
  14. if($_FILES[plik][type] == "image/pjpeg" || 
  15. $_FILES[plik][type] == "image/png" || 
  16. $_FILES[plik][type] == "image/gif"){ 
  17. //Dotarliśmy tutaj to wszystko jest OK = UPLOAD 
  18. if(move_uploaded_file($_FILES[plik][tmp_name], "./upload/".$_FILES[plik][name])){ 
  19. echo "Plik wrzucono pomyślnie! 
  20. "; 
  21. exit; 
  22. }else{ 
  23. echo "Wystąpił nieoczekiwany błąd podczas kopiowania! 
  24. "; 
  25. exit; 
  26. } 
  27. }else{ 
  28. echo "Możesz uploadować tylko obrazki! 
  29. "; 
  30. exit; 
  31. } 
  32. ?>
[PHP] pobierz, plaintext


Proszę o pomoc.
Cienki1980
15.05.2007, 20:39:25
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
15.05.2007, 21:21:21 
[PHP] pobierz, plaintext 
  1. <?php
  2. $_FILES[plik][type] == "image/pjpeg"  ...
  3. ?>
[PHP] pobierz, plaintext


Takie sprawdzanie jest niepoprawne. Powinieneś sprawdzić co się znajduje po ostatniej kropce czy to jest jpg, czy gif...
hausner15
16.05.2007, 16:19:21
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 
  1. <?php
  2. <form name="upload" method="post" enctype="multipart/form-data" action="http://mojastrona.pl/upload.php">
  3. <input type="file" name="plik" />
  4. <input type="hidden" name="MAX_FILE_SIZE" value="999999999999999999999999" />
  5. <input type="submit" value="Dodaj plik" />
  6. </form>
  7. ?>
[PHP] pobierz, plaintext


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

Jak mogę się przed tym zabezpieczyć??

Proszę o pomoc
k@zio
16.05.2007, 17:06:13
Cytat(hausner15 @ 16.05.2007, 17:19:21 ) *
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 
  1. <?php
  2. <form name="upload" method="post" enctype="multipart/form-data" action="http://mojastrona.pl/upload.php">
  3. <input type="file" name="plik" />
  4. <input type="hidden" name="MAX_FILE_SIZE" value="999999999999999999999999" />
  5. <input type="submit" value="Dodaj plik" />
  6. </form>
  7. ?>
[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 
  1. <?php
  2. if ($_SERVER['HTTP_REFERER'] == "http://mojastrona.pl/upload.php"){
  3.   // Sprawdzamy czy plik został wysłany 
  4. 	if(!is_uploaded_file($_FILES[plik][tmp_name])){
  5. 	  ................ 
  6. 	 }
  7. } else echo "Spadaj";
  8. ?>
[PHP] pobierz, plaintext

questionmark.gif
hausner15
17.05.2007, 13:35:17
Nie chce mi to działać sad.gif może macie jakiś inny sposób??
in5ane
17.05.2007, 13:46:18
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
17.05.2007, 13:54:47
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 smile.gif. A to z ifem to wpadłem właśnie przed chwilką zanim to przeczytałem smile.gif .
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.