The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



"Необдуманное исправление привело к неполному устранению уязв..."
Версия для распечатки Пред. тема | След. тема
Форум Разговоры, обсуждение новостей
Исходное сообщение [ Отслеживать ]
Присылайте удачные настройки в раздел примеров файлов конфигурации на WIKI.opennet.ru.
. "Необдуманное исправление привело к неполному устранению уязв..." +/
Сообщение от evkogan (?), 05-Сен-19, 09:58 
If никто не выкидывал.
Переменная определена не в том месте, проинициализирована значением n непойми зачем. Реальное присвоение значения index идет внутри if.
В том коде это не вызывало ошибок при выполнении, но это ошибка програмирования, так делать плохо.
И компилятор не зря ругался.
Это надо править, а не комменты писать.

Грег не вникая переставил строки внутри if местами, просто чтобы убрать ругань компилятора.
То что в 1 строке была инициализация переменной для второй, он не увидел.
Т.е. причин ошибки 2:
1. плохой стиль программирования в патче.
2. непродуманные правки Грега.

пох, там еще говорит, что в изначальном патче вообще была 1 строчка и ошибиться было невозможно.
и есть причина 0. требование от непонятных людей при приемке патча ввести лишнюю переменную.
Вопрос спорный. С одной стороны не такой там сложный код был, вполне читабельный, с другой вариант Линуса все же читается проще и в дальнейшем править его тоже проще.

Ответить | Правка | Наверх | Cообщить модератору

Оглавление
Необдуманное исправление привело к неполному устранению уязв..., opennews, 04-Сен-19, 12:01  [смотреть все]
Форумы | Темы | Пред. тема | След. тема



Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру