diff options
| author | Alexandra Yates <alexandra.yates@linux.intel.com> | 2014-06-12 23:18:59 -0700 |
|---|---|---|
| committer | Alexandra Yates <alexandra.yates@linux.intel.com> | 2014-06-18 14:16:26 -0700 |
| commit | fd7bb179c0559ba7bb2a3445423afd910bf6626c (patch) | |
| tree | 7ff65711397a0bfb8c29a08343667ed79a227e67 | |
| parent | 1d044cc75bc6600eefdb39e468eb052e81c5f3c8 (diff) | |
| download | powertop-2.0-fd7bb179c0559ba7bb2a3445423afd910bf6626c.tar.gz | |
fix-warnings: Ignoring return value
Added error handling to system() and write() calls.
Tested by: Christophe Prigent
Signed-off-by: Alexandra Yates <alexandra.yates@linux.intel.com>
| -rw-r--r-- | src/calibrate/calibrate.cpp | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/src/calibrate/calibrate.cpp b/src/calibrate/calibrate.cpp index db368e0..a9a0a7c 100644 --- a/src/calibrate/calibrate.cpp +++ b/src/calibrate/calibrate.cpp @@ -33,6 +33,7 @@ #include <pthread.h> #include <math.h> #include <sys/types.h> +#include <errno.h> #include "../parameters/parameters.h" extern "C" { @@ -239,7 +240,8 @@ static void *burn_disk(void *dummy) while (!stop_measurement) { lseek(fd, 0, SEEK_SET); - write(fd, buffer, 64*1024); + if(write(fd, buffer, 64*1024) == -1) + printf("Error: %s\n", strerror(errno)); fdatasync(fd); } close(fd); @@ -350,17 +352,21 @@ static void backlight_calibration(void) sleep(1); } printf(_("Calibrating idle\n")); - system("DISPLAY=:0 /usr/bin/xset dpms force off"); + if(!system("DISPLAY=:0 /usr/bin/xset dpms force off")) + printf("System is not available\n"); one_measurement(15, NULL); - system("DISPLAY=:0 /usr/bin/xset dpms force on"); + if(!system("DISPLAY=:0 /usr/bin/xset dpms force on")) + printf("System is not available\n"); } static void idle_calibration(void) { printf(_("Calibrating idle\n")); - system("DISPLAY=:0 /usr/bin/xset dpms force off"); + if(!system("DISPLAY=:0 /usr/bin/xset dpms force off")) + printf("System is not available\n"); one_measurement(15, NULL); - system("DISPLAY=:0 /usr/bin/xset dpms force on"); + if(!system("DISPLAY=:0 /usr/bin/xset dpms force on")) + printf("System is not available\n"); } |