LibreCAD中Line Angle动作导致的程序崩溃问题分析

问题概述

在LibreCAD 2.2.2-alpha版本中，当用户尝试使用"Line Angle"（角度线）功能时，程序会触发错误并崩溃。这个问题主要出现在使用Qt6构建的版本中，其根本原因与字符串处理中的潜在风险有关。

技术背景

LibreCAD是一款开源的2D CAD设计软件，使用C++编写，基于Qt框架。在图形处理中，角度线功能需要处理各种数值格式，包括去除数字字符串中不必要的零。

问题根源

崩溃发生在rs_dimensions.cpp文件的stripZerosLinear函数中，具体位置是对QString进行前导零去除操作时。Qt6的QString实现在调试模式下加入了更严格的检查，暴露了原本就存在的潜在问题。

详细分析

问题代码段如下：

while (i < ls-1 && linear[i] == QChar('0')) {
    i++;
}

这里存在三个关键问题：

1. 字符串长度ls在初始化后被固定保存
2. 字符串可能被从尾部修剪（如去除小数点后的零），但ls值未同步更新
3. 随后进行前导零去除时，使用过时的ls值可能导致访问越界

在Qt5中，由于QString内部缓冲区的实现方式，这种访问可能不会立即导致问题。但Qt6引入了更严格的检查机制，使得这种行为被立即捕获。

解决方案

正确的实现应该：

1. 在每次字符串修改后重新获取其长度
2. 确保所有索引访问都在有效范围内
3. 避免依赖字符串缓冲区的未定义行为

修复后的代码应该动态获取字符串长度，而不是依赖可能过时的长度变量。

影响范围

虽然这个问题在Line Angle功能中最先被发现，但类似的字符串处理模式可能存在于代码库的其他部分。开发团队已经将该修复反向移植到2.2.1发布候选分支，以防止影响更多用户。

经验教训

这个案例展示了：

1. 编译器/库版本升级可能暴露隐藏的代码问题
2. 字符串处理需要特别注意边界条件
3. 检查机制是发现潜在问题的有效工具
4. 未定义行为可能在特定环境下"正常工作"，但本质上是不安全的

对于开源项目维护者，这个案例强调了全面测试和代码审查的重要性，特别是在依赖库升级时。