commit fecede00a76fbb33a349f5121c0b2f9fbc04a777
author markus@openbsd.org <markus@openbsd.org> Thu Mar 26 19:32:19 2015 +0000
committer Damien Miller <djm@mindrot.org> Fri Mar 27 12:02:38 2015 +1100
tree 7daed3709d095d068ba54f33a1eb5949d858b14f
parent 1b0ef3813244c78669e6d4d54c624f600945327d

upstream commit

sigp and lenp are not optional in ssh_agent_sign(); ok
 djm@

authfd.c

diff --git a/authfd.c b/authfd.c
index 9ed5d2f..82915a4 100644
--- a/authfd.c
+++ b/authfd.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: authfd.c,v 1.96 2015/03/25 19:21:48 markus Exp $ */
+/* $OpenBSD: authfd.c,v 1.97 2015/03/26 19:32:19 markus Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -438,10 +438,8 @@
 	u_int flags = 0;
 	int r = SSH_ERR_INTERNAL_ERROR;
 
-	if (sigp != NULL)
-		*sigp = NULL;
-	if (lenp != NULL)
-		*lenp = 0;
+	*sigp = NULL;
+	*lenp = 0;
 
 	if (datalen > SSH_KEY_MAX_SIGN_DATA_SIZE)
 		return SSH_ERR_INVALID_ARGUMENT;
@@ -469,8 +467,7 @@
 	}
 	if ((r = sshbuf_get_string(msg, sigp, &len)) != 0)
 		goto out;
-	if (lenp != NULL)
-		*lenp = len;
+	*lenp = len;
 	r = 0;
  out:
 	if (blob != NULL) {