qmk/qmk_firmware

[Bug] UCIS input not cleared after resolving mnemonic

Open

#22.438 aperta il 10 nov 2023

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

Metriche repository

Star
 (20.368 star)
Metriche merge PR
 (Merge medio 20g 9h) (27 PR mergiate in 30 g)

Descrizione

Describe the Bug

After resolving a unicode mnemonic, successfully or not, the input buffer should be cleared. In the current code the old input is kept in the buffer, potentially leading to incorrect resolution of future inputs.

Ex. enter the mnemonic "alpha", input is now "alpha". If you then enter the mnemonic "beta", input contains "betaa", which is incorrect.

I suggest using memset to zero out the input buffer at the end of ucis_finish.

Am I missing something important?

Keyboard Used

No response

Link to product page (if applicable)

No response

Operating System

No response

qmk doctor Output

No response

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

No response

Guida contributor

Direzione di ricerca
Esamina la funzione ucis finish nel codebase QMK per comprendere il comportamento attuale. Aggiungi memset per cancellare il buffer di input dopo la risoluzione del mnemonico. Assicurati che nessun'altra parte del codice faccia affidamento sulla conservazione del buffer. Testa con più mnemonici in sequenza.
Tech stack
c
Dominio
embedded iot
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CGitQMK
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70