commit d0562b3cdc06c62ef54bc01870e3b642604f60be
author Damien Miller <djm@mindrot.org> Fri Oct 29 13:09:40 1999 +1000
committer Damien Miller <djm@mindrot.org> Fri Oct 29 13:09:40 1999 +1000
tree 91cfd87a0e5cea0cae6ab6effc77abb365d647de
parent 65b3c13132633189b49bf0d7002b55755280d6d3

Fix off by one error in PAM env patch

Tidy changelog

ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 04a5f7d..02eeb8a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -9,6 +9,8 @@
  - Removed redundant subdirectories
  - Integrated part of a patch from Dan Brosemer <odin@linuxfreak.com> for 
    building on Debian.
+ - Fixed off-by-one error in PAM env patch
+ - Released 1.2pre6
 
 19991028
  - Further PAM enhancements.

sshd.c

diff --git a/sshd.c b/sshd.c
index 028c495..a4af9e2 100644
--- a/sshd.c
+++ b/sshd.c
@@ -18,7 +18,7 @@
 */
 
 #include "includes.h"
-RCSID("$Id: sshd.c,v 1.8 1999/10/29 01:49:20 damien Exp $");
+RCSID("$Id: sshd.c,v 1.9 1999/10/29 03:09:40 damien Exp $");
 
 #include "xmalloc.h"
 #include "rsa.h"
@@ -2365,7 +2365,7 @@
     char **pam_env = pam_getenvlist((pam_handle_t *)pamh);
     for(this_var = 0; pam_env && pam_env[this_var]; this_var++)
       {
-        if(strlen(pam_env[this_var]) < sizeof(var_name))
+        if(strlen(pam_env[this_var]) < (sizeof(var_name) - 1))
           if((equal_sign = strstr(pam_env[this_var], "=")) != NULL)
             {
               memset(var_name, 0, sizeof(var_name));