Date: Sun, 30 Aug 1998 17:06:25 +0200 (CEST) From: Stephane Legrand <stephane@lituus.fr> To: FreeBSD-gnats-submit@FreeBSD.ORG Subject: kern/7781: Problem with setpassent(), getpwnam() and getpwuid() functions in libc Message-ID: <199808301506.RAA11886@sequoia.lituus.fr>
next in thread | raw e-mail | index | archive | help
>Number: 7781 >Category: kern >Synopsis: Problem with setpassent(), getpwnam() and getpwuid() functions in libc >Confidential: no >Severity: serious >Priority: medium >Responsible: freebsd-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: sw-bug >Submitter-Id: current-users >Arrival-Date: Sun Aug 30 08:20:00 PDT 1998 >Last-Modified: >Originator: Stephane Legrand >Organization: >Release: FreeBSD 3.0-CURRENT i386 >Environment: >Description: The manual page for the setpassent(int stayopen) function says : "The setpassent() function accomplishes two purposes. First, it causes getpwent() to ``rewind'' to the beginning of the database. Additionally, if stayopen is non-zero, file descriptors are left open, significantly speeding up subsequent accesses for all of the routines. (This latter functionality is unnecessary for getpwent() as it doesn't close its file descriptors by default.)" The problem is that in the file getpwent.c which implements the setpassent() function in the libc, the stayopen variable seems not used. And then, code like setpassent(1) doesn't work as expected. This problem is particularly annoying with ProFTPD (/usr/ports/net/proftpd) which uses a such code. >How-To-Repeat: >Fix: A patch has been proposed by Adam Mackler (mackler@barter.dewline.com). It adds a "if" to check the value of "_pw_stayopen" BEFORE calling "endpwent()". Adam Mackler writes: > > Date: Thu, 6 Aug 1998 17:50:08 -0400 > > From: Floody <flood@evcom.net> > > Reply-To: proftpd-l@evcom.net > > To: Karl Pielorz <kpielorz@tdx.co.uk> > > Cc: proftpd-l@evcom.net > > Subject: Re: [proftpd-l] New ProFTPd user - Security, Incoming and pwd.db? > > > > Ok. I put up a test FreeBSD 2.2.7 system. There appears to be a libc > > problem with the setpassent() function, which doesn't work on FreeBSD as > > documented in the man pages (or on any other BSD). This is the heart of > > the problem. There is no workaround until libc is fixed. > > Hi: > > I think the following patch may fix the problem, but I'm afraid > I don't know how to rebuild my c library. If you find out if this > works can you let me know? Thanks. > > > *** getpwent.c Wed Aug 19 02:00:13 1998 > --- getpwent.c.dist Wed Aug 19 01:58:33 1998 > *************** > *** 194,201 **** > if (rval && (_pw_passwd.pw_name[0] == '+'|| > _pw_passwd.pw_name[0] == '-')) rval = 0; > > ! if (!_pw_stayopen) > ! endpwent(); > return(rval ? &_pw_passwd : (struct passwd *)NULL); > } > > --- 194,200 ---- > if (rval && (_pw_passwd.pw_name[0] == '+'|| > _pw_passwd.pw_name[0] == '-')) rval = 0; > > ! endpwent(); > return(rval ? &_pw_passwd : (struct passwd *)NULL); > } > > > -- > Adam Mackler > Dewline Communications, LLC > 212-505-9149 > >Audit-Trail: >Unformatted: To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-bugs" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199808301506.RAA11886>