microsoft/cpprestsdk

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

オープン

#101 opened on 2016/03/23

GitHub で見る
 (1 件のコメント) (0 件のリアクション) (0 人の担当者)C++ (1,670 件のフォーク)batch import
bughelp wanted

Repository metrics

Stars
 (7,596 個のスター)
PR merge metrics
 (PR metrics pending)

説明

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 を超える文字に対して符号拡張が発生し、0x00ED ではなく 0xFFED のような誤った値が生成されることです。提案されている修正は、`utf16char` にキャストする前に `static cast<unsigned char>` を使用することです。確認するには、リポジトリ内のソースファイル `asyncrt utils.cpp` を調べ、'í'（0xED）などの Latin 1 文字で変換をテストしてください。メンテナーは修正を検証し、他のユースケースを壊さないことを確認する必要があります。
技術スタック
cpp
領域
backend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
新着
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
75