microsoft/cpprestsdk

utility::conversions::latin1_to_utf16 - Is there a bug for characters above 0x80?

Open

#101 创建于 2016年3月23日

在 GitHub 查看
 (1 评论) (0 反应) (0 负责人)C++ (1,670 fork)batch import
bughelp wanted

仓库指标

Star
 (7,596 star)
PR 合并指标
 (30 天内没有已合并 PR)

描述

I've tried to use utility::conversions::latin1_to_utf16 to convert latin1 string to UTF-16. The string contained 'í' character (0xed). After converting it should become 0x00ed but it seems that converting method replaced it with 0xffed.

I think this may be result of casting signed char to utf16char and dest[i] = utf16char(s[i]); in asyncrt_utils.cpp should be replaced by dest[i] = utf16char(static_cast<unsigned char>(s[i])); to fix it. Can someone check if I'm right?

贡献者指南

研究方向
该问题指出 `asyncrt utils.cpp` 中 `utility::conversions::latin1 to utf16` 函数存在一个潜在缺陷。问题在于将有符号 `char` 强制转换为 `utf16char`（可能为 `unsigned short`）时，对于大于 0x80 的字符会导致符号扩展，从而产生错误值（例如 0xFFED 而非 0x00ED）。建议的修复方法是在转换为 `utf16char` 前先使用 `static cast<unsigned char>`。为确认此问题，请检查仓库中的源文件 `asyncrt utils.cpp`，并使用拉丁 1字符（如 'í'，对应 0xED）测试转换功能。维护者应验证此修复，并确保不会破坏其他用例。
技术栈
cpp
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
75