microsoft/cpprestsdk

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

開放

#101 建立於 2016年3月23日

在 GitHub 查看
 (1 則留言) (0 個反應) (0 位負責人)C++ (1,670 個分叉)batch import
bughelp wanted

倉庫指標

星標
 (7,596 顆星)
PR 合併指標
 (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