diff options
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | modules/pam_rhosts/pam_rhosts_auth.c | 2 |
2 files changed, 8 insertions, 1 deletions
@@ -1,3 +1,10 @@ +2007-08-29 Steve Langasek <vorlon@debian.org> + + * modules/pam_rhosts/pam_rhosts_auth.c: getline returns -1 at + EOF, not 0. Check accordingly to fix an infinite loop. Thanks + to Stephan Springl <springl-rhosts@bfw-online.de> for catching + this. + 2007-08-28 Steve Langasek <vorlon@debian.org> * configure.in: call AC_CHECK_HEADERS instead of AC_CHECK_HEADER diff --git a/modules/pam_rhosts/pam_rhosts_auth.c b/modules/pam_rhosts/pam_rhosts_auth.c index d50ed478..a798c1f0 100644 --- a/modules/pam_rhosts/pam_rhosts_auth.c +++ b/modules/pam_rhosts/pam_rhosts_auth.c @@ -352,7 +352,7 @@ __ivaliduser (pam_handle_t *pamh, struct _options *opts, char *buf=NULL; int buflen=0; - while (getline(&buf,&buflen,hostf)) { + while (getline(&buf,&buflen,hostf) > 0) { #else char buf[MAXHOSTNAMELEN + 128]; /* host + login */ |