В ядре Linux устранена следующая уязвимость:
mm/damon/core: всегда ставить неудачно зафиксированные целевые pids
damon_commit_target() помещает и получает пункт назначения и исходную цель
пиды. Он помещает целевой идентификатор назначения, потому что он будет перезаписан.
по исходному целевому pid. Он получает исходный идентификатор, потому что вызывающий абонент
предполагалось в конечном итоге поставить пиды.
Более подробно звонящий позвонит
damon_destroy_ctx() после damon_commit_ctx(), чтобы уничтожить весь исходный код
контекст. В данном случае, очистка_target() набора операций [f]vadr
обратный вызов поместит PID. Операция фиксации выполняется на уровне контекста.
Операция может провалиться
в нескольких местах, в том числе в середине и после фиксации целей
операции. При любых таких сбоях ошибка немедленно возвращается в
вызывающий damon_commit_ctx(). Если некоторые или все исходные целевые идентификаторы
были зафиксированы в пункте назначения во время неудачной фиксации контекста
попытка, эти PID должны быть установлены дважды.
Исходный контекст будет выполнять операции put, используя описанное выше.
рутина. Однако предположим, что контекст назначения не был
первоначально использовался набор операций [f]vaddr, и фиксация не удалась до того, как
операции исходного контекста зафиксированы. Пункт назначения не имеет
Обратный вызов cleanup_target(), поэтому он не может поместить PID через
damon_destroy_ctx().
В результате происходит утечка пидов. Проблема в реальном мире будет
не очень часто. Функция фиксации предназначена для изменения параметров запуска
Контекст DAMON, наследуя внутренний статус, такой как мониторинг.
результаты.
Результаты мониторинга диапазона физических адресов не имеют
вещи, которые полезно наследовать в диапазонах виртуальных адресов
мониторинг. Таким образом, вызывающее проблемы управление DAMON будет не очень распространенным явлением.
в реальном мире. Тем не менее, это поддерживаемая функция.
И
Сбой damon_commit_target() из-за выделения памяти относительно
реалистично [1] при наличии огромного количества целевых регионов. Исправьте, поместив pids в операцию фиксации на случай сбоев. Проблема была обнаружена [2] Сашико.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: mm/damon/core: always put unsuccessfully committed target pids damon_commit_target() puts and gets the destination and the source target pids. It puts the destination target pid because it will be overwritten by the source target pid. It gets the source pid because the caller is supposed to eventually put the pids. In more detail, the caller will call damon_destroy_ctx() after damon_commit_ctx() to destroy the entire source context. And in this case, [f]vaddr operation set's cleanup_target() callback will put the pids. The commit operation is made at the context level. The operation can fail in multiple places including in the middle and after the targets commit operations. For any such failures, immediately the error is returned to the damon_commit_ctx() caller. If some or all of the source target pids were committed to the destination during the unsuccessful context commit attempt, those pids should be put twice. The source context will do the put operations using the above explained routine. However, let's suppose the destination context was not originally using [f]vaddr operation set and the commit failed before the ops of the source context is committed. The destination does not have the cleanup_target() ops callback, so it cannot put the pids via the damon_destroy_ctx(). As a result, the pids are leaked. The issue in the real world would be not very common. The commit feature is for changing parameters of running DAMON context while inheriting internal status like the monitoring results. The monitoring results of a physical address range ain't have things that are beneficial to be inherited to a virtual address ranges monitoring. So the problem-causing DAMON control would be not very common in the real world. That said, it is a supported feature. And damon_commit_target() failure due to memory allocation is relatively realistic [1] if there are a huge number of target regions. Fix by putting the pids in the commit operation in case of the failures. The issue was discovered [2] by Sashiko.