reopen 468143 found 468143 4.3.2-1 tags 468143 + patch pending fixed-upstream thanks
Hi, On Fri, Apr 18, 2008 at 08:22:07AM +0200, Florian Forster wrote: > while writing a reply and interpreting the log output you've provided, I > stumbled across the bug. The function which is responsible for > formatting the chip name does so depending on the type of the chip. The > returned status variable is only set in one out of three branches, > though :/ A schoolbook example of ``harmful initialization''.. D'oh :-/ Thanks for the patch, Florian! > Please give the attached patch a try and tell me if it worked or not. > I'm sure Sebastian or some howto can tell you how to do integrate that > into the package building process, if necessary. I've prepared a package of a 4.3.2 + that patch snapshot. You can find it at [1] - for instructions how to build the package see one of my follow-up e-mails to this bug report ([2]). Feedback would be very appreciated. I've reopened the bug for now. After some positive feedback, I will include Florian's patch in the next upload. An updated backports.org package will be available as soon as that new version migrates to testing. TIA, Sebastian [1] http://debian.tokkee.org/tmp/collectd/collectd_4.3.2.1.g33d82fc-1.dsc [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=468143#32 -- Sebastian "tokkee" Harl +++ GnuPG-ID: 0x8501C7FC +++ http://tokkee.org/ Those who would give up Essential Liberty to purchase a little Temporary Safety, deserve neither Liberty nor Safety. -- Benjamin Franklin
signature.asc
Description: Digital signature
