From owner-freebsd-bugs@FreeBSD.ORG Thu Jan 20 15:10:26 2005 Return-Path: Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 7027F16A4DD for ; Thu, 20 Jan 2005 15:10:26 +0000 (GMT) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id 51CF143D46 for ; Thu, 20 Jan 2005 15:10:26 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.13.1/8.13.1) with ESMTP id j0KFAQBq028895 for ; Thu, 20 Jan 2005 15:10:26 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.13.1/8.13.1/Submit) id j0KFAQhg028894; Thu, 20 Jan 2005 15:10:26 GMT (envelope-from gnats) Date: Thu, 20 Jan 2005 15:10:26 GMT Message-Id: <200501201510.j0KFAQhg028894@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: Rob Mason Subject: Re: misc/76494: passwd does not indicate a successful change - causes problems with Samba password sync X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list Reply-To: Rob Mason List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Jan 2005 15:10:26 -0000 The following reply was made to PR misc/76494; it has been noted by GNATS. From: Rob Mason To: Ceri Davies Cc: Subject: Re: misc/76494: passwd does not indicate a successful change - causes problems with Samba password sync Date: Thu, 20 Jan 2005 14:48:51 +0000 On Thu, Jan 20, 2005 at 10:17:50AM +0000, Ceri Davies wrote: > On Thu, Jan 20, 2005 at 09:46:31AM +0000, Rob Mason wrote: > > > The passwd command on 4-STABLE output the text "passwd: done" when a > > password was changed. This feature is used by the Samba-3 port to > > indicate a successful change in the 'passwd chat' config option. > > 5-STABLE does not indicate that the password change is successful > > What's wrong with a return status? Nothing from a functional point of view - however, this is not how the 4-STABLE passwd command behaved. I believe that other programs (like Samba) that are dependent upon core system utilities will break. This will be a major source of frustration for many admins. Perhaps I should have filed this PR under 'advocacy'? > > > Apply the following patch: > > > > *** 157,164 **** > > --- 157,165 ---- > > /* set new password */ > > pam_err = pam_chauthtok(pamh, 0); > > pam_check("pam_chauthtok"); > > > > end: > > + if (pam_err == PAM_SUCCESS) fprintf(stderr, "Password Changed\n"); > > pam_end(pamh, pam_err); > > exit(pam_err == PAM_SUCCESS ? 0 : 1); > > } > > It also strikes me that this is not the same as the 4.X output; why? Apologies - my mistake. This is the incorrect output for 4-STABLE. > > Ceri