Picom 窗口状态更新导致的崩溃问题分析与修复

问题背景

在Picom（一个流行的X11窗口合成器）的最新版本v12.1中，用户报告了一个严重的崩溃问题。当用户尝试在mpv视频播放器中切换全屏模式时，Picom会意外崩溃。这个问题特别出现在用户配置了针对_NET_WM_STATE_HIDDEN和_NET_WM_STATE_FULLSCREEN窗口状态的透明度规则时。

技术分析

通过深入分析崩溃日志和代码，我们发现问题的根源在于c2_window_state_update_one_from_reply函数中的内存对齐问题。具体表现为：

1. 内存对齐问题：当尝试将一个64位整型(int64_t)写入一个未对齐的内存地址(0x164)时，触发了硬件异常。64位数据类型需要8字节对齐，而目标地址显然不满足这一要求。

2. 窗口属性类型变更处理不足：核心问题在于Picom未能正确处理窗口属性类型的变化。当窗口状态属性从一种类型变为另一种类型时，系统没有进行适当的类型转换或内存重新分配，导致后续的内存访问越界。

3. X11协议交互问题：这个问题特别出现在处理X11窗口管理器状态属性(_NET_WM_STATE)更新时，尤其是当窗口在全屏和普通状态间切换时。

问题复现条件

要触发这个崩溃，需要满足以下条件：

• 使用i3窗口管理器环境
• 配置了针对_NET_WM_STATE_HIDDEN或_NET_WM_STATE_FULLSCREEN的透明度规则
• 使用mpv播放器进行全屏切换操作

解决方案

开发者通过以下方式解决了这个问题：

1. 完善属性类型变更处理：修改了c2_window_state_update_one_from_reply函数，使其能够正确处理窗口属性类型的变化情况。

2. 内存安全增强：增加了对内存访问的检查，确保不会写入未对齐或无效的内存地址。

3. 状态更新逻辑优化：改进了窗口状态更新的整体流程，使其更加健壮和可靠。

技术影响

这个修复不仅解决了mpv全屏切换时的崩溃问题，还提升了Picom处理动态窗口状态变化的整体稳定性。特别是对于以下场景有显著改善：

• 窗口在全屏和普通状态间快速切换
• 窗口隐藏和显示操作
• 窗口属性动态变化的情况

用户建议

对于遇到类似问题的用户，建议：

1. 更新到包含此修复的Picom版本
2. 如果无法立即更新，可以暂时移除针对_NET_WM_STATE相关状态的透明度规则
3. 在配置窗口规则时，注意测试各种窗口状态变化场景

这个问题的修复体现了开源社区快速响应和解决问题的能力，也提醒我们在处理窗口系统属性时需要特别注意内存安全和类型一致性。