From e776936ab69d309084e464e0c8dfccb822ba500a Mon Sep 17 00:00:00 2001 From: Tuukka Tikkanen Date: Tue, 13 Jan 2015 19:51:43 +0200 Subject: Energy model: Check for error from idlestat_load correctly Idlestat_load() returns ptr_error() rather than a NULL pointer in case of failure. The function make_energy_model_template checked for NULL instead of using the is_err() macro. This patch corrects the behavior. Signed-off-by: Tuukka Tikkanen --- energy_model.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/energy_model.c b/energy_model.c index 23463df..7a0ba8a 100644 --- a/energy_model.c +++ b/energy_model.c @@ -60,7 +60,7 @@ static int make_energy_model_template(struct program_options *options) fprintf(f, "# Replace ? with correct values\n"); datas = idlestat_load(options->filename); - if (!datas) { + if (is_err(datas)) { fclose(f); unlink(options->energy_model_filename); return -1; -- cgit v1.2.3