* hlfsd/homedir.c (homedir): use setgid() instead of setegid()

diff --git a/ChangeLog b/ChangeLog
index 5202ca5d5e36a12630cfc04cf8e1a1ac1ff292c8..4d46f6117443b226e41b9458c901dae51a0ed783 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,7 @@
 2002-06-22  Ion Badulescu  <lionut@gonzales.badula.org>
 
+       * hlfsd/homedir.c (homedir): use setgid() instead of setegid()
+
        * buildall: use ksh for configure on aix5.1
 
 2002-06-21  Ion Badulescu  <ion@guppy.limebrokerage.com>

diff --git a/hlfsd/homedir.c b/hlfsd/homedir.c
index b520f85b824a4442ca920cf993031c2a073df261..62450a51e36390f6c5007e166f7550a9948ade4c 100644 (file)
--- a/hlfsd/homedir.c
+++ b/hlfsd/homedir.c
@@ -38,7 +38,7 @@
  *
  *      %W% (Berkeley) %G%
  *
- * $Id: homedir.c,v 1.5.2.8 2002/05/03 21:23:40 ib42 Exp $
+ * $Id: homedir.c,v 1.5.2.9 2002/06/23 01:39:04 ib42 Exp $
  *
  * HLFSD was written at Columbia University Computer Science Department, by
  * Erez Zadok <ezk@cs.columbia.edu> and Alexander Dupuy <dupuy@cs.columbia.edu>
@@ -203,13 +203,13 @@ homedir(int userid, int groupid)
    *
    */
   am_set_mypid();              /* for logging routines */
-  if ((old_groupid = setegid(groupid)) < 0) {
-    plog(XLOG_WARNING, "could not setegid to %d: %m", groupid);
+  if ((old_groupid = setgid(groupid)) < 0) {
+    plog(XLOG_WARNING, "could not setgid to %d: %m", groupid);
     return linkval;
   }
   if ((old_userid = seteuid(userid)) < 0) {
     plog(XLOG_WARNING, "could not seteuid to %d: %m", userid);
-    setegid(old_groupid);
+    setgid(old_groupid);
     return linkval;
   }
   if (hlfsd_stat(linkval, &homestat) < 0) {
@@ -217,14 +217,14 @@ homedir(int userid, int groupid)
       /* don't use recursive mkdirs here */
       if (mkdir(linkval, PERS_SPOOLMODE) < 0) {
        seteuid(old_userid);
-       setegid(old_groupid);
+       setgid(old_groupid);
        plog(XLOG_WARNING, "can't make directory %s: %m", linkval);
        return alt_spooldir;
       }
       /* fall through to testing the disk space / quota */
     } else {                   /* the home dir itself must not exist then */
       seteuid(old_userid);
-      setegid(old_groupid);
+      setgid(old_groupid);
       plog(XLOG_WARNING, "bad link to %s: %m", linkval);
       return alt_spooldir;
     }
@@ -240,12 +240,12 @@ homedir(int userid, int groupid)
    */
   if (hlfsd_diskspace(linkval) < 0) {
     seteuid(old_userid);
-    setegid(old_groupid);
+    setgid(old_groupid);
     plog(XLOG_WARNING, "no more space in %s: %m", linkval);
     return alt_spooldir;
   } else {
     seteuid(old_userid);
-    setegid(old_groupid);
+    setgid(old_groupid);
     return linkval;
   }
 }