fix: 修复DMA-BUF平面访问缺少边界检查的问题 · Pull Request !366 · openKylin（孵化期）/ukui-quick

# 问题描述

在DMA-BUF缓冲区处理流程中，存在对固定大小数组缺少边界检查的问题，可能导致越界访问。
## 1. bufferWithPlaneCallback() 中的越界写入
文件：
`platform/wayland/kywc-capture-manager.cpp`
函数：
`KywcCaptureFrame::Private::bufferWithPlaneCallback()`
`BufferInfo::planes` 为固定大小的四元素数组：
```cpp
planes[4]
```
但来自 Wayland 协议事件 `buffer_with_plane` 的 `index` 参数被直接作为数组下标使用：
```cpp
p->currentBuffer.planes[index].fd = fd;
```
当收到异常的 `index` 值时，可能导致数组越界写入。
## 2. createDmabufResources() 中的越界读取
文件：
`modules/window-thumbnail/core/graphics-memory-buffer.cpp`
函数：
`GraphicsMemoryBuffer::createDmabufResources()`
函数内部定义了固定大小的数组：
```cpp
attr_names[4]
```
但循环次数直接使用：
```cpp
m_bufferInfo.n_planes
```
作为上限。
当 `n_planes` 超出数组容量时，可能访问 `attr_names` 和 `planes` 数组边界之外的数据。
# 分析
同文件中的 `releaseBuffer()` 已通过：
```cpp
std::clamp(d->currentBuffer.n_planes, 0, 4)
```
限制平面数量范围，说明当前实现默认最大平面数为4。
因此在数据写入路径和消费路径中增加边界检查，可以避免异常数据导致的越界访问，同时不会影响正常功能。
# 修复内容
## bufferWithPlaneCallback()
增加索引范围检查：
```cpp
if (index >= 4) {
    close(fd);
    return;
}
```
当收到非法索引时直接返回，并关闭接收到的文件描述符。
## createDmabufResources()
限制循环上界：
```cpp
const uint32_t planeCount = qMin(m_bufferInfo.n_planes, 4u);
```
确保访问固定大小数组时不会发生越界。
# 影响范围
本次修改仅增加边界检查逻辑。
对于合法输入行为保持不变，仅在收到异常平面索引或异常平面数量时进行保护处理，避免潜在的越界访问问题。
## 说明

本次提交来源于软件分析与安全课程的开源贡献实践。
由于课程作业存在时间要求，如有不妥之处欢迎指出，我会及时修改和完善。感谢审阅。

类型	指派人员	状态
审查	youdiansaodongxi static_assert iaom	进行中 (0/2人)
测试	youdiansaodongxi static_assert iaom	进行中 (0/1人)

openKylin（孵化期）/ukui-quick .gitee-modal { width: 500px !important; }

搜索帮助

openKylin（孵化期）/ukui-quick