From: Thomas Hindoe Paaboel Andersen Date: Sat, 5 Apr 2014 19:05:22 +0000 (+0200) Subject: journal-remote-parse: avoid passing null to memchr X-Git-Tag: v213~468 X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=609211792b83267ca202b40f25d1755a004c8c96;p=elogind.git journal-remote-parse: avoid passing null to memchr Found with scan-build --- diff --git a/src/journal/journal-remote-parse.c b/src/journal/journal-remote-parse.c index 142de0ed1..239ff3819 100644 --- a/src/journal/journal-remote-parse.c +++ b/src/journal/journal-remote-parse.c @@ -40,7 +40,7 @@ void source_free(RemoteSource *source) { static int get_line(RemoteSource *source, char **line, size_t *size) { ssize_t n, remain; - char *c; + char *c = NULL; char *newbuf = NULL; size_t newsize = 0; @@ -49,7 +49,9 @@ static int get_line(RemoteSource *source, char **line, size_t *size) { assert(source->filled <= source->size); assert(source->buf == NULL || source->size > 0); - c = memchr(source->buf, '\n', source->filled); + if (source->buf) + c = memchr(source->buf, '\n', source->filled); + if (c != NULL) goto docopy;