From d7cb6fe4670f85cd52dd7150cf856f368564eea8 Mon Sep 17 00:00:00 2001 From: Stefan Schmidt Date: Tue, 12 Jun 2012 10:01:56 +0000 Subject: [PATCH] e/modules/cpufreq: Check the return of fgets to make sure we operate on something sane If we get NULL as reply here the buf will not contain the stuff we are looking for. SVN revision: 72005 --- src/modules/cpufreq/e_mod_main.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/modules/cpufreq/e_mod_main.c b/src/modules/cpufreq/e_mod_main.c index 4c33fac53..3f9031046 100644 --- a/src/modules/cpufreq/e_mod_main.c +++ b/src/modules/cpufreq/e_mod_main.c @@ -604,7 +604,7 @@ _cpufreq_status_check_available(Status *s) s->frequencies = NULL; } - fgets(buf, sizeof(buf), f); + if (fgets(buf, sizeof(buf), f) == NULL) return 1; buf[sizeof(buf) - 1] = 0; fclose(f); @@ -638,7 +638,7 @@ _cpufreq_status_check_available(Status *s) s->governors = NULL; } - fgets(buf, sizeof(buf), f); + if (fgets(buf, sizeof(buf), f) == NULL) return 1; buf[sizeof(buf) - 1] = 0; fclose(f); @@ -698,7 +698,7 @@ _cpufreq_status_check_current(Status *s) f = fopen(buf, "r"); if (f) { - fgets(buf, sizeof(buf), f); + if (fgets(buf, sizeof(buf), f) == NULL) continue; buf[sizeof(buf) - 1] = 0; fclose(f); @@ -740,7 +740,7 @@ _cpufreq_status_check_current(Status *s) char *p; buf[0] = 0; - fgets(buf, sizeof(buf), f); + if (fgets(buf, sizeof(buf), f) == NULL) return ret;; buf[sizeof(buf) - 1] = 0; fclose(f); for (p = buf; (*p != 0) && (isalnum(*p)); p++);