microsoft/cpprestsdk

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

Aperta

#101 aperta il 23 mar 2016

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (1670 fork)batch import
bughelp wanted

Metriche repository

Star
 (7596 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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?

Guida contributor

Direzione di ricerca
Il problema identifica un potenziale bug nella funzione `utility::conversions::latin1 to utf16` in `asyncrt utils.cpp`. Il problema è che il cast di un `char` con segno a `utf16char` (probabilmente `unsigned short`) causa l'estensione del segno per caratteri superiori a 0x80, risultando in valori errati come 0xFFED invece di 0x00ED. La correzione suggerita è utilizzare `static cast<unsigned char>` prima del cast a `utf16char`. Per confermare, esaminare il file sorgente `asyncrt utils.cpp` nel repository e testare la conversione con un carattere Latin 1 come 'í' (0xED). Il manutentore dovrebbe verificare la correzione e assicurarsi che non rompa altri casi d'uso.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75