Форум Практика оптимизации Любые вопросы от новичков по оптимизации

PHP cкрипт загрузки файлов на сервер. Как избежать уязвимости скрипта ?

SE

13

S.E.O

8 марта 2007, 00:05

3455

PHP cкрипт загрузки файлов на сервер. Как избежать уязвимости скрипта ?

Пример скрипта (без фильтров)


<?php 

if(!empry($_FILES['filename'] ['tpm_name']))

{

  if (copy($_FILES['filename'] ['tpm_name'], $_FILES['filename']['name']))

  {

    echo" Файл успешно загружен - <a href=".

     $_FILES['filename']['name'].">".

     $_FILES['filename']['name']."</a>";

  }

}

?>

Обмен ссылками Авто-Мото, Пишу PHP скрипты любой сложности, Продажа Запчастей на Японские Авто и Спецтехнику

DU

105

DIMON-UA

8 марта 2007, 00:13

#1

Ложить файлы ниже корня сайта. Остальные методы оценить сложно, т.к. не извесно как Вы вызываете скрипт

775

edogs software

8 марта 2007, 00:23

#2

Не empry, а empty... не tpm_name, а tmp_name.. не copy, а move_uploaded_file.

Никакой уязвимости в таком скрипте нет и не может быть по определению... ибо Вы не сказали о каких-либо ограничениях.

Уязвимость (по определению) это когда скрипт делает что-то, что непредусмотрено создателем. Этот Ваш скрипт ничего такого делать не может.

Разработка крупных и средних проектов. Можно с криптой. Разумные цены. Хорошее качество. Адекватный подход. Продаем lenovo legion в спб, дешевле магазинов, новые, запечатанные. Есть разные. skype: edogssoft

eTarget 2011:Панельная дискуссия «Стратегия eTarget 2011: Круглый стол Могут ли «плохие» входящие

SE

13

S.E.O

8 марта 2007, 00:28

#3

DIMON-UA:
Ложить файлы ниже корня сайта.

Забыли добавить если есть такая возможность..

DIMON-UA:
Остальные методы оценить сложно, т.к. не извесно как Вы вызываете скрипт

Зайдя на сайт любой может добавить свой файл(картинку) через форму на сайте.

Обработчик формы см. выше. Скрипту передаються данные методом post

DU

105

DIMON-UA

8 марта 2007, 00:30

#4

edogs:
Не empry, а empty... не tpm_name, а tmp_name.. не copy, а move_uploaded_file.
Никакой уязвимости в таком скрипте нет и не может быть по определению... ибо Вы не сказали о каких-либо ограничениях.
Уязвимость (по определению) это когда скрипт делает что-то, что непредусмотрено создателем. Этот Ваш скрипт ничего такого делать не может.

Скрипт не может, загруженый файл может.

DU

105

DIMON-UA

8 марта 2007, 00:32

#5

S.E.O:
Забыли добавить если есть такая возможность..

Зайдя на сайт любой может добавить свой файл(картинку) через форму на сайте.
Обработчик формы см. выше. Скрипту передаються данные методом post

Необходимо проверять тип файла и загружать только разрешенный.

SE

13

S.E.O

8 марта 2007, 00:37

#6

edogs:
Не empry, а empty... не tpm_name, а tmp_name.. не copy, а move_uploaded_file.

Скрипт исправлять не нужно....просто нужно добавить фильтр..

edogs:
Этот Ваш скрипт ничего такого делать не может.

Он записывает файлы на сервер и тутже выводит ссылку на файл...Вы наверно не внимательно смотрели..

SE

13

S.E.O

8 марта 2007, 00:38

#7

DIMON-UA:
Необходимо проверять тип файла и загружать только разрешенный.

А примерчик на php если не трудно ?

DU

105

DIMON-UA

8 марта 2007, 00:46

#8

$_FILES['filename']['type']

SE

13

S.E.O

8 марта 2007, 00:54

#9

DIMON-UA,

С обработчиком что-то типа этого получиться ?


if(!empry($_FILES['filename'] ['tpm_name']))

{

  $ext = strtolower(strrchr($_FILES['filename']['name'),"."));
  $extentions = array(".jpg", ".gif");

  if(in_array($ext, $extentions))

 {



  if (copy($_FILES['filename'] ['tpm_name'], $_FILES['filename']['name']))

  {

    echo" Файл успешно загружен- <a href=".

     $_FILES['filename']['name'].">".

     $_FILES['filename']['name']."</a>";

  }

}



else

{

  Echo" Разрешена загрузка только изображений ";



}

}

SE

13

S.E.O

8 марта 2007, 00:58

#10

DIMON-UA:
$_FILES['filename']['type']

Как лучше делать то? Как я написал или как вы предложили ?