[PHP] typ pliku Opcje

[peter13135]

[PHP] pobierz, plaintext 
<?php
if($avek[type]==('image/gif' || 'image/png' || image/jpeg)) {$check_0=true;}
?>
[PHP] pobierz, plaintext 

czy powyższy kod jest prawidłowy??
a może coś w nim brakuje??
to jest część skryptu ładowania avatarów

[pyro]

a masz jakieś zabezpieczenia czy tylko tyle?

[peter13135]

mam sprawdzanie wagi/rozdzielczośći, ale najpierw jest sprawdzanie typu pliku,

wygląda to tak

[PHP] pobierz, plaintext 
<?php
if($avek[type]==('image/gif' || 'image/png' || 'image/jpeg')) {$check_0=true;} 
	else $check_0=false;
 
	if($kb_size<101) {$check_1=true;} 
	else  $check_1=false;
 
	if(($rozdzielczosc[0]<101) &&  ($rozdzielczosc[1]<101) ) {$check_2=true;}
	else $check_2=false;
 
	if(($check_1 && $check_2 && $check_0)==true)
{
zapisywanie
}
?>
[PHP] pobierz, plaintext 

Ten post edytował peter13135 13.08.2008, 12:58:14

[pyro]

możesz jeszcze sprawdzać rozszerzenie pliku

[peter13135]

funkcją substr() (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) a sam typ pliku niewystarczy?

[pyro]

sprawdzanie MIME, można w bardzo prosty sposób, sprawdzanie rozszerzenia pliku napewno nie zaszkodzi (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[revyag]

Internet Explorer i Firefox zwrócą rożne mime. Może użyj funkcji exif_imagetype().

http://pl2.php.net/manual/en/function.exif-imagetype.php

[peter13135]

[PHP] pobierz, plaintext 
<?php
	if($avek)
	{
	//sprawdzanie avka
 
	if($kb_size<101) {$check_1=true;} 
	else  $check_1=false;
 
	if(($rozdzielczosc[0]<101) &&  ($rozdzielczosc[1]<101) ) {$check_2=true;}
	else $check_2=false;
 
	$rozs=substr($avek[name], 3);
	$dozwolone= array('gif', 'png', 'peg', 'jpg', 'GIF', 'PNG', 'PEG', 'JPG' );
	if (in_array($rozs, $dozwolone)) {$check_3==true;}
 
 
	$random=rand(9999999,99999999999999999);
	if($check_1==true && $check_2==true  && $check_3==true)
	{
?>
[PHP] pobierz, plaintext 

może być tak??

temat do zamknięcia

zrobiłem to już troszke inaczej

błąd był między innymi w substr , zamiast -3, dałem 3

Ten post edytował peter13135 13.08.2008, 13:14:01