The OpenNET Project / Index page

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



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

Университет Миннесоты отстранён от разработки ядра Linux за отправку сомнительных патчей, opennews (??), 21-Апр-21, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


2. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +61 +/
Сообщение от Аноним (2), 21-Апр-21, 21:41 
ну вообще молодцы (и те кто отправлял и особенно те кто проверял), теперь я точно зная что Linux следят внимательно и пользоваться им безопасно.
Ответить | Правка | Наверх | Cообщить модератору

6. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +21 +/
Сообщение от Аноним (-), 21-Апр-21, 21:54 
>> приводит пример уязвимости CVE-2019-12819, вызванной исправлением, принятым в ядро в 2014 году.
>  теперь я точно зная что Linux следят внимательно

Главное при этом - не читать "неправильные" части новости.

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

18. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +12 +/
Сообщение от Dzen Python (ok), 21-Апр-21, 22:12 
А что не так? Не вижу противоречия. "Пранькеры" как обычно попались на мелочи, далее быстро и решительно (а главное - открыто, главная причина от чего у минисотунцов пригорает) патчи проверены и откачены.
Ревью не идеален, да.
Но в каком закрытом (да и положа руку на сердце - и большей части т.н. открытых) подобная реакция имела бы место? А не тихая мышиная возня за закрытыми дверями "а что люди подумают?".
Ответить | Правка | Наверх | Cообщить модератору

22. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  –10 +/
Сообщение от Аноним (-), 21-Апр-21, 22:19 
> А что не так?

Как минимум - не установлена причинно-следственная связь "экспериментаторы попались, потому что проверяли хорошо" или "проверяли хорошо, потому что знали об экспериментаторах".


> Но в каком закрытом (да и положа руку на сердце - и
> большей части т.н. открытых) подобная реакция

Скатывание в очередной whatsaboutism обычно не решает никаких проблем.

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

26. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +5 +/
Сообщение от Dzen Python (ok), 21-Апр-21, 22:27 
> Скатывание в очередной whatsaboutism обычно не решает никаких проблем.

Не, как раз решает. Сейчас интересно, как долго они будут перетряхивать простыни, но что точно будет - будет ужесточено ревью и удлинена проверка перед внедрением патча в код. При "тихом" откате такого резонанса в коммунити бы не было.

> Как минимум - не установлена причинно-следственная связь

"нашли намеренную закладку в патче" ->
"коммунити встало в стойку" ->
"начали выяснять чтозанах" ->
"профессоришке пришлось громко оправдываться" ->
"Грег проверил остальные их патчи и откатил их" ->
"в линуксе нет закладок от этих пранкеров-"бизапасннеков" " ->
"теперь вузик попал на карандаш и ВСЁ"

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

43. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  –3 +/
Сообщение от Аноним (43), 21-Апр-21, 23:08 

>> Как минимум - не установлена причинно-следственная связь
> "нашли намеренную закладку в патче" ->
> "коммунити встало в стойку" ->

Да-да. Именно так, а не "студент-ассистент со взором горящим послал анонимку" - "Мамой клянус!"

>> Скатывание в очередной whatsaboutism обычно не решает никаких проблем.
> Не, как раз решает.

Э-э, ты не знаешь, что такое whatsaboutism. Окай.

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

56. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  –1 +/
Сообщение от Аноним (56), 22-Апр-21, 00:19 
Хорошо бы еще знать, что s после what совершенно лишняя
Ответить | Правка | Наверх | Cообщить модератору

144. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  –2 +/
Сообщение от Аноним (144), 22-Апр-21, 10:29 
Все правильно, забанили же университет, а не студентов!

Потому что в университетах тупые преподы злостно эксплуатируют способных студентов, а все лавры приписывают своим бездарным кафедрам.

Теперь просто те студенты не будут присылать патчи от имени университета. А будут хотя бы анонимно пиарить свои анонимные гит-аккаунты. И меньше будут слушать преподов, которые говорили им что так делать нельзя, а код писать можно типа только с разрешения кафедры на которой учишься.

И это совсем не шутка, это реальность. Набирают в универы провинциалов, способных, но не знающих что почем. Те для преподов и для зав-кафедрам пишут код, который провозглашается кафедральными достижениями неизвестных студентов, умеющих то, что не умеют преподы с научными званиями.

Но самое главное, что при этом еще студентам втирают, что они якобы могут пользоваться open-source "программами" и публиковаться в open source только строго с разрешения кафедры, на которой учатся. А иначе накажут!

Думаете что это не так? А зачем бы иначе способным студентам из провинций были нужны эти универы, если бы они сразу узнали, а узнав сразу бы поверили, что они универам и преподам с учеными званиями ничего не должны!

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

8. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +3 +/
Сообщение от Аноним (8), 21-Апр-21, 21:56 
так, я не понял: на лоре "Исследователям удалось добавить в ядро Linux уязвимый код", а на опеннете, судя по тексту, грег бдит и не пропустил. кому верить?
Ответить | Правка | К родителю #2 | Наверх | Cообщить модератору

17. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +8 +/
Сообщение от Аноним (-), 21-Апр-21, 22:10 
> так, я не понял: на лоре "Исследователям удалось добавить в ядро Linux
> уязвимый код", а на опеннете, судя по тексту, грег бдит и не пропустил. кому верить?

Верить в наше время никому нельзя, даже себе. Мне - можно! (с)

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

57. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +1 +/
Сообщение от Аноним (57), 22-Апр-21, 00:43 
Лору, там в обсуждении пруфы на то, что сами васяны предупредили разработчиков, потому что разработчики ничего не нашли и отправили патчи в прод.
Ответить | Правка | К родителю #8 | Наверх | Cообщить модератору

76. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +1 +/
Сообщение от Аноним (76), 22-Апр-21, 03:20 
> Лору, там в обсуждении пруфы на то, что сами васяны предупредили разработчиков, потому что разработчики ничего не нашли и отправили патчи в прод.

Так кинь ссылку на пруф-то?

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

122. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  –3 +/
Сообщение от n00by (ok), 22-Апр-21, 09:02 
>> Лору, там в обсуждении пруфы на то, что сами васяны предупредили разработчиков, потому что разработчики ничего не нашли и отправили патчи в прод.
> Так кинь ссылку на пруф-то?

Раз ничего лучше не нашлось, пруфом будем считать https://joindiaspora.com/uploads/images/scaled_full_b39c6332...

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

114. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  –1 +/
Сообщение от Аноним (114), 22-Апр-21, 08:15 
Вообще-то сами исследователи написали, что не один их бажный патч не попал в ядро.

https://www-users.cs.umn.edu/~kjlu/papers/clarifications-hc.pdf

All the bug-introducing patches stayed only in the email exchanges, without being adopted or merged into any Linux branch, which was explicitly confirmed by maintainers. Therefore, the bug-introducing patches in
the email did not even become a Git commit in any Linux branch.

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

233. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (-), 24-Апр-21, 21:14 
> Вообще-то сами исследователи написали, что не один их бажный патч не попал в ядро.

А что тогда дропнул Грег?!

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

236. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (76), 24-Апр-21, 22:46 
>> Вообще-то сами исследователи написали, что не один их бажный патч не попал в ядро.
> А что тогда дропнул Грег?!

Грег перестраховался, и дропнул (точнее, попросил ещё раз проревьювить) все коммиты с того универа, на всякий случай.

Многие из дропнутых коммитов изначально прошли через ревью, были уже причёсаны и безобидны.

Но нашлось и несколько бажных. Например, https://github.com/torvalds/linux/commit/467a37fba9 — это явный баг: функция `sp8870_readreg()` в случае успеха возвращает считанное значение регистра, которое может быть совсем не нулевым.

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

93. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (93), 22-Апр-21, 06:17 
Верить надо оригиналу, а не местным рабиновичам.
Ответить | Правка | К родителю #8 | Наверх | Cообщить модератору

98. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от barmaglot (??), 22-Апр-21, 06:46 
Ещё раз для тупых: "Верить никому нельзя!"
Ответить | Правка | Наверх | Cообщить модератору

125. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +3 +/
Сообщение от би браза вотчинг ю (?), 22-Апр-21, 09:30 
а тебе можно верить что "верить никому нельзя"?
Ответить | Правка | Наверх | Cообщить модератору

167. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от z (??), 22-Апр-21, 12:21 
и ему как раз нельзя верить. Это не противоречивая фраза.
Ответить | Правка | Наверх | Cообщить модератору

168. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от bi brother (?), 22-Апр-21, 12:27 
Но тебе то хоть можно что ему нельзя?
Ответить | Правка | Наверх | Cообщить модератору

14. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +13 +/
Сообщение от Dzen Python (ok), 21-Апр-21, 22:02 
Когда попался на отправке хрени в ядро:
> Не-не-не, посоны, это пранк, воть! Мы безопасность исследовали, ёпта! А почему не сказали по факту принятия и прохождения ревью? Ну так забыли! Сначала закрутились с трампом, потом короновирус, потом блм, потом выборы, а потом - суп с котом! А-а-а-а! Хулиганы права на патчи лишают!
Ответить | Правка | К родителю #2 | Наверх | Cообщить модератору

137. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +2 +/
Сообщение от Lex (??), 22-Апр-21, 10:11 
Где-то в углу зарыдали БСДшники, ведь мысль о том, что им что-то кодят профессора да академики университетские заиграла совершенно новыми красками.. только, эти не попадутся, ведь сами и рецензируют :)
Ответить | Правка | Наверх | Cообщить модератору

172. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +1 +/
Сообщение от Аноним (172), 22-Апр-21, 12:55 
> Где-то в углу зарыдали БСДшники, ведь мысль о том, что им что-то
> кодят профессора да академики университетские заиграла совершенно новыми красками.. только, эти не попадутся, ведь сами и рецензируют :)

То ли дело профессиональные индусокодеры в венде и макоси - все будет еще хуже, но зато "без злого умысла"?

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

234. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (-), 24-Апр-21, 21:16 
> То ли дело профессиональные индусокодеры в венде и макоси - все будет
> еще хуже, но зато "без злого умысла"?

Там вон сбоку клевый детектив про гангстера Масика. С погонями со стрельбой (ну почти). Кто б мог подумать что кодить вайргад для бсд настолько увлекательно.

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

37. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от кек (?), 21-Апр-21, 22:53 
сначала хуавей, теперь минесотта
уже давно перекатился на опенок, там все штабильно и секурно
Ответить | Правка | К родителю #2 | Наверх | Cообщить модератору

115. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +2 +/
Сообщение от bi brother (?), 22-Апр-21, 08:18 
Cuck License.
Ответить | Правка | Наверх | Cообщить модератору

140. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Lex (??), 22-Апр-21, 10:23 
Там прост никто не ищет, ведь те же самые академики и рецензируют
Ответить | Правка | К родителю #37 | Наверх | Cообщить модератору

147. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (147), 22-Апр-21, 11:02 
и не работаёт половина железок.
Ответить | Правка | К родителю #37 | Наверх | Cообщить модератору

227. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (-), 23-Апр-21, 16:10 
Особенно секурно у них виртуализатор получился, их коллега из NetBSD очень хорошо обрисовал "security circus".
Ответить | Правка | К родителю #37 | Наверх | Cообщить модератору

55. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +1 +/
Сообщение от пукпук (?), 22-Апр-21, 00:13 
https://www.opennet.ru/opennews/art.shtml?num=54903
Вот эти комрады будут заниматься тем же, что и товарищи из минисоты?
Ответить | Правка | К родителю #2 | Наверх | Cообщить модератору

138. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +2 +/
Сообщение от Аноним (138), 22-Апр-21, 10:16 
> Вот эти комрады будут заниматься тем же, что и товарищи из минисоты?

Имя у тебя говорящее, конечно...

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

60. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +5 +/
Сообщение от Аноним (60), 22-Апр-21, 00:54 
> ну вообще молодцы (и те кто отправлял и особенно те кто
> проверял), теперь я точно зная что Linux следят внимательно
> и пользоваться им безопасно.

вообще-то с точностью до наоборот. намеренно вносились исправлениям с багами. и многие остались незамеченными. если бы исследователи об этом не рассказали - ты бы и дальше сидел в своём маня-мирке.

исследование для кого написали?

бегом читать - https://raw.githubusercontent.com/QiushiWu/qiushiwu.github.i...

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

83. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +3 +/
Сообщение от Аноним (76), 22-Апр-21, 05:02 
> вообще-то с точностью до наоборот. намеренно вносились исправлениям с багами. и многие остались незамеченными. если бы исследователи об этом не рассказали - ты бы и дальше сидел в своём маня-мирке.

Вообще-то, наоборот. "Исследователи" пытались заслать странные патчи. В патче раскусили баг.

Потом в следующем их патче нашли баг.

После очередного патча с багом нашли ихнее исследование (которое было опубликовано пару месяцев назад).

Сами исследователи отмазывались, что, мол.

> исследование для кого написали?
> бегом читать - https://raw.githubusercontent.com/QiushiWu/qiushiwu.github.i...

И где там написано, что "намеренно вносились исправлениям с багами и многие остались незамеченными"?

И вообще, дай ссылки, скажем, на 3 намеренно внесённых патча с багами, которые остались незамеченным?

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

84. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (76), 22-Апр-21, 05:04 
(мимотык)

> Сами исследователи отмазывались, что, мол.

Сами исследователи отмазывались, что, мол, это какой-то статический анализатор посоветовал такое исправление.

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

235. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (-), 24-Апр-21, 21:17 
> Сами исследователи отмазывались, что, мол, это какой-то статический анализатор посоветовал
> такое исправление.

ЧСХ разработчики линя еще и пользуются статическими анализаторами и поэтому догадываются как это работает.

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

87. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (-), 22-Апр-21, 05:37 
> Вообще-то, наоборот

что наоборот? что у тебя за извращенная логика?

> которые остались незамеченным

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

> И вообще, дай ссылки, скажем, на 3 намеренно внесённых патча
> с багами, которые остались незамеченным?

какие ссылки, если они ничего зловредного и не вносили, но провели анализ возможности внесения четырёх патчей, лол. об этом вообще бы шумиха не поднялась не подними её сами авторы исследования.

> И где там написано

да начиная с первой страницы об этом только и идёт речь.

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

From: Leon Romanovsky <leon@kernel.org>
To: "J. Bruce Fields" <bfields@fieldses.org>
Cc: Greg KH <gregkh@linuxfoundation.org>,
    Aditya Pakki <pakki001@umn.edu>,
    Chuck Lever <chuck.lever@oracle.com>,
    Trond Myklebust <trond.myklebust@hammerspace.com>,
    Anna Schumaker <anna.schumaker@netapp.com>,
    "David S. Miller" <davem@davemloft.net>,
    Jakub Kicinski <kuba@kernel.org>,
    Dave Wysochanski <dwysocha@redhat.com>,
    linux-nfs@vger.kernel.org, netdev@vger.kernel.org,
    linux-kernel@vger.kernel.org

Those commits are part of the following research:
https://github.com/QiushiWu/QiushiWu.github.io/blob/main/pap...

They introduce kernel bugs on purpose. Yesterday, I took a look on 4
accepted patches from Aditya and 3 of them added various severity security
"holes".

____4 accepted patches from Aditya____

сделаешь сам уже выборку по Aditya Pakki.

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

90. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  –1 +/
Сообщение от Аноним (76), 22-Апр-21, 05:57 
> но провели анализ возможности внесения четырёх патчей

Так написано в исследовании.

А теперь попробуй найти этому подтверждение. Найди те 4 вредоносных патча, которые они предложили, и которые не были реджектнуты?

> вообще контекст подачи новости на опеннете идиотский. подано так, как будто несколько неумелых программистов пытаются что-то там вставить, а их поймали. тогда как те ребята акцентировали внимание на возможности внесения гипотетических ошибок и убогости код-ревью.

Исследование акцентирует на этом внимание. И анализируют примеры, как неочевидные баги раньше попадали в ядро и фиксились там.

Но откуда известно, что им удалось внести в ядро свои баги?

> They introduce kernel bugs on purpose. Yesterday, I took a look on 4 accepted patches from Aditya and 3 of them added various severity security "holes".
> ____4 accepted patches from Aditya____

И почему-то не даёт ссылки на эти патчи.

Точнее даёт, но только на один, который написал НЕ Aditya Pakki.

> сделаешь сам уже выборку по Aditya Pakki.

Уже. Там куча патчей. Многие отказаны. По другим никаких комментов (ни принятия, ни отказа).

> вообще контекст подачи новости на опеннете идиотский. подано так, как будто несколько неумелых программистов пытаются что-то там вставить, а их поймали. тогда как те ребята акцентировали внимание на возможности внесения гипотетических ошибок и убогости код-ревью.

Может потому, что так и было?

Aditya Pakki выглядит как опытный говнокодер, насочинявший патчей в кучу разных мест.

А поскольку он говнокодер, то и патчи его часто реджектят.

Наконец GregKH психанул, и предложил забанить весь универ, чтобы они не отнимали их время. А заодно и имеющиеся патчи перепроверить, вдруг там ещё где-то говнокод пропустили. Особенно с учётом исследования, которое в этом универе опубликовали пару месяцев назад.

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

91. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (-), 22-Апр-21, 06:05 
> И почему-то не даёт ссылки на эти патчи.

пффф... если тебе нужна конкретика парси всё от этого лица, для меня достаточно указания этого факта от Leon Romanovsky <leon@kernel.org> в списке рассылки: 4 accepted patches.
всю эту лапшу в рассылке можешь сам разгребать. у меня лично нет повода не доверять кор-тим, о том, что были приняты 4 патча с багами.

> Наконец GregKH психанул, и предложил забанить
> весь универ, чтобы они не отнимали их время.
> А заодно и имеющиеся патчи перепроверить, вдруг
> там ещё где-то говнокод пропустили. Особенно с
> учётом исследования, которое в этом универе
> опубликовали пару месяцев назад.

ага, так это ты на лоре тоже пишешь?

> Чуваки из университета UMN присылали в ядро кривые патчи.
> Их заворачивал. Один завернули, другой, третий...
> После очередного кривого патча Greg KH психанул, и
> решил забанить весь универ, чтобы они не отнимали их
> время. А заодно перепроверить имеющиеся патчи, на
> случай если там ещё что-то кривое пропустили.

я тебе на лоре ответил:

у тебя интересная и убогая интерепретация событий… KH психанул только тогда, когда прочитал исследование и соотнёс это с присылаемыми патчами.. то бишь суть в том, что весь процесс строился не на скрупулёзном анализе кода, а не беспочвенном доверии то ли за красивые глазки, то ли ещё за что… они поняли что происходит только на четвёртом патче, да и то только тогда, когда в рассылке (!) скинули ссылку на этот пдф… в противном случае они бы сразу не принимали ничего от этих ребят из универа, которые проводят исследование, логично же? в итоге начали строить из себя обиженок, зачем-то подняв вопрос об неэтичности… хотя суть совсем не в этом…

> Узбагойся. Никто никого не поимел.
> Вообще ничего не произошло.

согласен. в произошедшем нет ничего такого. за страшными словами «разработка ядра» может скрываться довольно примитивная культура разработки и мягко говоря сомнительный контроль качества кода.

по сути эти чуваки произвели своего рода пентест изнутри.

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

95. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от Аноним (76), 22-Апр-21, 06:37 
> пффф... если тебе нужна конкретика парси всё от этого лица, для меня достаточно указания этого факта от Leon Romanovsky <leon@kernel.org> в списке рассылки:
> 4 accepted patches.
> у меня лично нет повода не доверять кор-тим, о том, что были приняты 4 патча с багами.

А у меня есть. Потому что этот кор-тим написал:

> From: Leon Romanovsky
> If you want to see another accepted patch that is already part of
> stable@, you are invited to take a look on this patch that has "built-in bug":
> 8e949363f017 ("net: mlx5: Add a missing check on idr_find, free buf")

Только вот этот "accepted patch" написал не он, а товарищ из Меланокса. А если кор-тим тут ошибся, то, может, и про "4 accepted patches" он тоже приврал?

Но если у тебя есть ссылки на эти патчи — поделись, мне тоже интересно.

> KH психанул только тогда, когда прочитал исследование и соотнёс это с присылаемыми патчами..

Ну да.

> то бишь суть в том, что весь процесс строился не на скрупулёзном анализе кода, а не беспочвенном доверии то ли за красивые глазки, то ли ещё за что…

Небыло никакого доверия. Поэтому куча других патчей от этих же UMN-овцев была реджектнута.

> они поняли что происходит только на четвёртом патче, да и то только тогда, когда в рассылке (!) скинули ссылку на этот пдф…

Ну да. А до этого они несколько лет реджектили патчи того же Aditya. Потому что бажный говнокод.

И патчей было далеко не четыре. Он за столько лет порядка сотни патчей накодил.

> в противном случае они бы сразу не принимали ничего от этих ребят из универа, которые проводят исследование, логично же?

Нет. Пока они не знали про исследование — они честно ревьювили и реджектили патчи. Либо допиливали их до юзабельного вида. А когда узнали — офигели от такой наглости и решили забанить весь универ. Логично же?

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

109. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от другой Аноним (?), 22-Апр-21, 08:02 
> по сути эти чуваки произвели своего рода пентест изнутри.

по сути это _внешние_ люди, а их "пентест" напоминает методы "лечения" "докторов" из https://ru.wikipedia.org/wiki/Отряд_731 на живых людях...

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

103. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  –7 +/
Сообщение от bOOster (ok), 22-Апр-21, 07:15 
Насмешил.

Такие проверки здравые люди проводят на альтернативных сборках ядра.

А во вторых - проверка указателя перед удалением это всегда Must have. Чувак Greg KH вообще дальше носа своего не видит заявляя что проверка указателя не имеет смысла. Это беспросветная тупость в многозадачных/многопоточных системах.

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

113. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +8 +/
Сообщение от Аноним (76), 22-Апр-21, 08:10 
> Чувак Greg KH вообще дальше носа своего не видит

Лол, да, Greg KH вообще в ядре не разбирается. 🙂

Ты же в курсе, кто такой Greg KH, да?

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

127. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от n00by (ok), 22-Апр-21, 09:31 
> заявляя что
> проверка указателя не имеет смысла. Это беспросветная тупость в многозадачных/многопоточных
> системах.

Заявил он по смыслу иное, цитирую:

> -    gss_release_msg(gss_msg);
> +    if (gss_msg)
> +        gss_release_msg(gss_msg);a

If you look at the code, this is impossible to have happen.

Вы подменили тезис, поставив под сомнение компетентность рецензента.

Давайте теперь вместе выполним просьбу Грега и посмотрим на код


static void
gss_pipe_destroy_msg(struct rpc_pipe_msg *msg)
{
    struct gss_upcall_msg *gss_msg = container_of(msg, struct gss_upcall_msg, msg);

    if (msg->errno < 0) {
        refcount_inc(&gss_msg->count);
        gss_unhash_msg(gss_msg);
        if (msg->errno == -ETIMEDOUT)
            warn_gssd();
        gss_release_msg(gss_msg);
    }
    gss_release_msg(gss_msg);
}

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

128. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от n00by (ok), 22-Апр-21, 09:38 

static const struct rpc_pipe_ops gss_upcall_ops_v1 = {
    .upcall        = gss_v1_upcall,
    .downcall    = gss_pipe_downcall,
    .destroy_msg    = gss_pipe_destroy_msg,
    .open_pipe    = gss_pipe_open_v1,
    .release_pipe    = gss_pipe_release,
};

/// ...

static int
rpc_pipe_release(struct inode *inode, struct file *filp)
{
    struct rpc_pipe *pipe;
    struct rpc_pipe_msg *msg;
    int last_close;

    inode_lock(inode);
    pipe = RPC_I(inode)->pipe;
    if (pipe == NULL)
        goto out;
    msg = filp->private_data;
    if (msg != NULL) {
        spin_lock(&pipe->lock);
        msg->errno = -EAGAIN;
        list_del_init(&msg->list);
        spin_unlock(&pipe->lock);
        pipe->ops->destroy_msg(msg);
    }

> Чувак Greg KH вообще дальше носа своего не видит

Вот тут  https://joindiaspora.com/uploads/images/scaled_full_b39c6332...
хорошо видно, куда направлены глаза.

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

212. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +1 +/
Сообщение от Аноним (76), 23-Апр-21, 05:44 
>[оверквотинг удален]
>    struct gss_upcall_msg *gss_msg = container_of(msg, struct gss_upcall_msg, msg);
>    if (msg->errno < 0) {
>        refcount_inc(&gss_msg->count);
>        gss_unhash_msg(gss_msg);
>        if (msg->errno == -ETIMEDOUT)
>            warn_gssd();
>        gss_release_msg(gss_msg);
>    }
>    gss_release_msg(gss_msg);
> }

Тут вся фишка в том, что `container_of(msg, struct gss_upcall_msg, msg)` возвращает разность вроде `(void*)msg - offsetof(struct gss_upcall_msg, msg)`.

То есть `gss_msg` в принципе не может быть NULL, и проверка `if (gss_msg)` абсолютно бесполезна.

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

215. "Университет Миннесоты отстранён от разработки ядра Linux за ..."  +/
Сообщение от n00by (ok), 23-Апр-21, 09:42 
>[оверквотинг удален]
>>        if (msg->errno == -ETIMEDOUT)
>>            warn_gssd();
>>        gss_release_msg(gss_msg);
>>    }
>>    gss_release_msg(gss_msg);
>> }
> Тут вся фишка в том, что `container_of(msg, struct gss_upcall_msg, msg)` возвращает разность
> вроде `(void*)msg - offsetof(struct gss_upcall_msg, msg)`.
> То есть `gss_msg` в принципе не может быть NULL, и проверка `if
> (gss_msg)` абсолютно бесполезна.

Эх :(
Мнение эксперта по тупости Грега в многозадачных/многопоточных системах теперь точно не увидим. :)

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

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




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

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