From owner-freebsd-current  Tue Nov 11 21:08:29 1997
Return-Path: <owner-freebsd-current>
Received: (from root@localhost)
          by hub.freebsd.org (8.8.7/8.8.7) id VAA05156
          for current-outgoing; Tue, 11 Nov 1997 21:08:29 -0800 (PST)
          (envelope-from owner-freebsd-current)
Received: from kithrup.com (kithrup.com [205.179.156.40])
          by hub.freebsd.org (8.8.7/8.8.7) with ESMTP id VAA05147
          for <current@freebsd.org>; Tue, 11 Nov 1997 21:08:25 -0800 (PST)
          (envelope-from sef@kithrup.com)
Received: (from sef@localhost)
	by kithrup.com (8.8.7/8.8.7) id VAA01909
	for current@freebsd.org; Tue, 11 Nov 1997 21:08:23 -0800 (PST)
	(envelope-from sef)
Date: Tue, 11 Nov 1997 21:08:23 -0800 (PST)
From: Sean Eric Fagan <sef@kithrup.com>
Message-Id: <199711120508.VAA01909@kithrup.com>
To: current@freebsd.org
Subject: Anyone object to this login.c patch?
Sender: owner-freebsd-current@freebsd.org
X-Loop: FreeBSD.org
Precedence: bulk

I need logging for failed logins, and I was unable to figure out how to get
it without this change.  It will log failures more often than the other
would (well, I was able to get *any*, when I wasn't before).

Index: login.c
===================================================================
RCS file: /usr/cvs/src/usr.bin/login/login.c,v
retrieving revision 1.30
diff -u -r1.30 login.c
--- login.c	1997/10/19 09:34:06	1.30
+++ login.c	1997/11/12 05:05:54
@@ -502,6 +502,7 @@
 
 		(void)printf("Login incorrect\n");
 		failures++;
+		badlogin(username);
 
 		/*
 		 * we allow up to 'retry' (10) tries,
@@ -509,7 +510,6 @@
 		 */
 		if (++cnt > backoff) {
 			if (cnt >= retries) {
-				badlogin(username);
 				sleepexit(1);
 			}
 			sleep((u_int)((cnt - 3) * 5));