From fa0c637c557108dffe1d4eff30d14d87b6ca26de Mon Sep 17 00:00:00 2001 From: Hisham Date: Wed, 25 May 2016 21:37:07 -0300 Subject: Silence warnings about seteuid return value. Closes #483. --- EnvScreen.c | 4 ++-- Process.c | 8 ++++---- Settings.c | 12 ++++++------ TraceScreen.c | 2 +- 4 files changed, 13 insertions(+), 13 deletions(-) diff --git a/EnvScreen.c b/EnvScreen.c index 4414c390..06f329c1 100644 --- a/EnvScreen.c +++ b/EnvScreen.c @@ -49,9 +49,9 @@ void EnvScreen_scan(InfoScreen* this) { Panel_prune(panel); uid_t euid = geteuid(); - seteuid(getuid()); + (void) seteuid(getuid()); char *env = Platform_getProcessEnv(this->process->pid); - seteuid(euid); + (void) seteuid(euid); if (env) { for (char *p = env; *p; p = strrchr(p, 0)+1) InfoScreen_addLine(this, p); diff --git a/Process.c b/Process.c index c6275de4..84815157 100644 --- a/Process.c +++ b/Process.c @@ -518,10 +518,10 @@ void Process_toggleTag(Process* this) { bool Process_setPriority(Process* this, int priority) { uid_t euid = geteuid(); - seteuid(getuid()); + (void) seteuid(getuid()); int old_prio = getpriority(PRIO_PROCESS, this->pid); int err = setpriority(PRIO_PROCESS, this->pid, priority); - seteuid(euid); + (void) seteuid(euid); if (err == 0 && old_prio != getpriority(PRIO_PROCESS, this->pid)) { this->nice = priority; } @@ -534,9 +534,9 @@ bool Process_changePriorityBy(Process* this, size_t delta) { void Process_sendSignal(Process* this, size_t sgn) { uid_t euid = geteuid(); - seteuid(getuid()); + (void) seteuid(getuid()); kill(this->pid, (int) sgn); - seteuid(euid); + (void) seteuid(euid); } long Process_pidCompare(const void* v1, const void* v2) { diff --git a/Settings.c b/Settings.c index a6bb355d..2975d62b 100644 --- a/Settings.c +++ b/Settings.c @@ -167,9 +167,9 @@ static bool Settings_read(Settings* this, const char* fileName) { FILE* fd; uid_t euid = geteuid(); - seteuid(getuid()); + (void) seteuid(getuid()); fd = fopen(fileName, "r"); - seteuid(euid); + (void) seteuid(euid); if (!fd) return false; @@ -277,9 +277,9 @@ bool Settings_write(Settings* this) { FILE* fd; uid_t euid = geteuid(); - seteuid(getuid()); + (void) seteuid(getuid()); fd = fopen(this->filename, "w"); - seteuid(euid); + (void) seteuid(euid); if (fd == NULL) { return false; } @@ -366,7 +366,7 @@ Settings* Settings_new(int cpuCount) { } legacyDotfile = String_cat(home, "/.htoprc"); uid_t euid = geteuid(); - seteuid(getuid()); + (void) seteuid(getuid()); (void) mkdir(configDir, 0700); (void) mkdir(htopDir, 0700); free(htopDir); @@ -379,7 +379,7 @@ Settings* Settings_new(int cpuCount) { free(legacyDotfile); legacyDotfile = NULL; } - seteuid(euid); + (void) seteuid(euid); } this->colorScheme = 0; this->changed = false; diff --git a/TraceScreen.c b/TraceScreen.c index 5ae0600c..2d6eb336 100644 --- a/TraceScreen.c +++ b/TraceScreen.c @@ -95,7 +95,7 @@ bool TraceScreen_forkTracer(TraceScreen* this) { this->child = fork(); if (this->child == -1) return false; if (this->child == 0) { - seteuid(getuid()); + (void) seteuid(getuid()); dup2(this->fdpair[1], STDERR_FILENO); int ok = fcntl(this->fdpair[1], F_SETFL, O_NONBLOCK); if (ok != -1) { -- cgit v1.2.3