philippelh/bitchx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Switch stripansi() to stripansicodes() in the CTCP reply parser.

Browse Source 
stripansi() actually just munged control-characters, so it just turned ANSI
codes into visible junk.

stripansicodes() instead leaves control characters alone but removes complete
ANSI sequences from the string.  I'm not sure if even that is necessary,
really - after all, we always allowed ^B, ^C etc style formatting in CTCP
replies.
This commit is contained in:
Kevin Easton
2015-08-30 22:03:38 +10:00
parent 498631e74a
commit 75bd647d15
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
source/ctcp.c
View File

@@ -1323,7 +1323,6 @@ extern char *do_notice_ctcp (char *from, char *to, char *str)
*ctcp_argument;
int i;
char *ptr;
char *tbuf = NULL;
int allow_ctcp_reply = 1;
#ifdef WANT_DLL
@@ -1341,9 +1340,7 @@ extern char *do_notice_ctcp (char *from, char *to, char *str)
if (!in_ctcp_flag)
in_ctcp_flag = -1;
tbuf = stripansi(str);
strlcpy(local_ctcp_buffer, tbuf, sizeof local_ctcp_buffer - 2);
new_free(&tbuf);
strlcpy(local_ctcp_buffer, stripansicodes(str), sizeof local_ctcp_buffer - 2);
for (;;strlcat(local_ctcp_buffer, last, sizeof local_ctcp_buffer - 2))
{