Показать сообщение отдельно
Старый 19.10.2009, 15:04     # 12
Hubbitus
мод
IMHO Кодер-200(6,7,8)
 
Регистрация: 29.03.2003
Адрес: Saint-Petersburg, Russia
Пол: Male
Сообщения: 2 734

Hubbitus Бог с наворотамиHubbitus Бог с наворотами
Hubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотамиHubbitus Бог с наворотами
Цитата:
Сообщение от killhunter Посмотреть сообщение
Т.е. надо ввести проверку на размер временного фаила, чтобы не ноль.
То есть надо проверить корректность загрузки, я же написал! $_FILES['image_path']['error'], is_uploaded_file...


Про жуткий код, ну я частично в скобках уже указал, ну например, что за велосипед:
Код:
$name=substr($filename1_1, 0, strrpos($filename1_1, "."));
Если в $filename1_1 будет имя файла "a.b.c.d.jpg"? Почему не воспользоваться стандартными http://php.net/pathinfo, http://ru2.php.net/basename? Там же и расширение файла. Про небезопасное использование copy я говорил. Ну и также далее, про длину файла в 8 символов (вообще не понятно откуда, ну да Бог с ней, может Вы в DOSе работаете, или того требует бизнес-логика) проверка по первой точке также не корректна...
__________________
Я делаю Линукс! Присоединяйтесь к свободным людям!

Связаться со мной всегда можно по джабберу: Hubbitus@jabber.ru
Pahan-Hubbitus.
Hubbitus вне форума