В ядре Linux устранена следующая уязвимость:
iio: event: исправить гонку сброса FIFO события.
`iio_event_getfd()` создает дескриптор файла события с
`anon_inode_getfd()`, который выделяет новый fd, создает анонимный
файл и устанавливает его в таблицу fd процесса, прежде чем вернуться к
звонящий. Код IIO сбрасывает FIFO событий после возврата `anon_inode_getfd()`,
но до того, как `IIO_GET_EVENT_FD_IOCTL` скопировал номер fd в пространство пользователя. Но поскольку таблицы fd используются потоками совместно, другой поток может догадаться,
вновь выделенный номер fd и выполнить для него `read()`, как только fd
был установлен.
Это означает, что `kfifo_to_user()` в `iio_event_chrdev_read()` может работать в
параллельно с `kfifo_reset_out()` в `iio_event_getfd()`. В документации kfifo говорится, что `kfifo_reset_out()` безопасен только тогда, когда он
вызывается из потока чтения, и существует только один одновременный модуль чтения. В противном случае это опасно, и с ним следует обращаться так же, как и с
`kfifo_reset()`.
Если это произойдет, `kfifo_to_user()` может переместить индекс `out` FIFO на основе
состояние до сброса, после сброса уже переместился `out`
index к текущему индексу `in`. Это может привести к тому, что ФИФО останется с «выходом».
индекс после индекса `in`. Более поздний вызов `read()` может увидеть опустошенное сообщение.
Длина FIFO и копирование большего количества данных, чем содержит буфер FIFO событий. Это
может привести к выходу за пределы чтения и утечке соседней памяти ядра в
пространство пользователя. Переместите сброс FIFO перед `anon_inode_getfd()`.
В этот момент событие fd
помечен как занятой, но новый диск еще не установлен, поэтому пользовательское пространство не может
получить к нему доступ, пока FIFO сброшен.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: iio: event: Fix event FIFO reset race `iio_event_getfd()` creates the event file descriptor with `anon_inode_getfd()`, which allocates a new fd, creates the anonymous file and installs it in the process fd table before returning to the caller. The IIO code resets the event FIFO after `anon_inode_getfd()` has returned, but before `IIO_GET_EVENT_FD_IOCTL` has copied the fd number to userspace. But since fd tables are shared between threads, another thread can guess the newly allocated fd number and issue a `read()` on it as soon as the fd has been installed. This means the `kfifo_to_user()` in `iio_event_chrdev_read()` can run in parallel with the `kfifo_reset_out()` in `iio_event_getfd()`. The kfifo documentation says that `kfifo_reset_out()` is only safe when it is called from the reader thread and there is only one concurrent reader. Otherwise it is dangerous and must be handled in the same way as `kfifo_reset()`. If that happens, `kfifo_to_user()` can advance the FIFO `out` index based on state from before the reset, after the reset has already moved the `out` index to the current `in` index. That can leave the FIFO with an `out` index past the `in` index. A later `read()` can then see an underflowed FIFO length and copy more data than the event FIFO buffer contains. This can result in an out-of-bounds read and leak adjacent kernel memory to userspace. Move the FIFO reset before `anon_inode_getfd()`. At that point the event fd is marked busy, but the new fd has not been installed yet, so userspace cannot access it while the FIFO is reset.
Характеристики атаки
Последствия
Строка CVSS v3.1