LINUX.ORG.RU
Форум — Linux-org-ru  

Кто-нибудь исправит когда-нибудь?


0

0
Запарили уже вечные

=== cut ===
String index out of range: 34
К сожалению, произошла исключительная ситуация при генерации страницы.

Произошла непредвиденая ошибка. Администраторы получили об этом сигнал.
=== cut ===

на любое более-менее сложное сообщение.

В данном случае оно отваливается на таком примере: http://snipt.org/Qph
★★★★★
← Хочу его забанить.
по поводу score → 
Убери кавычки от урла или убери Aurourl

http://www.linux.org.ru/jump-message.jsp?msgid=2713241&cid=2714463 подробности
wfrr ★★☆
()
Ответ на: комментарий от KRoN73
та вроде не, этож ява, в ней чтото вообще сложно провести, не то что иньекцию

Щас сорцы открыли можно глянуть код что в преформаттед рассавляет урлы, косяк в нем.
wfrr ★★☆
()
Ответ на: комментарий от wfrr
>та вроде не, этож ява, в ней чтото вообще сложно провести, не то что иньекцию

От языка это мало зависит. Больше от библиотек.

Мой опыт в Java показывает, что там программисты имеют привычку писать так, как в конструкторе Message в http://github.com/maxcom/lorsource/blob/ffb910b46e1eda1de80bc116ac5551c31dd68...

И сам так писал :) Hibernate или Ibatis мы в своё время так и не собрались прикрутить, очень сильно нужно было классы базовые рефакторить. А свой ORM так и не слепил.

А в том же PHP ORM лепится на коленке за пару дней и больше прямые SQL-запросы практически нигде не используются :)

...

Я уже не помню, когда в новом PHP-коде на своих проектах использовал SQL-запросы :)

Как следствие, в моём PHP-коде мест для иньекции нет уже несколько лет как. А на практике с таким не сталкивался ни разу. А вот на Java мы пару раз SQL-инъекции зевали. Правда, этого никто не заметил до исправления, но факт есть факт - было :)
KRoN73 ★★★★★
() автор топика
Ответ на: комментарий от KRoN73
В препаредстейтмент иньекцию не засунуть, как и если забивать резалтсет полученный от селекта.
wfrr ★★☆
()
Ответ на: комментарий от wfrr
>В препаредстейтмент иньекцию не засунуть

Угу. Вот только там, когда у тебя подставляется не один параметр, а два десятка, легко промахнуться с порядком параметров. И поэтому часто пишут так, как на приведённой ссылке ;)

...

В любом случае, это всё равно прямые SQL-вызовы и это очень плохо.
KRoN73 ★★★★★
() автор топика
Ответ на: комментарий от wfrr
http://github.com/maxcom/lorsource/blob/ffb910b46e1eda1de80bc116ac5551c31dd68...

Тут ищется конец ентити и не проверяется выход за границы строки, т. е., ентити всегда полагается завершенным. Судя по тесту на сообщении топикстартера, это не всегда так.

svr69 ★★
()
Ответ на: комментарий от maxcom
>fixed

Вот и прямая польза от открытия сорцов... А то сколько лет я на эту проблему ругался :)
KRoN73 ★★★★★
() автор топика
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.
← Хочу его забанить.
Linux-org-ru
по поводу score →