Исходное сообщение
"Необдуманное исправление привело к неполному устранению уязв..." Отправлено evkogan, 04-Сен-19 19:59
Устал читать пошел посмотрел, что и как. Изначально предлагали сделать if (n < HBP_NUM) {     struct perf_event *bp =thread->ptrace_bps[array_index_nospec(n, HBP_NUM)]; А потом затребовали сделать промежуточную переменную. Получилось int index = n;     if (n < HBP_NUM) {    index = array_index_nospec(index, HBP_NUM);    struct perf_event *bp = thread->ptrace_bps[index]; Линус исправил и сделал if (n < HBP_NUM) {    int index = array_index_nospec(index, HBP_NUM);    struct perf_event *bp = thread->ptrace_bps[index]; Да. первый вариант самый простой. Вариант Линуса тоже вполне прост, читается может чуть-чуть лучше. Но не критично. Вариант с переменной перед if самый трудно читаемый, хранит лишнюю переменную дольше чем она нужна и вызывает ругань компилятора. А как оказалось еще и провоцирует ошибки. Выводы: Да придирки были хрен знает к чему. Линус все еще торт. Все ядра перечисленные в новости поддерживает Грег Кроа-Хартман, который лопухнулся. Ну с одной стороны с кем не бывает, с другой как же мы потом без Линуса будем.