password API needed
Luke Kenneth Casson Leighton
lkcl at switchboard.net
Mon May 18 19:04:48 GMT 1998
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.
More information about the samba-technical
mailing list