parser

Написать ответ на текущее сообщение

 

 
   команды управления поиском

потом расскажите, зачем надо было-то? :)

Александр Петросян (PAF) 13.12.2005 10:50 / 13.12.2005 10:51

замечания:

1. недопустимо грузить файл с диска, если имя передал пользователь, слепо доверяя пользователю.
пользователь может быть и доверенный, но всё равно, пока доверенный пошёл кофе попить и незаблокировал рабочую машину, злоумышленник может наделать дел.
а если пользователь недоверенный, или их много и отличить одного от другого почему-то нельзя, то тем более нельзя.
моё мнение: так нельзя делать никогда.

2.
мелкое замечание по стилю:

моё мнение: имя "xxx_temp_xxx" недопустимо для переменной. это
* и возмутительная потеря времени на его набор;
* и читателю это имя ровно ничего не говорит.

для вашего примера предлагаю:
$file_preview
$table_preview
$result