Re: [PATCH] Allow syslogd to accept multiple configuration files

On Wed, Nov 09, 2011 at 10:27:44AM -0500, Ryan Stone wrote:
> I've written the following patch to allow syslogd to accept multiple
> configuration files by passing multiple -f options.  One use case for
> this is to specify a common configuration file that applies across
> multiple machines along with a second config file specific to the
> local machine.
> 
> The patch can also be found at
> http://people.freebsd.org/~rstone/patches/syslogd-multiconf.diff
> 
> (Oh, and before somebody asks, the reason that I converted struct
> filed to use a TAILQ was that at one point I found myself typing
> struct filed ***, then hit myself and refactored the code instead).

I gave it a once over and it looks good.

Do you happen to know why the code calloc's the struct filed's with 1's?
I didn't do any investigation but that's seems like an odd pattern.

-- Brooks