Hi Alex, 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''..
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. Regards, -octo -- Florian octo Forster Hacker in training GnuPG: 0x91523C3D http://verplant.org/
diff --git a/src/sensors.c b/src/sensors.c
index 2a0a9ea..173ed9c 100644
--- a/src/sensors.c
+++ b/src/sensors.c
@@ -189,14 +189,14 @@ static int sensors_snprintf_chip_name (char *buf, size_t buf_size,
}
else if (chip->bus == SENSORS_CHIP_NAME_BUS_DUMMY)
{
- snprintf (buf, buf_size, "%s-%s-%04x",
+ status = snprintf (buf, buf_size, "%s-%s-%04x",
chip->prefix,
chip->busname,
chip->addr);
}
else
{
- snprintf (buf, buf_size, "%s-i2c-%d-%02x",
+ status = snprintf (buf, buf_size, "%s-i2c-%d-%02x",
chip->prefix,
chip->bus,
chip->addr);
signature.asc
Description: Digital signature
