sbase

suckless unix tools
git clone git://git.suckless.org/sbase
Log | Files | Refs | README | LICENSE

commit 9f974301438f0063637e1330cdb829df12cd5081
parent 1e0c3a0ba60c3b65344efacd2e6a879ed7e5e0f7
Author: Hiltjo Posthuma <hiltjo@codemadness.org>
Date:   Sun, 29 Mar 2015 21:48:49 +0200

libutil/getlines: fix crash with no lines

because b->lines and b->nlines would be 0 with no lines read.

reproduce: printf '' | sort or cols

bug was introduced by commit: 66a5ea722d18fc76ce7c5c4323e8cfebe58e7517

Diffstat:
Mlibutil/getlines.c | 7++++---
1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/libutil/getlines.c b/libutil/getlines.c @@ -9,20 +9,21 @@ void getlines(FILE *fp, struct linebuf *b) { - char *line = NULL, **nline; + char *line = NULL; size_t size = 0, linelen; ssize_t len; while ((len = getline(&line, &size, fp)) > 0) { if (++b->nlines > b->capacity) { b->capacity += 512; - nline = erealloc(b->lines, b->capacity * sizeof(*b->lines)); - b->lines = nline; + b->lines = erealloc(b->lines, b->capacity * sizeof(*b->lines)); } linelen = len + 1; b->lines[b->nlines - 1] = memcpy(emalloc(linelen), line, linelen); } free(line); + if(!b->nlines || !b->lines) + return; if (!strchr(b->lines[b->nlines - 1], '\n')) { b->lines[b->nlines - 1] = erealloc(b->lines[b->nlines - 1], linelen + 1); b->lines[b->nlines - 1][linelen - 1] = '\n';