log_reader: strchr(), and some simplifying to make clang happier Clang's static analyzer thought 'line' might result in a null deref in the trillian log reader, if there was an empty "(Link: )" section (temp would be NULL), except that couldn't ever happen, because the data was *always* \n-terminated, so 'c' is alway non-NULL.