В ядре Linux устранена следующая уязвимость:
iio: accel: bmc150: ограничить количество кадров FIFO, сообщаемое устройством.
__bmc150_accel_fifo_flush() копирует количество выборок устройства
отчеты из своего аппаратного FIFO в буфер стека
буфер u16 [BMC150_ACCEL_FIFO_LENGTH * 3];
размер которого рассчитан не более чем на BMC150_ACCEL_FIFO_LENGTH (32) сэмплов.
количество кадров считывается из регистра FIFO_STATUS и маскируется только по его значению.
7 действительных битов:
счет = значение & 0x7F;
так что это может быть 0..127. Единственное другое ограничение, применяемое к нему, - это необязательное
Примерный бюджет, предоставленный вызывающим абонентом:
if (образцы && количество > образцы)
количество = образцы;
который не ограничивает количество на пути очистки всех (выборки == 0) и
оставляет его значительно выше 32, когда выборки больше. образцы подсчета
затем передается в буфер[]:
bmc150_accel_fifo_transfer(данные, (u8 *)буфер, количество);
bmc150_accel_fifo_transfer() считывает count * 6 байт через regmap, поэтому
неисправный, вредоносный или поддельный акселерометр (или злоумышленник
вмешательство в шину I2C/SPI), который сообщает о записи до 127 кадров.
до 762 байт в 192-байтовый буфер: запись up за пределами стека
до 570 байт, что забивает канарейку стека, сохраненные регистры и
обратный адрес. Ограничить счетчик до BMC150_ACCEL_FIFO_LENGTH, количество выборок в буфере[]
размер рассчитан на то, чтобы перед переносом зеркально отразить зажим водяного знака
сделано в bmc150_accel_set_watermark().
Максимум хорошо сформированные флеш-отчеты Кадры BMC150_ACCEL_FIFO_LENGTH, поэтому законные устройства не затрагиваются.
Показать оригинальное описание (EN)
In the Linux kernel, the following vulnerability has been resolved: iio: accel: bmc150: clamp the device-reported FIFO frame count __bmc150_accel_fifo_flush() copies the number of samples the device reports in its hardware FIFO into an on-stack buffer u16 buffer[BMC150_ACCEL_FIFO_LENGTH * 3]; which is sized for at most BMC150_ACCEL_FIFO_LENGTH (32) samples. The frame count is read from the FIFO_STATUS register and only masked to its 7 valid bits: count = val & 0x7F; so it can be 0..127. The only other limit applied to it is the optional caller-supplied sample budget: if (samples && count > samples) count = samples; which does not constrain count on the flush-all path (samples == 0), and leaves it well above 32 whenever samples is larger. count samples are then transferred into buffer[]: bmc150_accel_fifo_transfer(data, (u8 *)buffer, count); bmc150_accel_fifo_transfer() reads count * 6 bytes through regmap, so a malfunctioning, malicious or counterfeit accelerometer (or an attacker tampering with the I2C/SPI bus) that reports up to 127 frames writes up to 762 bytes into the 192-byte buffer: a stack out-of-bounds write of up to 570 bytes that clobbers the stack canary, saved registers and the return address. Clamp count to BMC150_ACCEL_FIFO_LENGTH, the number of samples buffer[] is sized for, before the transfer, mirroring the watermark clamp already done in bmc150_accel_set_watermark(). A well-formed flush reports at most BMC150_ACCEL_FIFO_LENGTH frames, so legitimate devices are unaffected.