[lm-sensors] [PATCH 1/3] sensord: Fix memory leaks revealed by valgrind
Guenter Roeck
guenter.roeck at ericsson.com
Sun Mar 4 07:42:19 CET 2012
On Sat, Mar 03, 2012 at 04:25:36PM -0500, Jean Delvare wrote:
> The leak in daemonize() is harmless, we're about to exit anyway. Fix
> it still to make valgrind happy.
>
> The leak in do_features() is real, as the function is called
> periodically by the daemon, for all actions. If the intervals at set
> low and the system has many sensors, the leak could be significant,
> maybe 150 kB/day.
> ---
> prog/sensord/sense.c | 22 +++++++++++++---------
> prog/sensord/sensord.c | 6 +++++-
> 2 files changed, 18 insertions(+), 10 deletions(-)
>
Nice catch series looks good.
Guenter
More information about the lm-sensors
mailing list