diff options
| author | Jochen Sprickerhof <git@jochen.sprickerhof.de> | 2021-12-27 11:41:42 +0100 |
|---|---|---|
| committer | sinanmohd <pcmsinan@gmail.com> | 2023-01-12 08:21:56 +0530 |
| commit | e937b5ac4bfdddd4da6a3b8f0e986b5c9faeef57 (patch) | |
| tree | 60f72f319983a431a3f2f420d954144712544875 /st.c | |
| parent | 03ba5a1053b39352ae4620b28f7ef031a6368966 (diff) | |
Fix null pointer access in strhandle
According to the spec the argument is optional for 104, so p can be
NULL as can be tested with printf '\x1b]104\x07'. This is a regression
of 8e31030.
Diffstat (limited to 'st.c')
| -rw-r--r-- | st.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -2022,10 +2022,10 @@ strhandle(void) break; p = strescseq.args[2]; /* FALLTHROUGH */ - case 104: /* color reset, here p = NULL */ + case 104: /* color reset */ j = (narg > 1) ? atoi(strescseq.args[1]) : -1; - if (!strcmp(p, "?")) + if (p && !strcmp(p, "?")) osc4_color_response(j); else if (xsetcolorname(j, p)) { if (par == 104 && narg <= 1) |