use xmalloc, dont hardcode length of string, and get rid of pointless call to memset
This commit is contained in:
parent
f8346038e9
commit
bf2d9905ca
@ -625,12 +625,11 @@ static char *process_regex_on_line(char *line, regex_t *pattern)
|
|||||||
insert_highlights, and then the line is returned. */
|
insert_highlights, and then the line is returned. */
|
||||||
|
|
||||||
int match_status;
|
int match_status;
|
||||||
char *line2 = (char *) malloc((sizeof(char) * (strlen(line) + 1)) + 64);
|
char *line2 = (char *) xmalloc((sizeof(char) * (strlen(line) + 1)) + 64);
|
||||||
char sub_line[256];
|
char sub_line[256];
|
||||||
int prev_eo = 0;
|
int prev_eo = 0;
|
||||||
regmatch_t match_structs;
|
regmatch_t match_structs;
|
||||||
|
|
||||||
memset(sub_line, 0, 256);
|
|
||||||
strcpy(line2, line);
|
strcpy(line2, line);
|
||||||
|
|
||||||
match_found = 0;
|
match_found = 0;
|
||||||
@ -638,7 +637,7 @@ static char *process_regex_on_line(char *line, regex_t *pattern)
|
|||||||
|
|
||||||
while (match_status == 0) {
|
while (match_status == 0) {
|
||||||
|
|
||||||
memset(sub_line, 0, 256);
|
memset(sub_line, 0, sizeof(sub_line));
|
||||||
|
|
||||||
if (match_found == 0)
|
if (match_found == 0)
|
||||||
match_found = 1;
|
match_found = 1;
|
||||||
|
Loading…
Reference in New Issue
Block a user