0

Хочу сделать проверку, чтобы на сайт можно было закачать только изображения.

  $filename = $_FILES["img1"]["tmp_name"]; // В переменную $filename заносим точное имя файла (включая расширение).
  $allowed_filetypes = array('.jpg','.jpeg','.gif','.png'); 
 echo $ext = substr($filename, strpos($filename,'.'), strlen($filename)-1);

 if(!in_array($ext,$allowed_filetypes))
 die('Данный тип файла фотографий не поддерживается. Вернитесь назад и выберите фотографию с друугим разрешением .jpeg, .gif или .png ');

echo $ext выдает .tmp Как разрешить эту проблему?

6
  • используйте $ext=pathinfo($filename)['extension']; чтобы корректно расширения получать, print_r($allowed_types); в студию
    – zb'
    13 фев 2013 в 22:19
  • У меня в $filename тоже временный файл лежит. 13 фев 2013 в 22:24
  • а, не заметил, а зачем у вас там имя временного файла ? $ext=pathinfo($_FILES["img1"]['name'])['extension'];
    – zb'
    13 фев 2013 в 22:26
  • в переменную получается нельзя класть $_FILES["img1"]['name'] ? 13 фев 2013 в 22:34
  • можно, только у вас там tmp_name
    – zb'
    13 фев 2013 в 22:40

1 ответ 1

1

Чтобы проверить валидность файла нужно не расширение смотреть, а тип. Иначе вам под "правильным" расширением подсунут что угодно, с какими угодно последствиями. Для картинок можете воспользоваться функцией getimagesize(), заодно и устранится возможное несоответствие типа (например, если жпег зальют с расширением png), и пригодится для дальнейшей обработки картинки, если такая планируется, и проверки ширины/высоты на допустимость.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.