Исходное сообщение
"Необдуманное исправление привело к неполному устранению уязв..." Отправлено evkogan, 05-Сен-19 09:58
If никто не выкидывал. Переменная определена не в том месте, проинициализирована значением n непойми зачем. Реальное присвоение значения index идет внутри if. В том коде это не вызывало ошибок при выполнении, но это ошибка програмирования, так делать плохо. И компилятор не зря ругался. Это надо править, а не комменты писать. Грег не вникая переставил строки внутри if местами, просто чтобы убрать ругань компилятора. То что в 1 строке была инициализация переменной для второй, он не увидел. Т.е. причин ошибки 2: 1. плохой стиль программирования в патче. 2. непродуманные правки Грега. пох, там еще говорит, что в изначальном патче вообще была 1 строчка и ошибиться было невозможно. и есть причина 0. требование от непонятных людей при приемке патча ввести лишнюю переменную. Вопрос спорный. С одной стороны не такой там сложный код был, вполне читабельный, с другой вариант Линуса все же читается проще и в дальнейшем править его тоже проще.