Bug#491480: [Pkg-xfce-devel] Bug#491480: I think i fixed the bug

2008-08-06 Thread Yves-Alexis Perez
On Wed, Aug 06, 2008 at 02:40:23PM +0200, Daniel Gibson wrote: > On Wed, Aug 6, 2008 at 7:59 AM, Yves-Alexis Perez <[EMAIL PROTECTED]> wrote: > > Hi, > I do have a battery, but no /proc/acpi/battery/ dir - in kernel > 2.4.25. as far as I remember from fixing the battery-plugin the > battery-stuff

Bug#491480: [Pkg-xfce-devel] Bug#491480: I think i fixed the bug

2008-08-06 Thread Daniel Gibson
On Wed, Aug 6, 2008 at 7:59 AM, Yves-Alexis Perez <[EMAIL PROTECTED]> wrote: > > Ok I guess that's why I don't have crashes on this box: I don't have > anything in /proc/acpi/fan so the STATE part is never entered. > > I'll try to build the plugin with the patch and report back, stay tuned. > > btw

Bug#491480: I think i fixed the bug

2008-08-06 Thread Daniel Gibson
Good new everyone, I found the bug (or at least the one that made the plugin crash for _me_). It's in panel-plugin/acpi.c in refresh_acpi(). The (broken) code is at line 475: case STATE: file = g_strdup_printf ("%s/%s/state", ACPI_DIR_FAN, cf->devicename); cf->raw_value = strcmp(g

Bug#491480: [Pkg-xfce-devel] Bug#491480: I think i fixed the bug

2008-08-06 Thread Yves-Alexis Perez
On mer, 2008-08-06 at 03:20 +0200, Daniel Gibson wrote: > Good new everyone, > > I found the bug (or at least the one that made the plugin crash for > _me_). > It's in panel-plugin/acpi.c in refresh_acpi(). > The (broken) code is at line 475: > > case STATE: > file = g_strdup_printf ("%s/