[Goodies-dev] [Bug 4265] sensors plugin crash when trying to refresh ACPI FAN state

[bugzilla-daemon at xfce.org]

DO NOT REPLY TO THIS EMAIL.  Also, do not reply via email to the person
whose email is mentioned below.  To comment on this bug, please visit:
http://bugzilla.xfce.org/show_bug.cgi?id=4265





------- Comment #3 from corsac at debian.org  2008-08-07 20:52 UTC -------
(In reply to comment #2)
> (In reply to comment #0)
> 
> Thanks, but the patch isn't perfect as it does not free() properly. See changes
> in repository. I can't test the free()ing of char *state as I don't have fan
> state with Linux 2.6.22, 2.6.24 and 2.6.25.

I'll try the repository and report back.
> 
> There are no plans for /sys/. I guess I'll skip this step and wait until
> there's a convenient proc-like file system again. Who cares about deprecation
> if there's something good, reliable and convenient? I don't.

A convenient proc-like filesystem? There won't be. Switch to sysfs, that's
you're “something good, reliable and convenient”. I don't think you have
another choice. Better switch before removal :)

Cheers,


-- 
Configure bugmail: http://bugzilla.xfce.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.