From 280afcff8065a8421f0c8246be49d3905da25624 Mon Sep 17 00:00:00 2001 From: jules Date: Mon, 8 Apr 2013 15:48:39 +0100 Subject: [PATCH] Warning fix for linux. --- .../juce_core/native/juce_linux_Threads.cpp | 21 +++++-------------- 1 file changed, 5 insertions(+), 16 deletions(-) diff --git a/modules/juce_core/native/juce_linux_Threads.cpp b/modules/juce_core/native/juce_linux_Threads.cpp index 6e999b278f..710c2f5a48 100644 --- a/modules/juce_core/native/juce_linux_Threads.cpp +++ b/modules/juce_core/native/juce_linux_Threads.cpp @@ -77,22 +77,11 @@ JUCE_API bool JUCE_CALLTYPE Process::isRunningUnderDebugger() return juce_isRunningUnderDebugger(); } -void Process::raisePrivilege() +static void swapUserAndEffectiveUser() { - // If running suid root, change effective user to root - if (geteuid() != 0 && getuid() == 0) - { - setreuid (geteuid(), getuid()); - setregid (getegid(), getgid()); - } + (void) setreuid (geteuid(), getuid()); + (void) setregid (getegid(), getgid()); } -void Process::lowerPrivilege() -{ - // If runing suid root, change effective user back to real user - if (geteuid() == 0 && getuid() != 0) - { - setreuid (geteuid(), getuid()); - setregid (getegid(), getgid()); - } -} +void Process::raisePrivilege() { if (geteuid() != 0 && getuid() == 0) swapUserAndEffectiveUser(); } +void Process::lowerPrivilege() { if (geteuid() == 0 && getuid() != 0) swapUserAndEffectiveUser(); }