[RESEND PATCH v3 2/2] cli: Correct handling of invalid escape sequences in cread_line()
Heinrich Schuchardt
xypron.glpk at gmx.de
Tue Mar 28 18:43:50 CEST 2023
On 3/27/23 21:34, Simon Glass wrote:
> The second call to cli_ch_process() is in the wrong place, meaning that
> the one of the characters of an invalid escape sequence is swallowed
> instead of being returned.
>
> Fix the bug and add a test to cover this.
>
> This behaviour matches that of the code before cli_getch() was
> introduced. This was verified on the commit before b08e9d4b66 i.e.:
>
> 7d850f85aad ("sandbox: Enable mmc command and legacy images")
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Reported-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
Thanks for the fix. Some strange behavior still remains like
<SHIFT><CTRL><F4> giving '6S' on the command line. But at least no error
code.
Best regards
Heinrich
More information about the U-Boot
mailing list