From owner-freebsd-bugs@FreeBSD.ORG Fri Jul 11 04:22:41 2008 Return-Path: Delivered-To: freebsd-bugs@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 5DB27106564A for ; Fri, 11 Jul 2008 04:22:41 +0000 (UTC) (envelope-from unga888@yahoo.com) Received: from web57007.mail.re3.yahoo.com (web57007.mail.re3.yahoo.com [66.196.97.111]) by mx1.freebsd.org (Postfix) with SMTP id E7D798FC1C for ; Fri, 11 Jul 2008 04:22:40 +0000 (UTC) (envelope-from unga888@yahoo.com) Received: (qmail 62949 invoked by uid 60001); 11 Jul 2008 03:55:59 -0000 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=s1024; d=yahoo.com; h=Received:X-Mailer:Date:From:Reply-To:Subject:To:MIME-Version:Content-Type:Message-ID; b=rateIBbDWcZFMSZugH3Ozc7X+XJibVOBAJaUCrnXQ+iSkrtHzcdz0CKdbQtWC0TBkWBOtDmiN4Y1pCBupUZI4/fMoPktldTdymjIhnEgpH+gh/VBTWKutglmSbi/r6sc0t5FD788Gcu+NoQpIH/+rS2MAHe8CV0oQrl9C18GXZw=; Received: from [165.21.155.117] by web57007.mail.re3.yahoo.com via HTTP; Thu, 10 Jul 2008 20:55:58 PDT X-Mailer: YahooMailWebService/0.7.199 Date: Thu, 10 Jul 2008 20:55:58 -0700 (PDT) From: Unga To: freebsd-bugs@FreeBSD.org MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="0-1777159550-1215748558=:62863" Message-ID: <127065.62863.qm@web57007.mail.re3.yahoo.com> X-Mailman-Approved-At: Fri, 11 Jul 2008 04:30:00 +0000 Cc: Subject: [PATCH] pw_update() return success on failure X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: unga888@yahoo.com List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Jul 2008 04:22:41 -0000 --0-1777159550-1215748558=:62863 Content-Type: text/plain; charset=us-ascii Hi all In the present implementation if the pwdb() of /usr/src/usr.sbin/pw/pwupd.c fails, pw_update() (of the same file) still return rc=0, therefore, subsequent functions still continue to proceed without passwd and master.passwd been updated. The attached patch fixes this issue. If this patch is acceptable, appreciate if it could be applied to both current and RELENG_7. Kind regards Unga --0-1777159550-1215748558=:62863 Content-Type: application/octet-stream; name="pw_update-return-code-fix.patch" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="pw_update-return-code-fix.patch" ZGlmZiAtTmF1ciBwdy5vcmlnL3B3dXBkLmMgcHcvcHd1cGQuYwotLS0gcHcu b3JpZy9wd3VwZC5jCTIwMDgtMDctMDcgMTA6Mjg6NTIuMDAwMDAwMDAwICsw ODAwCisrKyBwdy9wd3VwZC5jCTIwMDgtMDctMDcgMTE6Mjc6MzcuMDAwMDAw MDAwICswODAwCkBAIC0xMzksNyArMTM5LDcgQEAKIHN0YXRpYyBpbnQKIHB3 X3VwZGF0ZShzdHJ1Y3QgcGFzc3dkICogcHdkLCBjaGFyIGNvbnN0ICogdXNl ciwgaW50IG1vZGUpCiB7Ci0JaW50ICAgICAgICAgICAgIHJjID0gMDsKKwlp bnQgcmM7CiAKIAlFTkRQV0VOVCgpOwogCkBAIC0xNDgsNyArMTQ4LDggQEAK IAkgKiBOb3RlOiAtQyBpcyBvbmx5IGF2YWlsYWJsZSBpbiBGcmVlQlNEIDIu MiBhbmQgYWJvdmUKIAkgKi8KICNpZmRlZiBIQVZFX1BXREJfQwotCWlmIChw d2RiKCItQyIsIChjaGFyICopTlVMTCkgPT0gMCkgewkvKiBDaGVjayBvbmx5 ICovCisJcmMgPSBwd2RiKCItQyIsIChjaGFyICopTlVMTCk7IC8qIENoZWNr IG9ubHkgKi8KKwlpZiAocmMgPT0gMCkgewogI2Vsc2UKIAl7CQkJCS8qIE5v IC1DICovCiAjZW5kaWYK --0-1777159550-1215748558=:62863--