Re: password API needed

Luke Kenneth Casson Leighton (lkcl@switchboard.net)
Mon, 18 May 1998 19:04:48 +0000 (GMT)

Date:	Mon, 18 May 1998 19:04:48 +0000 (GMT)
From:	Luke Kenneth Casson Leighton <lkcl@switchboard.net>
To:	Jeremy Allison <jallison@whistle.com>
Subject: Re: password API needed
In-Reply-To: <3560845E.FF6D5DF@whistle.com>

On Mon, 18 May 1998, Jeremy Allison wrote:

> Luke,
>
> I am looking through what you have done in
> passdb.c etc. You have confused functions that return
> smb_passwd entries and funtions that return sam_passwd
> entries.
>
> As an example :
>
> You now have
>
> getsampwnam and getsampwuid returning a struct smb_passwd.

yes, jean-francois has pointed this out, too.

> This makes no sense - if they return a smb_passwd
> they should be called getsmbpwnam and getsmbpwuid (as they
> were originally).
>
> You then have getsam21pwnam returning a struct sam_passwd.
> This is ok.
>
> You need to make explicit in the function name
> what it will return.
>
> This means that (to stick to your current naming
> structure) anything returning a sam_passwd struct
> should be called get/setsam21XXXX(), to make it
> clear it's returning something different from a
> smb_passwd.
>
> All functions returning an smb_passwd must be
> called get/setsmbXXXXX.
>
> I will make these changes, as it will clarify a
> great deal of code.

such a name change will conflict with those functions so already named in
smbpass.c, which is why i haven't done it.