[BACK]Return to uidswap.c CVS log [TXT][DIR] Up to [local] / src / usr.bin / ssh

Diff for /src/usr.bin/ssh/uidswap.c between version 1.41 and 1.42

version 1.41, 2018/07/18 11:34:04 version 1.42, 2019/06/28 13:35:04
Line 63 
Line 63 
         privileged = 1;          privileged = 1;
         temporarily_use_uid_effective = 1;          temporarily_use_uid_effective = 1;
         saved_egroupslen = getgroups(NGROUPS_MAX, saved_egroups);          saved_egroupslen = getgroups(NGROUPS_MAX, saved_egroups);
         if (saved_egroupslen < 0)          if (saved_egroupslen == -1)
                 fatal("getgroups: %.100s", strerror(errno));                  fatal("getgroups: %.100s", strerror(errno));
   
         /* set and save the user's groups */          /* set and save the user's groups */
         if (user_groupslen == -1 || user_groups_uid != pw->pw_uid) {          if (user_groupslen == -1 || user_groups_uid != pw->pw_uid) {
                 if (initgroups(pw->pw_name, pw->pw_gid) < 0)                  if (initgroups(pw->pw_name, pw->pw_gid) == -1)
                         fatal("initgroups: %s: %.100s", pw->pw_name,                          fatal("initgroups: %s: %.100s", pw->pw_name,
                             strerror(errno));                              strerror(errno));
                 user_groupslen = getgroups(NGROUPS_MAX, user_groups);                  user_groupslen = getgroups(NGROUPS_MAX, user_groups);
                 if (user_groupslen < 0)                  if (user_groupslen == -1)
                         fatal("getgroups: %.100s", strerror(errno));                          fatal("getgroups: %.100s", strerror(errno));
                 user_groups_uid = pw->pw_uid;                  user_groups_uid = pw->pw_uid;
         }          }
         /* Set the effective uid to the given (unprivileged) uid. */          /* Set the effective uid to the given (unprivileged) uid. */
         if (setgroups(user_groupslen, user_groups) < 0)          if (setgroups(user_groupslen, user_groups) == -1)
                 fatal("setgroups: %.100s", strerror(errno));                  fatal("setgroups: %.100s", strerror(errno));
         if (setegid(pw->pw_gid) < 0)          if (setegid(pw->pw_gid) == -1)
                 fatal("setegid %u: %.100s", (u_int)pw->pw_gid,                  fatal("setegid %u: %.100s", (u_int)pw->pw_gid,
                     strerror(errno));                      strerror(errno));
         if (seteuid(pw->pw_uid) == -1)          if (seteuid(pw->pw_uid) == -1)
Line 102 
Line 102 
                 fatal("restore_uid: temporarily_use_uid not effective");                  fatal("restore_uid: temporarily_use_uid not effective");
         debug("restore_uid: %u/%u", (u_int)saved_euid, (u_int)saved_egid);          debug("restore_uid: %u/%u", (u_int)saved_euid, (u_int)saved_egid);
         /* Set the effective uid back to the saved privileged uid. */          /* Set the effective uid back to the saved privileged uid. */
         if (seteuid(saved_euid) < 0)          if (seteuid(saved_euid) == -1)
                 fatal("seteuid %u: %.100s", (u_int)saved_euid, strerror(errno));                  fatal("seteuid %u: %.100s", (u_int)saved_euid, strerror(errno));
         if (setgroups(saved_egroupslen, saved_egroups) < 0)          if (setgroups(saved_egroupslen, saved_egroups) == -1)
                 fatal("setgroups: %.100s", strerror(errno));                  fatal("setgroups: %.100s", strerror(errno));
         if (setegid(saved_egid) < 0)          if (setegid(saved_egid) == -1)
                 fatal("setegid %u: %.100s", (u_int)saved_egid, strerror(errno));                  fatal("setegid %u: %.100s", (u_int)saved_egid, strerror(errno));
         temporarily_use_uid_effective = 0;          temporarily_use_uid_effective = 0;
 }  }
Line 122 
Line 122 
                 fatal("permanently_set_uid: temporarily_use_uid effective");                  fatal("permanently_set_uid: temporarily_use_uid effective");
         debug("permanently_set_uid: %u/%u", (u_int)pw->pw_uid,          debug("permanently_set_uid: %u/%u", (u_int)pw->pw_uid,
             (u_int)pw->pw_gid);              (u_int)pw->pw_gid);
         if (setresgid(pw->pw_gid, pw->pw_gid, pw->pw_gid) != 0)          if (setresgid(pw->pw_gid, pw->pw_gid, pw->pw_gid) == -1)
                 fatal("setresgid %u: %s", (u_int)pw->pw_gid, strerror(errno));                  fatal("setresgid %u: %s", (u_int)pw->pw_gid, strerror(errno));
         if (setresuid(pw->pw_uid, pw->pw_uid, pw->pw_uid) != 0)          if (setresuid(pw->pw_uid, pw->pw_uid, pw->pw_uid) == -1)
                 fatal("setresuid %u: %s", (u_int)pw->pw_uid, strerror(errno));                  fatal("setresuid %u: %s", (u_int)pw->pw_uid, strerror(errno));
 }  }
Legend:
Removed from v.1.41  
changed lines
  Added in v.1.42