[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