sensord now installed, but --rrd-file argument is fatal

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Apr 03, 2004 at 09:07:38PM +0200, Jean Delvare wrote:
> > I investigated this and sensord seems to not like using -a and -r
> > together (logging average load and logging to rrd). Try removing the
> > -a from the init.d script.
> > 
> > Is this a known problem? Any cure? :)
> 
> We had a report about segfault in sensord (2.8.5) some times ago:
> http://archives.andrew.net.au/lm-sensors/msg06918.html
> 
> This was triggered by --load-average, so I wouldn't be surprised that
> this is the same problem here.
> 
> A fix has been commited to CVS (dated 8 Mar 2004) and was reported to
> work. I insert it here so that you can give it a try:
> 
> Index: rrd.c
> ===================================================================
> RCS file: /home/cvs/lm_sensors2/prog/sensord/rrd.c,v
> retrieving revision 1.7
> retrieving revision 1.8
> diff -u -r1.7 -r1.8
> +++ rrd.c	8 Mar 2004 18:03:42 -0000	1.8
> @@ -195,7 +195,7 @@
>      char *ptr = rrdBuff + data->num * RRD_BUFF;
>      const char *min, *max;
>      data->argv[data->num ++] = ptr;
> -    switch (feature->type) { /* arbitrary sanity limits */
> +    switch (feature ? feature->type : DataType_other) { /* arbitrary sanity limits */
>        case DataType_voltage:
>          min="-25";
>          max="25";
> 
> 
> Let us know if it doesn't fix it, and we'll investigate.

Thanks for the patch! If 2.8.6 is to be released and contains that
patch I better wait for it, or not?
-- 
Axel.Thimm at ATrpms.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20040404/143a2eb6/attachment.bin 


[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux