Revert that last stupid setuid detection fix.
authorWerner Koch <wk@gnupg.org>
Thu, 17 Jul 2008 19:49:51 +0000 (19:49 +0000)
committerWerner Koch <wk@gnupg.org>
Thu, 17 Jul 2008 19:49:51 +0000 (19:49 +0000)
common/ChangeLog
common/exechelp.c
g10/ChangeLog
g10/exec.c
g10/gpg.c

index 6fdf436..dcee95b 100644 (file)
@@ -1,8 +1,3 @@
-2008-07-17  Werner Koch  <wk@g10code.com>
-
-       * exechelp.c (gnupg_spawn_process_detached): Do not run the setuid
-       test for root.
-
 2008-06-26  Werner Koch  <wk@g10code.com>
 
        * estream.c (es_write_sanitized): Loose check for control
index 90071e7..dcbbe90 100644 (file)
@@ -826,7 +826,7 @@ gnupg_spawn_process_detached (const char *pgmname, const char *argv[],
   pid_t pid;
   int i;
 
-  if (getuid() && getuid() != geteuid())
+  if (getuid() != geteuid())
     return gpg_error (GPG_ERR_BUG);
 
   if (access (pgmname, X_OK))
index 342c012..4aa1ed1 100644 (file)
@@ -1,8 +1,3 @@
-2008-07-17  Werner Koch  <wk@g10code.com>
-
-       * gpg.c (main): Do not run the setuid test for root.
-       * exec.c (exec_write): Ditto.
-
 2008-06-25  Marcus Brinkmann  <marcus@g10code.de>
 
        * gpg.c (enum cmd_and_opt_values): Remove option
index 40561e8..9ecd2ab 100644 (file)
@@ -318,8 +318,8 @@ int exec_write(struct exec_info **info,const char *program,
 
 #if defined(HAVE_GETUID) && defined(HAVE_GETEUID)
   /* There should be no way to get to this spot while still carrying
-     setuid privs.  Just in case, bomb out if we are (and are not root). */
-  if (getuid () && getuid () != geteuid ())
+     setuid privs.  Just in case, bomb out if we are. */
+  if ( getuid () != geteuid ())
     BUG ();
 #endif
 
index c226b1e..0601f50 100644 (file)
--- a/g10/gpg.c
+++ b/g10/gpg.c
@@ -2015,8 +2015,8 @@ main (int argc, char **argv)
       got_secmem = 1; 
 #if defined(HAVE_GETUID) && defined(HAVE_GETEUID)
     /* There should be no way to get to this spot while still carrying
-       setuid privs.  Just in case, bomb out if we are (and are not root). */
-    if (getuid () && getuid () != geteuid ())
+       setuid privs.  Just in case, bomb out if we are. */
+    if ( getuid () != geteuid () )
       BUG ();
 #endif
     maybe_setuid = 0;