midfavila/sbase
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

libutil/getlines: fix potential crash

Browse Source 
linelen was uninitialized if for example:

$ > empty
$ sort ls.c empty
This commit is contained in:
Hiltjo Posthuma 2015-05-08 17:10:39 +02:00 committed by sin
parent e3cdcce2ec
commit 29649762b3
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libutil/getlines.c
View File

@ -10,7 +10,7 @@ void
getlines(FILE *fp, struct linebuf *b) getlines(FILE *fp, struct linebuf *b)
{ {
char *line = NULL; char *line = NULL;
size_t size = 0, linelen; size_t size = 0, linelen = 0;
ssize_t len; ssize_t len;
while ((len = getline(&line, &size, fp)) > 0) { while ((len = getline(&line, &size, fp)) > 0) {
@ -22,7 +22,7 @@ getlines(FILE *fp, struct linebuf *b)
b->lines[b->nlines - 1] = memcpy(emalloc(linelen + 1), line, linelen + 1); b->lines[b->nlines - 1] = memcpy(emalloc(linelen + 1), line, linelen + 1);
} }
free(line); free(line);
if (b->lines && b->nlines && b->lines[b->nlines - 1][linelen - 1] != '\n') { if (b->lines && b->nlines && linelen && b->lines[b->nlines - 1][linelen - 1] != '\n') {
b->lines[b->nlines - 1] = erealloc(b->lines[b->nlines - 1], linelen + 2); b->lines[b->nlines - 1] = erealloc(b->lines[b->nlines - 1], linelen + 2);
b->lines[b->nlines - 1][linelen] = '\n'; b->lines[b->nlines - 1][linelen] = '\n';
b->lines[b->nlines - 1][linelen + 1] = '\0'; b->lines[b->nlines - 1][linelen + 1] = '\0';