ivan-lev

Рейтинг
435
Регистрация
20.04.2007

MODx shopkeeper посмотрите.. там и заказы можно принимать.

Или просто раздел со страничками создать.. картинки-описания..

Для вывода каталога можно Ditto использовать.

RedOK:
, так что кавычки не катят.

:) катят.. они в базу попадают..

А потом, при авторизации $_POST['login'] (экранированный, как положено) берётся для поиска юзера (в базе, с кавычкой.. юзер, естественно находится, если он там есть...), после чего этот $_POST['login'] проходит валидацию в Auth::test_token и уже дальше в "чистом виде" (т.е. без экранирования) попадает в функции..

А дальше.. дальше - хз.. может там ничего и нет..

RedOK:
проверка на валидность этого login в Auth::test_token где как раз и фильтруется login

А валидность берётся из базы, в которую попасть можно через регистрацию.. 🤪

denis920:
В двух словах, человек это сделал неосознанно в силу спешки или малого опыта и ему можно отдать на доработку либо все это специально сделано и лучше поискать другого исполнителя?

Ждём главного телепата.. :) Ну или можете у самого исполнителя спросить.. Я больше к "не специально" склоняюсь..

Доработка, кстати, может и не пригодиться.. (как с Неуловимым Джо)

А так - формально, прокатывает регистрация с логином вроде

' OR 1 = 1 OR 1 = '1

А он чуть позже передаётся без экранирования.. (как уже отметил выше RedOK)

Возможно, там дальше и "нет ничего"..

RedOK:
да, я не программист

Эта часть сообщения ТС-у адресована.. не нужно на свой счёт

RedOK:
либо я уже слишком не трезв для этого суббота же

🍻🍻

насчёт redirect.php line 19 - там чуть выше $_GET['id'] на is_numeric проверяется..

Но всё равно "некрасиво"

ivan-lev:
"чистый" $_GET в запросах к базе..
denis920:
То есть, явных специально сделанных дыр Вы не обнаружили?

Да, я уже понял, что Вы не программист.. про инъекции написал.

RedOK:
давайте без сарказма только. объяснил же.

Эм.. видимо, собственно объяснение я проглядел.. Можно "для особо одарённых" повторить?

p.s. Надеюсь, не будет принято на свой счёт.. Или принято, но, исключительно, в хорошем смысле :)

RedOK:
и мне и мне! я тоже хочу

Не-не.. тут профессионал, ему код не нужен 😂 Он без исходников с теми же шансами ищет..

В целом скрипт более-менее понятный (ну не.. счас начнётся "да я бы в 100500 раз лучше написал") я к тому, что не прослеживается намерений "запутать"..

Проскакивают местами косячки - "чистый" $_GET в запросах к базе.. возможно, так задумано, а может "просто"..

Ставить-тестить не стал...

theBlackWolf:
В упор не могу понять что это за борода

firebug в помощь..

Выкладывайте уже ваш код.. Мы его быстренько разберём 😂

RedOK:
нет, шансов не больше. просто времени это займёт намного меньше.

Эм.. или мы друг друга не понимаем..

Даю адрес (URL т.е) без исходного кода.. даю гарантию 146%, что есть дыра-дырищща .. найдёте?

Потом тоже самое, но с исходником.. И расскажите, что "шансов не больше"

chmod 777:
10 секунд - медленно?

смотря какая страница.. если хитромудрый отчёт какой из 100500 строк - возможно и нормально..

А для "обычного сайта", "обычного посетителя", и поискового робота - многовато..

Всего: 4907