Builder collectd-60-solaris10-i386 Build #39

Results:

Build successful

SourceStamp:

BuildSlave:

Reason:

The AnyBranchScheduler scheduler named 'schedule-collectd-60' triggered this build

Steps and Logfiles:

1. git update ( 6 secs )
   1. stdio
2. setproperty property 'ciflags' set ( 0 secs )
   1. stdio
   2. property changes
3. shell '/opt/csw/bin/bash ./build.sh' ( 27 secs )
   1. stdio
4. shell_1 './configure --prefix=/opt/csw ...' ( 31 secs )
   1. stdio
   2. config.log
5. shell_2 'gmake -k ...' ( 1 mins, 40 secs )
   1. stdio
6. shell_3 'gmake check' ( 13 secs )
   1. stdio
   2. test-suite.log

Build Properties:

Forced Build Properties:

Responsible Users:

1. Leonard Göhrs

   l.goehrsohnoyoudont@pengutronix.de

Timing:

All Changes:

:

1. Change #170798

   Category	None
   Changed by	Leonard Göhrs <l.goehrsohnoyoudont@pengutronix.de>
   Changed at	Fri 24 Mar 2023 15:28:55
   Repository	https://github.com/collectd/collectd
   Project	collectd/collectd
   Branch	collectd-6.0
   Revision	df772cc24494a6328a1c359effa5003b74ae4521

   Comments

   src/daemon/plugin.c: don't store references to stack allocated values
   
   The changes in commit
   
     55efb56a ([collectd 6] src/daemon/plugin.c: Use one thread per write plugin)
   
   wrongly assume that the references passed in to plugin_register_write()
   somehow outlive the spawned write thread.
   While this is true for some plugins that pass staticly allocated strings
   and global user_data_t structs to plugin_register_write() it is not correct
   for all plugins.
   
   See [1] for an example of a plugin (write_http) mis-behaving due to this.
   
   Store owned versions of the passed values instead. For user_data this means
   the content of the struct and for the name it means a strdup()ed version of
   the string.
   
   [1]: https://github.com/collectd/collectd/issues/4098
   
   Signed-off-by: Leonard Göhrs <l.goehrs@pengutronix.de>

   Changed files

   • src/daemon/plugin.c