132f8f8
diff -up openssh-6.8p1/channels.c.coverity openssh-6.8p1/channels.c
132f8f8
--- openssh-6.8p1/channels.c.coverity	2015-03-18 17:21:51.815265002 +0100
132f8f8
+++ openssh-6.8p1/channels.c	2015-03-18 17:21:51.896264833 +0100
132f8f8
@@ -243,11 +243,11 @@ channel_register_fds(Channel *c, int rfd
Jan F. Chadima 3b545be
 	channel_max_fd = MAX(channel_max_fd, wfd);
Jan F. Chadima 3b545be
 	channel_max_fd = MAX(channel_max_fd, efd);
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
-	if (rfd != -1)
Jan F. Chadima 3b545be
+	if (rfd >= 0)
Jan F. Chadima 3b545be
 		fcntl(rfd, F_SETFD, FD_CLOEXEC);
Jan F. Chadima 3b545be
-	if (wfd != -1 && wfd != rfd)
Jan F. Chadima 3b545be
+	if (wfd >= 0 && wfd != rfd)
Jan F. Chadima 3b545be
 		fcntl(wfd, F_SETFD, FD_CLOEXEC);
Jan F. Chadima 3b545be
-	if (efd != -1 && efd != rfd && efd != wfd)
Jan F. Chadima 3b545be
+	if (efd >= 0 && efd != rfd && efd != wfd)
Jan F. Chadima 3b545be
 		fcntl(efd, F_SETFD, FD_CLOEXEC);
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
 	c->rfd = rfd;
132f8f8
@@ -265,11 +265,11 @@ channel_register_fds(Channel *c, int rfd
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
 	/* enable nonblocking mode */
Jan F. Chadima 3b545be
 	if (nonblock) {
Jan F. Chadima 3b545be
-		if (rfd != -1)
Jan F. Chadima 3b545be
+		if (rfd >= 0)
Jan F. Chadima 3b545be
 			set_nonblock(rfd);
Jan F. Chadima 3b545be
-		if (wfd != -1)
Jan F. Chadima 3b545be
+		if (wfd >= 0)
Jan F. Chadima 3b545be
 			set_nonblock(wfd);
Jan F. Chadima 3b545be
-		if (efd != -1)
Jan F. Chadima 3b545be
+		if (efd >= 0)
Jan F. Chadima 3b545be
 			set_nonblock(efd);
Jan F. Chadima 3b545be
 	}
Jan F. Chadima 3b545be
 }
132f8f8
diff -up openssh-6.8p1/monitor.c.coverity openssh-6.8p1/monitor.c
132f8f8
--- openssh-6.8p1/monitor.c.coverity	2015-03-18 17:21:51.887264852 +0100
132f8f8
+++ openssh-6.8p1/monitor.c	2015-03-18 17:21:51.897264831 +0100
132f8f8
@@ -444,7 +444,7 @@ monitor_child_preauth(Authctxt *_authctx
8a29ded
 	mm_get_keystate(pmonitor);
Jan F. Chadima cff1d0c
 
Jan F. Chadima cff1d0c
 	/* Drain any buffered messages from the child */
Jan F. Chadima cff1d0c
-	while (pmonitor->m_log_recvfd != -1 && monitor_read_log(pmonitor) == 0)
Jan F. Chadima cff1d0c
+	while (pmonitor->m_log_recvfd >= 0 && monitor_read_log(pmonitor) == 0)
Jan F. Chadima cff1d0c
 		;
Jan F. Chadima cff1d0c
 
8a29ded
 	close(pmonitor->m_sendfd);
132f8f8
diff -up openssh-6.8p1/monitor_wrap.c.coverity openssh-6.8p1/monitor_wrap.c
132f8f8
--- openssh-6.8p1/monitor_wrap.c.coverity	2015-03-18 17:21:51.888264849 +0100
132f8f8
+++ openssh-6.8p1/monitor_wrap.c	2015-03-18 17:21:51.897264831 +0100
132f8f8
@@ -533,10 +533,10 @@ mm_pty_allocate(int *ptyfd, int *ttyfd,
Jan F. Chadima cff1d0c
 	if ((tmp1 = dup(pmonitor->m_recvfd)) == -1 ||
Jan F. Chadima cff1d0c
 	    (tmp2 = dup(pmonitor->m_recvfd)) == -1) {
Jan F. Chadima cff1d0c
 		error("%s: cannot allocate fds for pty", __func__);
Jan F. Chadima cff1d0c
-		if (tmp1 > 0)
Jan F. Chadima cff1d0c
+		if (tmp1 >= 0)
Jan F. Chadima cff1d0c
 			close(tmp1);
Jan F. Chadima cff1d0c
-		if (tmp2 > 0)
Jan F. Chadima cff1d0c
-			close(tmp2);
Jan F. Chadima cff1d0c
+		/*DEAD CODE if (tmp2 >= 0)
Jan F. Chadima cff1d0c
+			close(tmp2);*/
Jan F. Chadima cff1d0c
 		return 0;
Jan F. Chadima cff1d0c
 	}
Jan F. Chadima cff1d0c
 	close(tmp1);
132f8f8
diff -up openssh-6.8p1/openbsd-compat/bindresvport.c.coverity openssh-6.8p1/openbsd-compat/bindresvport.c
132f8f8
--- openssh-6.8p1/openbsd-compat/bindresvport.c.coverity	2015-03-17 06:49:20.000000000 +0100
132f8f8
+++ openssh-6.8p1/openbsd-compat/bindresvport.c	2015-03-18 17:21:51.897264831 +0100
132f8f8
@@ -58,7 +58,7 @@ bindresvport_sa(int sd, struct sockaddr
Jan F. Chadima ea97ffa
 	struct sockaddr_in6 *in6;
Jan F. Chadima ea97ffa
 	u_int16_t *portp;
Jan F. Chadima ea97ffa
 	u_int16_t port;
Jan F. Chadima ea97ffa
-	socklen_t salen;
Jan F. Chadima ea97ffa
+	socklen_t salen = sizeof(struct sockaddr_storage);
Jan F. Chadima ea97ffa
 	int i;
Jan F. Chadima ea97ffa
 
Jan F. Chadima ea97ffa
 	if (sa == NULL) {
132f8f8
diff -up openssh-6.8p1/scp.c.coverity openssh-6.8p1/scp.c
132f8f8
--- openssh-6.8p1/scp.c.coverity	2015-03-18 17:21:51.868264891 +0100
132f8f8
+++ openssh-6.8p1/scp.c	2015-03-18 17:21:58.281251460 +0100
132f8f8
@@ -156,7 +156,7 @@ killchild(int signo)
Jan F. Chadima 3b545be
 {
Jan F. Chadima 3b545be
 	if (do_cmd_pid > 1) {
Jan F. Chadima 3b545be
 		kill(do_cmd_pid, signo ? signo : SIGTERM);
Jan F. Chadima 3b545be
-		waitpid(do_cmd_pid, NULL, 0);
Jan F. Chadima 3b545be
+		(void) waitpid(do_cmd_pid, NULL, 0);
Jan F. Chadima 3b545be
 	}
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
 	if (signo)
132f8f8
diff -up openssh-6.8p1/servconf.c.coverity openssh-6.8p1/servconf.c
132f8f8
--- openssh-6.8p1/servconf.c.coverity	2015-03-18 17:21:51.893264839 +0100
132f8f8
+++ openssh-6.8p1/servconf.c	2015-03-18 17:21:58.281251460 +0100
132f8f8
@@ -1475,7 +1475,7 @@ process_server_config_line(ServerOptions
Jan F. Chadima ea97ffa
 			fatal("%s line %d: Missing subsystem name.",
Jan F. Chadima ea97ffa
 			    filename, linenum);
Jan F. Chadima ea97ffa
 		if (!*activep) {
Jan F. Chadima ea97ffa
-			arg = strdelim(&cp;;
Jan F. Chadima ea97ffa
+			/*arg =*/ (void) strdelim(&cp;;
Jan F. Chadima ea97ffa
 			break;
Jan F. Chadima ea97ffa
 		}
Jan F. Chadima ea97ffa
 		for (i = 0; i < options->num_subsystems; i++)
132f8f8
@@ -1566,8 +1566,9 @@ process_server_config_line(ServerOptions
Jan F. Chadima cff1d0c
 		if (*activep && *charptr == NULL) {
Jan F. Chadima cff1d0c
 			*charptr = tilde_expand_filename(arg, getuid());
Jan F. Chadima cff1d0c
 			/* increase optional counter */
Jan F. Chadima cff1d0c
-			if (intptr != NULL)
Jan F. Chadima cff1d0c
-				*intptr = *intptr + 1;
Jan F. Chadima cff1d0c
+			/* DEAD CODE intptr is still NULL ;)
Jan F. Chadima cff1d0c
+  			 if (intptr != NULL)
Jan F. Chadima cff1d0c
+				*intptr = *intptr + 1; */
Jan F. Chadima cff1d0c
 		}
Jan F. Chadima cff1d0c
 		break;
Jan F. Chadima cff1d0c
 
132f8f8
diff -up openssh-6.8p1/serverloop.c.coverity openssh-6.8p1/serverloop.c
132f8f8
--- openssh-6.8p1/serverloop.c.coverity	2015-03-17 06:49:20.000000000 +0100
132f8f8
+++ openssh-6.8p1/serverloop.c	2015-03-18 17:28:45.616436080 +0100
Jan F. Chadima 3b545be
@@ -147,13 +147,13 @@ notify_setup(void)
Jan F. Chadima 3b545be
 static void
Jan F. Chadima 3b545be
 notify_parent(void)
Jan F. Chadima 3b545be
 {
Jan F. Chadima 3b545be
-	if (notify_pipe[1] != -1)
Jan F. Chadima 3b545be
+	if (notify_pipe[1] >= 0)
84822b5
 		(void)write(notify_pipe[1], "", 1);
Jan F. Chadima 3b545be
 }
Jan F. Chadima 3b545be
 static void
Jan F. Chadima 3b545be
 notify_prepare(fd_set *readset)
Jan F. Chadima 3b545be
 {
Jan F. Chadima 3b545be
-	if (notify_pipe[0] != -1)
Jan F. Chadima 3b545be
+	if (notify_pipe[0] >= 0)
Jan F. Chadima 3b545be
 		FD_SET(notify_pipe[0], readset);
Jan F. Chadima 3b545be
 }
Jan F. Chadima 3b545be
 static void
Jan F. Chadima 3b545be
@@ -161,8 +161,8 @@ notify_done(fd_set *readset)
Jan F. Chadima 3b545be
 {
Jan F. Chadima 3b545be
 	char c;
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
-	if (notify_pipe[0] != -1 && FD_ISSET(notify_pipe[0], readset))
Jan F. Chadima 3b545be
-		while (read(notify_pipe[0], &c, 1) != -1)
Jan F. Chadima 3b545be
+	if (notify_pipe[0] >= 0 && FD_ISSET(notify_pipe[0], readset))
Jan F. Chadima 3b545be
+		while (read(notify_pipe[0], &c, 1) >= 0)
Jan F. Chadima 3b545be
 			debug2("notify_done: reading");
Jan F. Chadima 3b545be
 }
Jan F. Chadima 3b545be
 
132f8f8
@@ -337,7 +337,7 @@ wait_until_can_do_something(fd_set **rea
Jan F. Chadima 3b545be
 		 * If we have buffered data, try to write some of that data
Jan F. Chadima 3b545be
 		 * to the program.
Jan F. Chadima 3b545be
 		 */
Jan F. Chadima 3b545be
-		if (fdin != -1 && buffer_len(&stdin_buffer) > 0)
Jan F. Chadima 3b545be
+		if (fdin >= 0 && buffer_len(&stdin_buffer) > 0)
Jan F. Chadima 3b545be
 			FD_SET(fdin, *writesetp);
Jan F. Chadima 3b545be
 	}
Jan F. Chadima 3b545be
 	notify_prepare(*readsetp);
44f0ac8
@@ -477,7 +477,7 @@ process_output(fd_set *writeset)
Jan F. Chadima 3b545be
 	int len;
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
 	/* Write buffered data to program stdin. */
Jan F. Chadima 3b545be
-	if (!compat20 && fdin != -1 && FD_ISSET(fdin, writeset)) {
Jan F. Chadima 3b545be
+	if (!compat20 && fdin >= 0 && FD_ISSET(fdin, writeset)) {
Jan F. Chadima 3b545be
 		data = buffer_ptr(&stdin_buffer);
Jan F. Chadima 3b545be
 		dlen = buffer_len(&stdin_buffer);
Jan F. Chadima 3b545be
 		len = write(fdin, data, dlen);
132f8f8
@@ -590,7 +590,7 @@ server_loop(pid_t pid, int fdin_arg, int
Jan F. Chadima 3b545be
 	set_nonblock(fdin);
Jan F. Chadima 3b545be
 	set_nonblock(fdout);
Jan F. Chadima 3b545be
 	/* we don't have stderr for interactive terminal sessions, see below */
Jan F. Chadima 3b545be
-	if (fderr != -1)
Jan F. Chadima 3b545be
+	if (fderr >= 0)
Jan F. Chadima 3b545be
 		set_nonblock(fderr);
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
 	if (!(datafellows & SSH_BUG_IGNOREMSG) && isatty(fdin))
132f8f8
@@ -614,7 +614,7 @@ server_loop(pid_t pid, int fdin_arg, int
Jan F. Chadima 3b545be
 	max_fd = MAX(connection_in, connection_out);
Jan F. Chadima 3b545be
 	max_fd = MAX(max_fd, fdin);
Jan F. Chadima 3b545be
 	max_fd = MAX(max_fd, fdout);
Jan F. Chadima 3b545be
-	if (fderr != -1)
Jan F. Chadima 3b545be
+	if (fderr >= 0)
Jan F. Chadima 3b545be
 		max_fd = MAX(max_fd, fderr);
Jan F. Chadima 3b545be
 #endif
Jan F. Chadima 3b545be
 
132f8f8
@@ -644,7 +644,7 @@ server_loop(pid_t pid, int fdin_arg, int
Jan F. Chadima 3b545be
 		 * If we have received eof, and there is no more pending
Jan F. Chadima 3b545be
 		 * input data, cause a real eof by closing fdin.
Jan F. Chadima 3b545be
 		 */
Jan F. Chadima 3b545be
-		if (stdin_eof && fdin != -1 && buffer_len(&stdin_buffer) == 0) {
Jan F. Chadima 3b545be
+		if (stdin_eof && fdin >= 0 && buffer_len(&stdin_buffer) == 0) {
Jan F. Chadima 3b545be
 			if (fdin != fdout)
Jan F. Chadima 3b545be
 				close(fdin);
Jan F. Chadima 3b545be
 			else
132f8f8
@@ -740,15 +740,15 @@ server_loop(pid_t pid, int fdin_arg, int
Jan F. Chadima 3b545be
 	buffer_free(&stderr_buffer);
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
 	/* Close the file descriptors. */
Jan F. Chadima 3b545be
-	if (fdout != -1)
Jan F. Chadima 3b545be
+	if (fdout >= 0)
Jan F. Chadima 3b545be
 		close(fdout);
Jan F. Chadima 3b545be
 	fdout = -1;
Jan F. Chadima 3b545be
 	fdout_eof = 1;
Jan F. Chadima 3b545be
-	if (fderr != -1)
Jan F. Chadima 3b545be
+	if (fderr >= 0)
Jan F. Chadima 3b545be
 		close(fderr);
Jan F. Chadima 3b545be
 	fderr = -1;
Jan F. Chadima 3b545be
 	fderr_eof = 1;
Jan F. Chadima 3b545be
-	if (fdin != -1)
Jan F. Chadima 3b545be
+	if (fdin >= 0)
Jan F. Chadima 3b545be
 		close(fdin);
Jan F. Chadima 3b545be
 	fdin = -1;
Jan F. Chadima 3b545be
 
132f8f8
@@ -950,7 +950,7 @@ server_input_window_size(int type, u_int
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
 	debug("Window change received.");
Jan F. Chadima 3b545be
 	packet_check_eom();
Jan F. Chadima 3b545be
-	if (fdin != -1)
Jan F. Chadima 3b545be
+	if (fdin >= 0)
Jan F. Chadima 3b545be
 		pty_change_window_size(fdin, row, col, xpixel, ypixel);
132f8f8
 	return 0;
Jan F. Chadima 3b545be
 }
132f8f8
@@ -1043,7 +1043,7 @@ server_request_tun(void)
Jan F. Chadima 3b545be
 	}
Jan F. Chadima 3b545be
 
Jan F. Chadima 3b545be
 	tun = packet_get_int();
Jan F. Chadima 3b545be
-	if (forced_tun_device != -1) {
Jan F. Chadima 3b545be
+	if (forced_tun_device >= 0) {
Jan F. Chadima 3b545be
 		if (tun != SSH_TUNID_ANY && forced_tun_device != tun)
Jan F. Chadima 3b545be
 			goto done;
Jan F. Chadima 3b545be
 		tun = forced_tun_device;
132f8f8
diff -up openssh-6.8p1/sftp.c.coverity openssh-6.8p1/sftp.c
132f8f8
--- openssh-6.8p1/sftp.c.coverity	2015-03-17 06:49:20.000000000 +0100
132f8f8
+++ openssh-6.8p1/sftp.c	2015-03-18 17:21:58.283251456 +0100
132f8f8
@@ -223,7 +223,7 @@ killchild(int signo)
84822b5
 {
84822b5
 	if (sshpid > 1) {
84822b5
 		kill(sshpid, SIGTERM);
84822b5
-		waitpid(sshpid, NULL, 0);
84822b5
+		(void) waitpid(sshpid, NULL, 0);
84822b5
 	}
84822b5
 
84822b5
 	_exit(1);
132f8f8
diff -up openssh-6.8p1/ssh-agent.c.coverity openssh-6.8p1/ssh-agent.c
132f8f8
--- openssh-6.8p1/ssh-agent.c.coverity	2015-03-17 06:49:20.000000000 +0100
132f8f8
+++ openssh-6.8p1/ssh-agent.c	2015-03-18 17:21:58.284251454 +0100
132f8f8
@@ -1166,8 +1166,8 @@ main(int ac, char **av)
Jan F. Chadima ea97ffa
 	sanitise_stdfd();
Jan F. Chadima ea97ffa
 
Jan F. Chadima ea97ffa
 	/* drop */
Jan F. Chadima ea97ffa
-	setegid(getgid());
Jan F. Chadima ea97ffa
-	setgid(getgid());
Jan F. Chadima ea97ffa
+	(void) setegid(getgid());
Jan F. Chadima ea97ffa
+	(void) setgid(getgid());
Jan F. Chadima ea97ffa
 
5878ebb
 	platform_disable_tracing(0);	/* strict=no */
5878ebb
 
132f8f8
diff -up openssh-6.8p1/sshd.c.coverity openssh-6.8p1/sshd.c
132f8f8
--- openssh-6.8p1/sshd.c.coverity	2015-03-18 17:21:51.893264839 +0100
132f8f8
+++ openssh-6.8p1/sshd.c	2015-03-18 17:21:58.284251454 +0100
132f8f8
@@ -778,8 +778,10 @@ privsep_preauth(Authctxt *authctxt)
13073f8
 
13073f8
 		privsep_preauth_child();
Jan F. Chadima cff1d0c
 		setproctitle("%s", "[net]");
Jan F. Chadima cff1d0c
-		if (box != NULL)
Jan F. Chadima cff1d0c
+		if (box != NULL) {
Jan F. Chadima cff1d0c
 			ssh_sandbox_child(box);
84822b5
+			free(box);
Jan F. Chadima cff1d0c
+		}
Jan F. Chadima cff1d0c
 
Jan F. Chadima cff1d0c
 		return 0;
Jan F. Chadima cff1d0c
 	}
132f8f8
@@ -1518,6 +1520,9 @@ server_accept_loop(int *sock_in, int *so
Jan F. Chadima 3b545be
 		if (num_listen_socks < 0)
Jan F. Chadima 3b545be
 			break;
Jan F. Chadima 3b545be
 	}
Jan F. Chadima 3b545be
+
Jan F. Chadima 3b545be
+	if (fdset != NULL)
84822b5
+		free(fdset);
Jan F. Chadima 3b545be
 }
Jan F. Chadima 3b545be
 
5878ebb
 /*