diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-06-14 10:02:25 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-06-14 10:02:25 -0400 |
| commit | dd52a184937d4ffa38d07615f893b06f75dd332e (patch) | |
| tree | 4f1e109ccbba3dc0eac8ce6e2152200d78d6f952 /tools/start-stop-daemon.c | |
| parent | d17840aa8a5d3c96f828caae46f11782c028f750 (diff) | |
| parent | 657f9948ed593387d29d391832c18dc04326a5a9 (diff) | |
Merge pull request #2436 from pacovn/Coverity_1143220_Argument_cannot_be_negative
tools: return check (Coverity 1143220)
Diffstat (limited to 'tools/start-stop-daemon.c')
| -rw-r--r-- | tools/start-stop-daemon.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tools/start-stop-daemon.c b/tools/start-stop-daemon.c index 5f13b90be3..8c3fe0c3c5 100644 --- a/tools/start-stop-daemon.c +++ b/tools/start-stop-daemon.c @@ -1030,8 +1030,10 @@ int main(int argc, char **argv) umask(022); /* set a default for dumb programs */ setpgid(0, 0); /* set the process group */ fd = open("/dev/null", O_RDWR); /* stdin */ - dup(fd); /* stdout */ - dup(fd); /* stderr */ + if (fd >= 0) { + dup(fd); /* stdout */ + dup(fd); /* stderr */ + } } if (nicelevel) { errno = 0; |
