Форум на Исходниках.RU

прверка рисунка на вшивость

sanweb — Mon, 29 Jan 2007 09:37:58 +0000

sanweb: да нет
все нормально :D

прверка рисунка на вшивость

Pr0[)!9Y — Mon, 29 Jan 2007 09:07:33 +0000

Pr0[)!9Y:

Цитата sanweb @ 29.01.07, 07:04

PS: сделал бы изначально скрипт обработки формы с проверкой расширения файла то дебатов по поводу некорректного использования функции не былобы!

Сделал бы, а чего плохого в дебатах?

прверка рисунка на вшивость

sanweb — Mon, 29 Jan 2007 07:04:25 +0000

sanweb: Если форма делается не для закачки изображения в папку доступную по http, а например дла аттача файла к почтовому сообщению, то достаточно проверять гетимжсайзом
Если бы форма делалась для пользователя, то в скрипте конечно же делалась бы проверка расширения файла

PS: сделал бы изначально скрипт обработки формы с проверкой расширения файла то дебатов по поводу некорректного использования функции не былобы!
>:(

прверка рисунка на вшивость

SiMM — Fri, 26 Jan 2007 19:48:59 +0000

SiMM: Уже лучше - только точка в данном случае не означает точку ;) Здесь не критично - но при таком подходе когда-нибудь можно и нарваться на непонятный глюк ;)

прверка рисунка на вшивость

Pr0[)!9Y — Fri, 26 Jan 2007 19:46:14 +0000

Pr0[)!9Y:

Цитата SiMM @ 26.01.07, 19:39

Садись, два

И правда, недопечатал)

Цитата SiMM @ 26.01.07, 19:39

preg_match('/.(gif|png|jpg|jpeg)$/i',$_FILES['userfile']['name'])

прверка рисунка на вшивость

SiMM — Fri, 26 Jan 2007 19:39:51 +0000

SiMM:

Цитата sanweb @ 26.01.07, 14:19

а каким образом в гиф файл дописать пхп код?

GIF89<?print_r(getimagesize(__FILE__))?>

Цитата sanweb @ 26.01.07, 14:19

а какие еще баги в пхп есть?

А при чём здесь PHP?

Цитата Pr0[)!9Y @ 26.01.07, 14:34

preg_match('/.(gif|png|jpg|jpeg)/i',$_FILES['userfile']['name'])

Садись, два :)

$_FILES['userfile']['name'] = 'mysuperpng.php';

прверка рисунка на вшивость

Pr0[)!9Y — Fri, 26 Jan 2007 14:34:08 +0000

Pr0[)!9Y:

Цитата sanweb @ 26.01.07, 14:19

а какие еще баги в пхп есть?

Была тут тема разок, но ее благополучно забили >:(

2Moders:
Может создадим топик с багами идырами распросраненными? И не будем пинать мол в нете много материала? И все это ламерство и гуру все знают. Для новичков и то полезно будет.

Добавлено 26.01.07, 14:39

Цитата sanweb @ 26.01.07, 14:31

с какими графическими расширениями такое еще моно проделать?

Предполагаю что с любыми, потому что чаще всего информация о изображении сосредоточена в начале и функция не смотрит на окончание файла.

Цитата sanweb @ 26.01.07, 14:31

этож тогда можно ломануть любой сайт где есть форма закачки! а эта фигня только с гифами?

Навряд ли, потому что вряд ли на многих сайтах есть такая некорректная (имхо) проверка.

Цитата sanweb @ 26.01.07, 14:31

и как сделать чтобы скрипт закачки такие видоизмененные файлы не исполнял?

Проверять расширение самостоятельно

preg_match('/.(gif|png|jpg|jpeg)/i',$_FILES['userfile']['name'])

pathinfo()

прверка рисунка на вшивость

sanweb — Fri, 26 Jan 2007 14:31:22 +0000

sanweb: и как сделать чтобы скрипт закачки такие видоизмененные файлы не исполнял?

закачивать их в папку где скрипты не имеют право на исполнение или как?
этож тогда можно ломануть любой сайт где есть форма закачки! а эта фигня только с гифами?
с какими графическими расширениями такое еще моно проделать?

прверка рисунка на вшивость

Pr0[)!9Y — Fri, 26 Jan 2007 14:24:08 +0000

Pr0[)!9Y:

Цитата sanweb @ 26.01.07, 14:19

ну вопервых проверку проходят только картинки

Ты никогда не знаешь что закачает пользователь! (он по определению хакер)

Цитата sanweb @ 26.01.07, 14:19

вовторых форма лежит в админке

Как только хакер попал в админку, он ищет чего бы такого сотворить (выполнить произвольный код)

Цитата sanweb @ 26.01.07, 14:19

в четвертых я никогда не сохранял бы такие картинки в одной папке с пхп скриптами

А ты их сохраняешь куда? В папку (например images), и почему то я уверен что в этой папке наверняка скрипты тоже имеют право исполнятся ;)

Цитата sanweb @ 26.01.07, 14:19

PS: а каким образом в гиф файл дописать пхп код?

По секрету и только тебе :D

notepad

прверка рисунка на вшивость

sanweb — Fri, 26 Jan 2007 14:19:21 +0000

sanweb: ну вопервых проверку проходят только картинки
вовторых форма лежит в админке
в третих это форма выбора файла для аттача в емайл

в четвертых я никогда не сохранял бы такие картинки в одной папке с пхп скриптами
котлеты отдельно мухи отдельно!

PS: а каким образом в гиф файл дописать пхп код?

тоесть открываеш гиф в блокноте дописываеш пхп код переименовываеш в пхп
и скрипт это воспринимает как рисунок да еще и исполняет? крутооооооо.....

тааак
а какие еще баги в пхп есть? :ph34r:

прверка рисунка на вшивость

Pr0[)!9Y — Fri, 26 Jan 2007 13:30:32 +0000

Pr0[)!9Y:

Цитата

...и форматы отличные от gif, png, jpg на сервер не закачивались...
The getimagesize() function will determine the size of any GIF, JPG, PNG, SWF, SWC, PSD, TIFF, BMP, IFF, JP2, JPX, JB2, JPC, XBM, or WBMP...

Нестыковочка
Такая проверка извращение по-моему, зная заведомо что это не картинка прогонять ее через такую обработку...нехорошо.
Вроде как она для этого не предназначена :blink:

Добавлено 26.01.07, 13:40

Цитата SiMM @ 26.01.07, 13:27

Расширение всё равно проверять надо. Если файл доступен по HTTP и лежит в папке, где отрабатываются PHP-скрипты.

Хм, не поверил - решил проверить.
И действительно! Дописываем в гиф файл пхп код, переименовываем в .php и о чудо! Он прекрасно исполняется и проходит проверку с помощью getimagesize. Вот и уязвимость...

прверка рисунка на вшивость

SiMM — Fri, 26 Jan 2007 13:27:56 +0000

SiMM: Расширение всё равно проверять надо. Если файл доступен по HTTP и лежит в папке, где отрабатываются PHP-скрипты.

Добавлено 26.01.07, 13:28
Либо менять его принудительно.

прверка рисунка на вшивость

sanweb — Fri, 26 Jan 2007 13:05:31 +0000

sanweb: размер определяю как и раньше а тип гетимжсайзом!

прверка рисунка на вшивость

Pr0[)!9Y — Fri, 26 Jan 2007 10:58:27 +0000

Pr0[)!9Y: Это как? Если размер определился то картинка иначе нет, так чтоли?
Изврат.

прверка рисунка на вшивость

sanweb — Fri, 26 Jan 2007 10:44:16 +0000

sanweb: сменя закрытие темы
все получилось гетимыджсайзом!
всем спасибо! :D

а в ифе при проверке типа я писал image/jpg
это я для краткости написал только расширение

прверка рисунка на вшивость

Pr0[)!9Y — Fri, 26 Jan 2007 10:11:55 +0000

Pr0[)!9Y:

Цитата sanweb @ 26.01.07, 10:02

$_FILES[file][type]=="jpg"

Цитата

$_FILES['userfile']['type']
Mime-тип файла, в случае, если браузер предоставил такую информацию. Пример: "image/gif".

Добавлено 26.01.07, 10:18

Цитата Pr0[)!9Y @ 26.01.07, 10:11

если браузер предоставил такую информацию

Так что неплохо бы самому отрезать расширение и проверять.

Цитата sanweb @ 26.01.07, 08:48

if($_FILES[file][size]<102400 AND ($_FILES[file][type]=="gif"||$_FILES[file][type]=="png"||$_FILES[file][type]=="jpg"))
{
//код закачки на сервак
}

Заче вложенность плодить для такой задачи? Разница то небольшая, но так удобнее

Добавлено 26.01.07, 10:31
http://php.net/pathinfo

Добавлено 26.01.07, 10:33
Еще вариант:

preg_match('/.(gif|png|jpg|jpeg)/i',$_FILES['userfile']['name'])

прверка рисунка на вшивость

Рысь — Fri, 26 Jan 2007 10:08:53 +0000

Рысь: sanweb, с тебя следующее:
- вывод ошибок/варнингов, если есть
- вывод функции print_r($_FILES)
- кавычки в индексах массива
тогда будем говорить дальше.

И внимательное курение http://www.php.net/manual/ru/features.file-upload.php ибо там все написано!!!

прверка рисунка на вшивость

sanweb — Fri, 26 Jan 2007 10:02:16 +0000

sanweb: он в темп закачан!!!!
да какая разница какой код!
мне нужно чтобы код обрабатывался при соблюдении условий!

if($_FILES[file][type]=="gif"||$_FILES[file][type]=="png"||$_FILES[file][type]=="jpg")
заменю на getimagesize

прверка рисунка на вшивость

Рысь — Fri, 26 Jan 2007 09:57:07 +0000

Рысь:

Цитата SiMM @ 26.01.07, 09:52

К тому времени, когда скрипт запущен - файл УЖЕ закачан на сервак.

он закачан в куда-то в /tmp, и сдохнет после завершения скрипта ...
В общем с известной натяжкой "кодом закачки на сервак" можно считать move_uploaded_file.

Добавлено 26.01.07, 09:58
sanweb, подробнее.
print_r($_FILES); хотя б глянь...

прверка рисунка на вшивость

SiMM — Fri, 26 Jan 2007 09:52:44 +0000

SiMM:

Цитата sanweb @ 26.01.07, 08:48

вот этот вот код при файле 30кб jpg не работатет

http://phpfaq.ru/debug
http://php.net/getimagesize

Добавлено 26.01.07, 09:53

Цитата sanweb @ 26.01.07, 08:48

//код закачки на сервак

Садись, два. К тому времени, когда скрипт запущен - файл УЖЕ закачан на сервак.

прверка рисунка на вшивость

sanweb — Fri, 26 Jan 2007 08:48:12 +0000

sanweb: как сделать чтобы файлы больше 100 кб и форматы отличные от gif, png, jpg на сервер не закачивались

if($_FILES[file][size]<102400)

{

if($_FILES[file][type]=="gif"||$_FILES[file][type]=="png"||$_FILES[file][type]=="jpg")

{

//код закачки на сервак

}

вот этот вот код при файле 30кб jpg не работатет :wall: