deepblue/ipmitool
1
0
Fork 0
mirror of https://github.com/ipmitool/ipmitool.git synced 2025-05-10 18:47:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[compiler-warnings-fixes] ipmi_start_daemon: check return values

Browse Source 
Some return values were being ignored in ipmi_start_daemon. This adds
checks to the values and changes some hard-coded numbers into named
values.

Partially resolves ipmitool/ipmitool#13

Signed-off-by: Vernon Mauery <vernon.mauery@intel.com>
This commit is contained in:
Vernon Mauery 2018-10-31 13:38:03 -07:00 committed by Alexander Amelkin
parent dc9b4ebfc9
commit f2780c5d86
1 changed files with 20 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
lib/helper.c
View File

@ -823,6 +823,7 @@ ipmi_start_daemon(struct ipmi_intf *intf)
{
pid_t pid;
int fd;
int ret;
#ifdef SIGHUP
sigset_t sighup;
#endif
@ -866,7 +867,11 @@ ipmi_start_daemon(struct ipmi_intf *intf)
exit(0);
#endif
chdir("/");
ret = chdir("/");
if (ret) {
lprintf(LOG_ERR, "chdir failed: %s (%d)", strerror(errno), errno);
exit(1);
}
umask(0);
for (fd=0; fd<64; fd++) {
@ -875,9 +880,20 @@ ipmi_start_daemon(struct ipmi_intf *intf)
}
fd = open("/dev/null", O_RDWR);
assert(0 == fd);
dup(fd);
dup(fd);
if (fd != STDIN_FILENO) {
lprintf(LOG_ERR, "failed to reset stdin: %s (%d)", strerror(errno), errno);
exit(1);
}
ret = dup(fd);
if (ret != STDOUT_FILENO) {
lprintf(LOG_ERR, "failed to reset stdout: %s (%d)", strerror(errno), errno);
exit(1);
}
ret = dup(fd);
if (ret != STDOUT_FILENO) {
lprintf(LOG_ERR, "failed to reset stderr: %s (%d)", strerror(errno), errno);
exit(1);
}
}
/* eval_ccode - evaluate return value of _ipmi_* functions and print error error