tools/power/x86/intel-speed-select: Handle open() failure case

Add handling for open() failure case to make sure a valid file
descriptor is passed to dup().

Signed-off-by: Zhang Rui <rui.zhang@intel.com>
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

diff --git a/tools/power/x86/intel-speed-select/isst-daemon.c b/tools/power/x86/intel-speed-select/isst-daemon.c
index 0699137c090147b01b5cdedbbdaaddde00f22bd0..4ad6a64f1545c4d82fe7dc9df8112a845d28fdf7 100644 (file)
--- a/tools/power/x86/intel-speed-select/isst-daemon.c
+++ b/tools/power/x86/intel-speed-select/isst-daemon.c
@@ -174,6 +174,8 @@ static void daemonize(char *rundir, char *pidfile)
                close(i);
 
        i = open("/dev/null", O_RDWR);
+       if (i < 0)
+               exit(EXIT_FAILURE);
        ret = dup(i);
        if (ret == -1)
                exit(EXIT_FAILURE);