utils_cmd_*: Terminate all lines printed to the socket with newline.

This is a backport of 1ca0b79a4274784e4cc15d45e46e2cfa0c2039cf.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

contrib/redhat/collectd.spec: Updated the description of the package.

Also updated the packager and vendor fields and added the redhat/ subdirectory
to contrib/README.

contrib/redhat/: Added a spec file and affiliated config files.

Merge branch 'collectd-4.2' into collectd-4.3

perl plugin: Fixed the "EnableDebugger after LoadPlugin" warning.

For some strange reason my original patch (perl plugin: Warn if
"EnableDebugger" has been used after "LoadPlugin".) has been applied at the
wrong position (it has been applied to perl_config_includedir() instead of
perl_config_enabledebugger() - I have absolutely no clue why though).

Also, the check for the precondition of this warning has been fixed. In some
cases aTHX does not seem to be set, even though the Perl interpreter has
already been initialized. Now, perl_threads is used to check for that
condition.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

configfile.c: Include more than one files in lexicographical order.

Using qsort() and strcmp() the list of files (after reading the contents
of a directory or expanding globs) is sorted before inclusion. As the
order of options in the config file matters this is more convenient.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

liboconfig/oconfig.c: Free all allocated memory in oconfig_free().

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

configfile.c: Abort if any config file could not be read.

So far, when including all files from some directory, errors while reading
any of those config files have been reported but otherwise ignored. So,
collectd would run with some potentially incomplete configuration which is
not what I would expect.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

configfile.c: Fixed a memory leak in cf_read_generic().

In case stat(2) failed, the memory allocated for the return value has not
been freed.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

configfile.c: Fixed a segfault after a parse error.

In cf_read_generic(), the parse result had not been checked to not be
NULL, which caused a segfault when trying to access any of its members.
Now, an error will be returned in that case.

Also, cf_ci_append_children() has been made more robust in that respect.
It now detects an empty source and does nothing in that case.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

AUTHORS: Speak about `plugins', not `modules'.

Also sorted the plugins and put them all before any other patches.

Merge branch 'pull/collectd-4.3' into collectd-4.3

src/utils_{cache,threshold}.c: Fix the concept of failed and missing values.

Apparently I was confused at the time - and still am ;)

Merge branch 'pull/collectd-4.3' into collectd-4.3

src/utils_cache.c: Fix the handling of `Persist'.

And reorganized/cleaned up the code a bit.

contrib/rrd_filter.px: Added the ability to add (empty) data sources.

src/{mbmon,plugin,unixsock}.c: Simple fixes for Solaris 5.8

contrib/rrd_filter.px: Added a scale and shift filter.

src/utils_threshold.c: Fix the configuration of {Failure,Warning}Max and inverted checking.

The first issue is a simple copy'n'paste error, the second a nasty oversight.

contrib/rrd_filter.px: Remove the (unused) Data::Dumper.

Bumped version to 4.3.1; Updated ChangeLog.

contrib/rrd_filter.px: Renamed the old `extractDS.px'.

It can do more than extracting DSes now.

network plugin: Use `sstrncpy' instead of `strncpy'.

This removes a few "buf[buflen - 1] = '\0';".

src/common.[ch]: Changed the signature of `sstrncpy' to match that of `strncpy'.

Merge branch 'collectd-4.2' into collectd-4.3

Conflicts:

ChangeLog
src/network.c
version-gen.sh

Bumped version to 4.2.5; Updated ChangeLog.

configure.in: Fix a typo.

network plugin: Tested and fixed the receiving part on Sparc.

network plugin: Use `memcpy' when parsing packages, too.

This should prevent crashes due to unaligned memory access when running as
server.

ChangeLog: Wrote an entry for 4.2.5.

Merge branch 'collectd-4.2' into collectd-4.3

src/common.c: Use uint8_t's instead of (signed) char's.

src/common.[ch]: Implemented `htond' and `ntohd' and applied it to the network plugin.

The functions convert doubles to the x86 representation or from the x86
representation to the representation used on the host. On x86 systems, this is
a NOP.

configure.in: Added checks to find out how doubles are stored.

Merge branch 'collectd-4.2' into collectd-4.3

src/utils_cache.c: Fix SPARC alignment problems.

network plugin: Align write access to the send buffer.

SPARC and possibly other architectures cannot access arbitrary memory
locations. This caused a `bus error' on SPARC when the network plugin was
loaded. This change hopefully fixes this problem.

src/Makefile.am: Added dependencies on `utils_cmd_putnotif.[ch]' to the exec plugin.

Merge branch 'collectd-4.2' into collectd-4.3

tcpconns plugin: Fix for compilation under FreeBSD.

rrdtool plugin: Simplified heartbeat calculation

Looks simpler, should do the same.

Signed-off-by: Florian Forster <octo@huhu.verplant.org>

hddtemp plugin: Use SCSI_DISK8_MAJOR thru SCSI_DISK15_MAJOR only if available.

Some Linux versions don't seem to provide the major numbers 8 thru 15.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'collectd-4.2' into collectd-4.3

ping plugin: Fixed a typo when setting the TTL.

PING_DEF_TIMEOUT used to be used instead of PING_OPT_TTL as option
argument of ping_setopt(). This caused really strange effects when pinging
hosts, like a failed assertion in ping_timeval_sub() in liboping
indicating a latency <= 0.

Thanks to Romain Francoise for pointing this out and a lot of helpful
debugging information.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

exec plugin: Don't pass an invalid egid to setgroups() if no group was given.

In case that no group has been specified in the config file, egid = -1 used to
be passed to setgroups() which, obviously, is not what we want.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

perl plugin: Warn if "EnableDebugger" has been used after "LoadPlugin".

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'collectd-4.2' into collectd-4.3

Merge branch 'collectd-4.1' into collectd-4.2

ping plugin: Deactivate the plugin if (pingobj == NULL).

If (pingobj == NULL), then the init-function will return (-1) so that the
plugin is deactivated.

src/collectd.c: Fix the arguments passed to an error message.

Unfortunately the compiler doesn't check va-args, so this was unnoticed :/

Merge branch 'collectd-4.2' of octo@verplant.org:/var/lib/git/collectd into collectd-4.2

swap plugin: Reapply a patch for Solaris.

This is a previously applied change that somehow got lost.

See http://git.verplant.org/?p=collectd.git;a=commitdiff;h=ee1113650b0b565776fc3d62162d84741e234f05
and http://git.verplant.org/?p=collectd.git;a=commitdiff;h=f68b68c5825efbd23c51531ff4e43c78a3bb2b3b

contrib/extractDS.px: Enhanced the script to modify RRD files a bit.

It can now also change the stepsize and add an RRA.

ChangeLog: Removed the legacy message about Nagios integration in the exec plugin.

README: Moved "libxml2" from the "Features" section to "Prerequisites".

D'oh!

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'pull/collectd-4.1' into collectd-4.1

src/collectd.c: Write a more informative error message if looking up the FQDN fails.

As requested by Micha Krause.

exec plugin: Use `setgroups' to set the list of supplementary group IDs.

Thanks to Peter Holik for suggesting this.

src/configfile.c: Fix another access of free'd memory.

This led to a crash with the default config of the Debian package.

liboconfig: Make sure that option-strings can always be freed.

This resolved a nasty crash when including files.

Bumped version to 4.3.0; Updated ChangeLog.

exec plugin: Removed the Nagios logic from the plugin.

Instead point users to the `exec-nagios.px' script in collectd-exec(5).

migrate-3-4.px: Do not translate the disk names used by hddtemp.

In version 4, hddtemp still uses major/minor device numbers to identify disks.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

migrate-3-4.px: Do not translate the disk names used by hddtemp.

In version 4, hddtemp still uses major/minor device numbers to identify disks.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Merge branch 'collectd-4.2'

Merge branch 'collectd-4.1' into collectd-4.2

Merge branch 'sh/hddtemp'

hddtemp plugin, collectd.conf(5): Added "TranslateDevicename" config option.

If enabled, translate the disk names to major/minor device numbers. For
backwards compatibility this defaults to true but it it recommended to disable
it as it will probably be removed in the next major version. Any other parts
of collectd have changed in a similar way with version 4.0, so hddtemp should
do so as well.

This option is disabled in new versions of the config file.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

rrdtool plugin: Sort the list of user defined `RRATimespan's.

This is required by the logic calculating the RRA sized.

Merge branch 'master' of octo@verplant.org:/var/lib/git/collectd

perl plugin: Do not abort while preparing the stack for a subroutine call.

Else the stack would probably be left in an inconsistent case and might even
leak memory.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

collectd.conf(5): Added a note that libvirt's RefreshInterval may be disabled.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

README: Added libxml2 to the list of prerequisites.

The libvirt plugin uses this library to parse data provided by libvirt.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

README: Fixed a typo.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Updated collectd.conf.in.

 * Missing plugins and config options have been added.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

contrib/README: Removed the section about PerlLib/.

This subdirectory no longer exists - it has been moved to bindings/perl/.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Makefile.am: Create `$(localstatedir)/log' so the logfile plugin works out-of-the-box.

And don't search for `.svn' directories anymore - they're long gone.

exec plugin: Changed the format in which notifications are passed to the programs.

Using a `Message:' header field restricts us to one-line messages, which is
probably not very farsighted.

exec plugin: Disabled the `NagiosExec' option.

All references to this option have been removed from the manpages, too.

collectd.conf(5), ChangeLog: Documented the new `Include' features.

src/configfile.c: Use wordexp(3) to expand shell wildcards if available.

contrib/migrate-3-4.px: Handle `hddtemp' graphs correctly.

contrib/exec-nagios.px: Added a Perl script which handles Nagios plugins.

It handles most of the Nagios plugin API, including multi-line output (a
Nagios 3 feature) and performance data. It's basic on purpose so that easy
stuff is possible but complex stuff is about as hard as rewriting the plugin ;)

contrib/exec-munin.px: Fix a typo in the config handling functions.

src/utils_ignorelist.c: Remove an annoying debug message.

configure: Fix a syntax error.

Merge branch 'master' of octo@verplant.org:/var/lib/git/collectd

configure: Correctly handle all cases when using AC_ARG_WITH().

In a lot of cases only "$withval" != "yes" and "no" had been handled when the
option had been given. Therefore, the option had been ignored if no argument
had been passed to the option, making --with-<option> and / or
--without-<option> a no-op. This patch fixes this issue.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

contrib/examples/: Added an example for a notification callback.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

collectd(1), README: Document notifications and related plugins.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

collectdmon(1): Added a section about supported signals.

Documented the effects of the SIGINT, SIGTERM and SIGHUP signals.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

README: Added "uuid" plugin to the list of available plugins.

 * Added an optional dependency on libhal.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

README: Updated package description.

Added a note about the monitoring support (this has been copied from the
Debian package description).

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

src/configfile.c: Extend the `Include' statement to handle directories, too.

Directories are included recursively. Files and directories begnning with a dot
are ignored. Symlinks are followed. You can crash collectd by looping the
symlinks.

configure: Warn if librrd is not thread safe.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

plugin.c: Use an AVL tree to store the data sets.

The list of data sets is queried quite frequently but hardly ever modified.
Using an AVL tree instead of a linked list improves the search time from O(n)
to O(log n).

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

exec plugin: Fixed a possible (though very improbable) memory leak.

In case of an error, the program_list_and_notification_t pointer has not been
freed before calling pthread_exit() from exec_notification_one().

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

Collectd.pm: Use $interval_g instead of a hard-coded value.

The hard-coded value was still legacy code from the time before the global
variables have been exported to Perl.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

perl.c: Unregister notifications in perl_shutdown().

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>

perl plugin: Fixed a bug in Collectd::plugin_unregister_data_set().

The function tried to access a nonexistent second argument.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>