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

Diff for /src/usr.bin/ssh/session.c between version 1.315 and 1.316

version 1.315, 2019/02/22 03:37:11 version 1.316, 2019/06/28 13:35:04
Line 384 
Line 384 
                 fatal("do_exec_no_pty: no session");                  fatal("do_exec_no_pty: no session");
   
         /* Allocate pipes for communicating with the program. */          /* Allocate pipes for communicating with the program. */
         if (pipe(pin) < 0) {          if (pipe(pin) == -1) {
                 error("%s: pipe in: %.100s", __func__, strerror(errno));                  error("%s: pipe in: %.100s", __func__, strerror(errno));
                 return -1;                  return -1;
         }          }
         if (pipe(pout) < 0) {          if (pipe(pout) == -1) {
                 error("%s: pipe out: %.100s", __func__, strerror(errno));                  error("%s: pipe out: %.100s", __func__, strerror(errno));
                 close(pin[0]);                  close(pin[0]);
                 close(pin[1]);                  close(pin[1]);
                 return -1;                  return -1;
         }          }
         if (pipe(perr) < 0) {          if (pipe(perr) == -1) {
                 error("%s: pipe err: %.100s", __func__,                  error("%s: pipe err: %.100s", __func__,
                     strerror(errno));                      strerror(errno));
                 close(pin[0]);                  close(pin[0]);
Line 410 
Line 410 
                 fatal("do_exec_no_pty: no session");                  fatal("do_exec_no_pty: no session");
   
         /* Uses socket pairs to communicate with the program. */          /* Uses socket pairs to communicate with the program. */
         if (socketpair(AF_UNIX, SOCK_STREAM, 0, inout) < 0) {          if (socketpair(AF_UNIX, SOCK_STREAM, 0, inout) == -1) {
                 error("%s: socketpair #1: %.100s", __func__, strerror(errno));                  error("%s: socketpair #1: %.100s", __func__, strerror(errno));
                 return -1;                  return -1;
         }          }
         if (socketpair(AF_UNIX, SOCK_STREAM, 0, err) < 0) {          if (socketpair(AF_UNIX, SOCK_STREAM, 0, err) == -1) {
                 error("%s: socketpair #2: %.100s", __func__,                  error("%s: socketpair #2: %.100s", __func__,
                     strerror(errno));                      strerror(errno));
                 close(inout[0]);                  close(inout[0]);
Line 450 
Line 450 
                  * Create a new session and process group since the 4.4BSD                   * Create a new session and process group since the 4.4BSD
                  * setlogin() affects the entire process group.                   * setlogin() affects the entire process group.
                  */                   */
                 if (setsid() < 0)                  if (setsid() == -1)
                         error("setsid failed: %.100s", strerror(errno));                          error("setsid failed: %.100s", strerror(errno));
   
 #ifdef USE_PIPES  #ifdef USE_PIPES
Line 459 
Line 459 
                  * pair, and make the child side the standard input.                   * pair, and make the child side the standard input.
                  */                   */
                 close(pin[1]);                  close(pin[1]);
                 if (dup2(pin[0], 0) < 0)                  if (dup2(pin[0], 0) == -1)
                         perror("dup2 stdin");                          perror("dup2 stdin");
                 close(pin[0]);                  close(pin[0]);
   
                 /* Redirect stdout. */                  /* Redirect stdout. */
                 close(pout[0]);                  close(pout[0]);
                 if (dup2(pout[1], 1) < 0)                  if (dup2(pout[1], 1) == -1)
                         perror("dup2 stdout");                          perror("dup2 stdout");
                 close(pout[1]);                  close(pout[1]);
   
                 /* Redirect stderr. */                  /* Redirect stderr. */
                 close(perr[0]);                  close(perr[0]);
                 if (dup2(perr[1], 2) < 0)                  if (dup2(perr[1], 2) == -1)
                         perror("dup2 stderr");                          perror("dup2 stderr");
                 close(perr[1]);                  close(perr[1]);
 #else  #else
Line 482 
Line 482 
                  */                   */
                 close(inout[1]);                  close(inout[1]);
                 close(err[1]);                  close(err[1]);
                 if (dup2(inout[0], 0) < 0)      /* stdin */                  if (dup2(inout[0], 0) == -1)    /* stdin */
                         perror("dup2 stdin");                          perror("dup2 stdin");
                 if (dup2(inout[0], 1) < 0)      /* stdout (same as stdin) */                  if (dup2(inout[0], 1) == -1)    /* stdout (same as stdin) */
                         perror("dup2 stdout");                          perror("dup2 stdout");
                 close(inout[0]);                  close(inout[0]);
                 if (dup2(err[0], 2) < 0)        /* stderr */                  if (dup2(err[0], 2) == -1)      /* stderr */
                         perror("dup2 stderr");                          perror("dup2 stderr");
                 close(err[0]);                  close(err[0]);
 #endif  #endif
Line 551 
Line 551 
          * Do this before forking (and cleanup in the child) so as to           * Do this before forking (and cleanup in the child) so as to
          * detect and gracefully fail out-of-fd conditions.           * detect and gracefully fail out-of-fd conditions.
          */           */
         if ((fdout = dup(ptyfd)) < 0) {          if ((fdout = dup(ptyfd)) == -1) {
                 error("%s: dup #1: %s", __func__, strerror(errno));                  error("%s: dup #1: %s", __func__, strerror(errno));
                 close(ttyfd);                  close(ttyfd);
                 close(ptyfd);                  close(ptyfd);
                 return -1;                  return -1;
         }          }
         /* we keep a reference to the pty master */          /* we keep a reference to the pty master */
         if ((ptymaster = dup(ptyfd)) < 0) {          if ((ptymaster = dup(ptyfd)) == -1) {
                 error("%s: dup #2: %s", __func__, strerror(errno));                  error("%s: dup #2: %s", __func__, strerror(errno));
                 close(ttyfd);                  close(ttyfd);
                 close(ptyfd);                  close(ptyfd);
Line 588 
Line 588 
                 pty_make_controlling_tty(&ttyfd, s->tty);                  pty_make_controlling_tty(&ttyfd, s->tty);
   
                 /* Redirect stdin/stdout/stderr from the pseudo tty. */                  /* Redirect stdin/stdout/stderr from the pseudo tty. */
                 if (dup2(ttyfd, 0) < 0)                  if (dup2(ttyfd, 0) == -1)
                         error("dup2 stdin: %s", strerror(errno));                          error("dup2 stdin: %s", strerror(errno));
                 if (dup2(ttyfd, 1) < 0)                  if (dup2(ttyfd, 1) == -1)
                         error("dup2 stdout: %s", strerror(errno));                          error("dup2 stdout: %s", strerror(errno));
                 if (dup2(ttyfd, 2) < 0)                  if (dup2(ttyfd, 2) == -1)
                         error("dup2 stderr: %s", strerror(errno));                          error("dup2 stderr: %s", strerror(errno));
   
                 /* Close the extra descriptor for the pseudo tty. */                  /* Close the extra descriptor for the pseudo tty. */
Line 720 
Line 720 
         fromlen = sizeof(from);          fromlen = sizeof(from);
         if (ssh_packet_connection_is_on_socket(ssh)) {          if (ssh_packet_connection_is_on_socket(ssh)) {
                 if (getpeername(ssh_packet_get_connection_in(ssh),                  if (getpeername(ssh_packet_get_connection_in(ssh),
                     (struct sockaddr *)&from, &fromlen) < 0) {                      (struct sockaddr *)&from, &fromlen) == -1) {
                         debug("getpeername: %.100s", strerror(errno));                          debug("getpeername: %.100s", strerror(errno));
                         cleanup_exit(255);                          cleanup_exit(255);
                 }                  }
Line 1305 
Line 1305 
 #endif  #endif
   
         /* Change current directory to the user's home directory. */          /* Change current directory to the user's home directory. */
         if (chdir(pw->pw_dir) < 0) {          if (chdir(pw->pw_dir) == -1) {
                 /* Suppress missing homedir warning for chroot case */                  /* Suppress missing homedir warning for chroot case */
                 r = login_getcapbool(lc, "requirehome", 0);                  r = login_getcapbool(lc, "requirehome", 0);
                 if (r || !in_chroot) {                  if (r || !in_chroot) {
Line 1654 
Line 1654 
                                 s->is_subsystem = SUBSYSTEM_INT_SFTP;                                  s->is_subsystem = SUBSYSTEM_INT_SFTP;
                                 debug("subsystem: %s", prog);                                  debug("subsystem: %s", prog);
                         } else {                          } else {
                                 if (stat(prog, &st) < 0)                                  if (stat(prog, &st) == -1)
                                         debug("subsystem: cannot stat %s: %s",                                          debug("subsystem: cannot stat %s: %s",
                                             prog, strerror(errno));                                              prog, strerror(errno));
                                 s->is_subsystem = SUBSYSTEM_EXT;                                  s->is_subsystem = SUBSYSTEM_EXT;
Line 1743 
Line 1743 
             (r = sshpkt_get_end(ssh)) != 0)              (r = sshpkt_get_end(ssh)) != 0)
                 sshpkt_fatal(ssh, r, "%s: parse packet", __func__);                  sshpkt_fatal(ssh, r, "%s: parse packet", __func__);
   
         if (s->ptymaster == -1 || tcsendbreak(s->ptymaster, 0) < 0)          if (s->ptymaster == -1 || tcsendbreak(s->ptymaster, 0) == -1)
                 return 0;                  return 0;
         return 1;          return 1;
 }  }
Line 1965 
Line 1965 
          * the pty cleanup, so that another process doesn't get this pty           * the pty cleanup, so that another process doesn't get this pty
          * while we're still cleaning up.           * while we're still cleaning up.
          */           */
         if (s->ptymaster != -1 && close(s->ptymaster) < 0)          if (s->ptymaster != -1 && close(s->ptymaster) == -1)
                 error("close(s->ptymaster/%d): %s",                  error("close(s->ptymaster/%d): %s",
                     s->ptymaster, strerror(errno));                      s->ptymaster, strerror(errno));
   
Line 2265 
Line 2265 
         }          }
   
         /* Set up a suitable value for the DISPLAY variable. */          /* Set up a suitable value for the DISPLAY variable. */
         if (gethostname(hostname, sizeof(hostname)) < 0)          if (gethostname(hostname, sizeof(hostname)) == -1)
                 fatal("gethostname: %.100s", strerror(errno));                  fatal("gethostname: %.100s", strerror(errno));
         /*          /*
          * auth_display must be used as the displayname when the           * auth_display must be used as the displayname when the
Legend:
Removed from v.1.315  
changed lines
  Added in v.1.316