snark/st
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Do not eat ESC character if control string is not properly terminated.

Browse source 
Currently tputc handles the case of too long control string waiting for
the end of control string.

Another case is when there is ESC character is encountered but is not
followed by '\\'.  In this case st stops processing control string,
but ESC character is ignored.

After this patch st processes ESC characters in control strings properly.

Test case:
printf '\e]0;abc\e[1mBOLD\e[0m'

Also ^[\ is actually processed in the code that handles ST.
According to ECMA-048 ST stands for STRING TERMINATOR and is used to
close control strings.
This commit is contained in:
noname 2014-04-26 00:12:41 +02:00 committed by Roberto E. Vargas Caballero
parent c4b79b055d
commit 02d2df5790
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
st.c
View file

@ -2452,10 +2452,6 @@ tputc(char *c, int len) {
csiparse(); csiparse();
csihandle(); csihandle();
} }
} else if(term.esc & ESC_STR_END) {
term.esc = 0;
if(ascii == '\\')
strhandle();
} else if(term.esc & ESC_ALTCHARSET) { } else if(term.esc & ESC_ALTCHARSET) {
tdeftran(ascii); tdeftran(ascii);
tselcs(); tselcs();
@ -2545,7 +2541,9 @@ tputc(char *c, int len) {
tcursor(CURSOR_LOAD); tcursor(CURSOR_LOAD);
term.esc = 0; term.esc = 0;
break; break;
case '\\': /* ST -- Stop */ case '\\': /* ST -- String Terminator */
if(term.esc & ESC_STR_END)
strhandle();
term.esc = 0; term.esc = 0;
break; break;
default: default: