В ядре Linux устранена следующая уязвимость:
PCI/sysfs: исправлено чтение за пределами границ в pci_write_legacy_io().
pci_write_legacy_io() загружает 4 байта из буфера записи kernfs
независимо от того, сколько байт пользовательское пространство написало:
if (count != 1 && count != 2 && count != 4)
вернуть -ЭИНВАЛ;
return pci_legacy_write(bus, off, *(u32 *)buf, count);
kernfs_fop_write_iter() выделяет буфер с помощью kmalloc(len + 1),
поэтому запись 1 байта в файл sysfs Legacy_io выделяет 2 байта и
безусловная загрузка u32 считывает до 2 байтов после конца
распределение, о котором KASAN сообщает как о чтении за пределами поля. Аналогично, 2-байтовая запись пересчитывается на 1 байт. Таким образом, читайте только то количество байтов, которое запрошено с помощью get_unaligned_le16().
и get_unaligned_le32() для случаев с 2 и 4 байтами, интерпретируя
буфер с прямым порядком байтов, чтобы соответствовать порядку байтов порта ввода-вывода PCI
пространство.
Реализация PowerPC ранее компенсировала общий
32-битная загрузка кода с собственным порядком байтов путем смещения значения на место
для случаев 1 и 2 байта. Переключения были правильными только
ядра с прямым порядком байтов. На PowerPC с прямым порядком байтов (POWER8 и новее) они извлекли неправильный
байт, поэтому при записи 1 байта вместо байта записывается выходной байт.
запрошенное значение.
В случае с прямым порядком байтов собственная загрузка также вызывала out_le16().
и out_le32() для изменения байтов пользователя в проводе на 2 и 4 байта.
пишет. Помощники с прямым порядком байтов решают обе проблемы, поэтому сдвиги
удаляются. Для платформы Alpha никаких изменений не требуется.
Файл Legacy_io предназначен только для root-доступа и существует только на Alpha и PowerPC.
две архитектуры, определяющие HAVE_PCI_LEGACY.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: PCI/sysfs: Fix out-of-bounds read in pci_write_legacy_io() pci_write_legacy_io() loads 4 bytes from the kernfs write buffer regardless of how many bytes userspace wrote: if (count != 1 && count != 2 && count != 4) return -EINVAL; return pci_legacy_write(bus, off, *(u32 *)buf, count); kernfs_fop_write_iter() allocates the buffer with kmalloc(len + 1), so a 1-byte write to the legacy_io sysfs file allocates 2 bytes and the unconditional u32 load reads up to 2 bytes past the end of the allocation, which KASAN reports as a slab-out-of-bounds read. Similarly, a 2-byte write overreads by 1 byte. Thus, read only the number of bytes requested using get_unaligned_le16() and get_unaligned_le32() for the 2 and 4 byte cases, interpreting the buffer as little-endian to match the byte ordering of PCI I/O port space. The PowerPC implementation previously compensated for the generic code's native-endian 32-bit load by shifting the value into place for the 1 and 2 byte cases. The shifts were only correct on big-endian kernels. On little-endian PowerPC (POWER8 and later), they extracted the wrong bytes, so a 1-byte write wrote an out-of-bounds byte instead of the requested value. On big-endian, the native load also caused out_le16() and out_le32() to reverse the user's bytes on the wire for 2 and 4 byte writes. The little-endian helpers resolve both issues, so the shifts are removed. No changes are needed for the Alpha platform. The legacy_io file is root-only and exists only on Alpha and PowerPC, the two architectures that define HAVE_PCI_LEGACY.