rpms/openssh/devel openssh-5.1p1-bannerlen.patch, NONE, 1.1 openssh.spec, 1.130, 1.131

[Tomáš Mráz (tmraz)]

Author: tmraz

Update of /cvs/pkgs/rpms/openssh/devel
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv18922

Modified Files:
	openssh.spec 
Added Files:
	openssh-5.1p1-bannerlen.patch 
Log Message:
* Thu Jul 31 2008 Tomas Mraz <tmraz at redhat.com> - 5.1p1-2
- fixed zero length banner problem (#457326)


openssh-5.1p1-bannerlen.patch:

--- NEW FILE openssh-5.1p1-bannerlen.patch ---
diff -up openssh-5.1p1/sshconnect2.c.bannerlen openssh-5.1p1/sshconnect2.c
--- openssh-5.1p1/sshconnect2.c.bannerlen	2008-07-23 19:18:15.000000000 +0200
+++ openssh-5.1p1/sshconnect2.c	2008-07-31 11:16:51.000000000 +0200
@@ -387,8 +387,9 @@ input_userauth_banner(int type, u_int32_
 	if (options.log_level >= SYSLOG_LEVEL_INFO) {
 		if (len > 65536)
 			len = 65536;
-		msg = xmalloc(len * 4); /* max expansion from strnvis() */
+		msg = xmalloc(len * 4 + 1); /* max expansion from strnvis() */
 		strnvis(msg, raw, len * 4, VIS_SAFE|VIS_OCTAL);
+		msg[len*4] = '\0';
 		fprintf(stderr, "%s", msg);
 		xfree(msg);
 	}


Index: openssh.spec
===================================================================
RCS file: /cvs/pkgs/rpms/openssh/devel/openssh.spec,v
retrieving revision 1.130
retrieving revision 1.131
diff -u -r1.130 -r1.131
--- openssh.spec	23 Jul 2008 17:33:16 -0000	1.130
+++ openssh.spec	31 Jul 2008 09:22:18 -0000	1.131
@@ -63,7 +63,7 @@
 Summary: The OpenSSH implementation of SSH protocol versions 1 and 2
 Name: openssh
 Version: 5.1p1
-Release: 1%{?dist}%{?rescue_rel}
+Release: 2%{?dist}%{?rescue_rel}
 URL: http://www.openssh.com/portable.html
 #Source0: ftp://ftp.openbsd.org/pub/OpenBSD/OpenSSH/portable/openssh-%{version}.tar.gz
 #Source1: ftp://ftp.openbsd.org/pub/OpenBSD/OpenSSH/portable/openssh-%{version}.tar.gz.asc
@@ -96,6 +96,7 @@
 Patch54: openssh-5.1p1-gssapi-role.patch
 Patch55: openssh-5.1p1-cloexec.patch
 Patch62: openssh-5.1p1-scp-manpage.patch
+Patch63: openssh-5.1p1-bannerlen.patch
 
 License: BSD
 Group: Applications/Internet
@@ -227,6 +228,7 @@
 %patch54 -p1 -b .gssapi-role
 %patch55 -p1 -b .cloexec
 %patch62 -p1 -b .manpage
+%patch63 -p1 -b .bannerlen
 
 autoreconf
 
@@ -478,6 +480,9 @@
 %endif
 
 %changelog
+* Thu Jul 31 2008 Tomas Mraz <tmraz at redhat.com> - 5.1p1-2
+- fixed zero length banner problem (#457326)
+
 * Wed Jul 23 2008 Tomas Mraz <tmraz at redhat.com> - 5.1p1-1
 - upgrade to new upstream release
 - fixed a problem with public key authentication and explicitely