syslogd: Fix handling of configuration errors

[markj]

Summary

• If parse_selector() returns NULL, we should exit immediately.
• Add newlines to a couple of debug messages. Otherwise they don't get printed if the process crashes before the stdio buffer is flushed.

Reported by: Doug Hardie <[e-mail]>
Fixes: f4b4a10abb26 ("syslogd: Move selector parsing into its own function")

[eugen_grosbein.net]

Since our syslogd's default configuration started including additional configurations installed with non-base packages, it is no more acceptable for it to use err*() functions based on syntax errors in such configuration files. Instead, it should issue a warning, skip bad line and continue with execution. Please do not add new err*() to the syslogd but warn*() with graceful recovery.

[jfree]

Like eugen suggested, it might be better to skip the line and log a warning on syntax parse failure instead of terminating. It looks like this would require some more surgery to handle other parse errors though.

This otherwise looks good. Perhaps it might be better to ship this now to fix the bug. A separate feature request could be opened for better parser error handling.