summaryrefslogtreecommitdiff
path: root/source3/lib
diff options
context:
space:
mode:
authorLuke Leighton <lkcl@samba.org>1999-07-06 22:08:55 +0000
committerLuke Leighton <lkcl@samba.org>1999-07-06 22:08:55 +0000
commita0b0fa7bb1eb83ff26931831fa1006000d29e861 (patch)
tree280d672fe08af37d41d4ec3766bd935f321fdd58 /source3/lib
parent7be65db6400304a9ff94ebaa19d312a3d8481834 (diff)
downloadsamba-a0b0fa7bb1eb83ff26931831fa1006000d29e861.tar.gz
samba-a0b0fa7bb1eb83ff26931831fa1006000d29e861.tar.bz2
samba-a0b0fa7bb1eb83ff26931831fa1006000d29e861.zip
using jeremy's sys_getpwnam() call in the more critical area: Get_Pwnam().
made sure that hashed_getpwnam() has the copy-passwd-struct-wrapper around it, too. TODO: replace all calls of getpwnam() with sys_getpwnam(). (This used to be commit 436a89145524d3539b3a247f98c1e71f0616dd70)
Diffstat (limited to 'source3/lib')
-rw-r--r--source3/lib/system.c6
-rw-r--r--source3/lib/username.c128
2 files changed, 20 insertions, 114 deletions
diff --git a/source3/lib/system.c b/source3/lib/system.c
index 7bb64ab723..0bd1fea140 100644
--- a/source3/lib/system.c
+++ b/source3/lib/system.c
@@ -546,7 +546,7 @@ int sys_getgroups(int setlen, gid_t *gidset)
Helper function for getpwnam/getpwuid wrappers.
****************************************************************************/
-static struct passwd *setup_pwret(struct passwd *pass)
+struct passwd *copy_passwd_struct(struct passwd *pass)
{
static pstring pw_name;
static pstring pw_passwd;
@@ -582,7 +582,7 @@ static struct passwd *setup_pwret(struct passwd *pass)
struct passwd *sys_getpwnam(const char *name)
{
- return setup_pwret(getpwnam(name));
+ return copy_passwd_struct(getpwnam(name));
}
/**************************************************************************
@@ -591,5 +591,5 @@ struct passwd *sys_getpwnam(const char *name)
struct passwd *sys_getpwuid(uid_t uid)
{
- return setup_pwret(getpwuid(uid));
+ return copy_passwd_struct(getpwuid(uid));
}
diff --git a/source3/lib/username.c b/source3/lib/username.c
index cca38dbc26..87fe39466f 100644
--- a/source3/lib/username.c
+++ b/source3/lib/username.c
@@ -228,14 +228,12 @@ static BOOL have_passwd_hash(void)
struct passwd *hashed_getpwnam(const char *name)
{
-#ifndef USE_HASHED_GETPWNAM
- return getpwnam(name);
-#else
struct passwd_hash_table_s *pht=&passwd_hash_table;
DEBUG(5,("getpwnam(%s)\n", name));
- if (have_passwd_hash()) {
+ if (have_passwd_hash())
+ {
int name_i=name_hash_function(name);
int hash_index=pht->names[name_i];
while(hash_index!=-1) {
@@ -249,7 +247,7 @@ struct passwd *hashed_getpwnam(const char *name)
pass->pw_gecos,
pass->pw_dir,
pass->pw_shell));
- return pass;
+ return copy_passwd_struct(pass);
}
hash_index=pht->entries[hash_index].next;
}
@@ -258,9 +256,9 @@ struct passwd *hashed_getpwnam(const char *name)
DEBUG(5,("%s not found\n",name));
return NULL;
}
+
/* Fall back to real getpwnam() */
- return getpwnam(name);
-#endif
+ return sys_getpwnam(name);
}
/*******************************************************************
@@ -422,122 +420,30 @@ BOOL map_username(char *user)
return mapped_user;
}
+
/****************************************************************************
-Get_Pwnam wrapper
+ Get_Pwnam wrapper
****************************************************************************/
static struct passwd *_Get_Pwnam(char *s)
{
- static struct passwd mypasswd;
- static char *mypwdp = NULL;
- static char *myname = NULL;
- struct passwd *ret;
-
- if (mypwdp)
- {
- free(mypwdp);
- mypwdp = NULL;
- }
- if (myname)
- {
- free(myname);
- myname = NULL;
- }
- ret = hashed_getpwnam(s);
- if (ret)
- {
- memcpy(&mypasswd, ret, sizeof(mypasswd));
- ret = &mypasswd;
- /* Deal with password information stored in shadows. Due to the
- dynamic allocation of password cache stuff, the original password
- needs to be freed and the new password mallocated to avoid
- crashing the cache destructor code. */
-
-#ifdef HAVE_GETSPNAM
- {
- struct spwd *spass;
-
- /* many shadow systems require you to be root to get
- the password, in most cases this should already be
- the case when this function is called, except
- perhaps for IPC password changing requests */
-
- spass = getspnam(ret->pw_name);
- if (spass && spass->sp_pwdp) {
- ret->pw_passwd = mypwdp = strdup(spass->sp_pwdp);
- }
- }
-#elif defined(IA_UINFO)
- {
- /* Need to get password with SVR4.2's ia_ functions
- instead of get{sp,pw}ent functions. Required by
- UnixWare 2.x, tested on version
- 2.1. (tangent@cyberport.com) */
- /* Not sure how large the new password string should
- be so I'm using a pstring instead. If anyone has
- access to a UnixWare system perhaps they could
- optimise this. (tpot@samba.org) */
- uinfo_t uinfo;
- if (ia_openinfo(ret->pw_name, &uinfo) != -1) {
- ret->pw_passwd = mypwdp = malloc(FSTRING_LEN);
- ia_get_logpwd(uinfo, &(ret->pw_passwd));
- }
- }
-#endif
-
-#ifdef HAVE_GETPRPWNAM
- {
- struct pr_passwd *pr_pw = getprpwnam(ret->pw_name);
- if (pr_pw && pr_pw->ufld.fd_encrypt) {
- ret->pw_passwd = mypwdp = strdup(pr_pw->ufld.fd_encrypt);
- }
- }
-#endif
-
-#ifdef OSF1_ENH_SEC
- {
- struct pr_passwd *mypasswd;
- DEBUG(5,("Checking password for user %s in OSF1_ENH_SEC\n",
- user));
- mypasswd = getprpwnam (user);
- if (mypasswd) {
- ret->pw_name = myname = strdup(mypasswd->ufld.fd_name);
- ret->pw_passwd = mypwdp = strdup(mypasswd->ufld.fd_encrypt);
- } else {
- DEBUG(5,("No entry for user %s in protected database !\n",
- user));
- return(False);
- }
- }
-#endif
-
-#ifdef ULTRIX_AUTH
- {
- AUTHORIZATION *ap = getauthuid(ret->pw_uid);
- if (ap) {
- ret->pw_passwd = mypwdp = strdup(ap->a_password);
- endauthent();
- }
- }
-#endif
+ struct passwd *ret;
+ ret = hashed_getpwnam(s);
#ifdef HAVE_GETPWANAM
- struct passwd_adjunct *pwret;
- pwret = getpwanam(s);
- if (pwret)
+ if (ret)
{
-/* ????????? I don't know what kind of ptr pwret->pwa_passwd is ????????? */
-/* ????????? mayby the code should look like the next line ????????? */
-/* ????????? ret->pw_passwd = mypwdp = strdup(pwret->pwa_passwd); ????????? */
- ret->pw_passwd = pwret->pwa_passwd;
+ struct passwd_adjunct *pwret;
+ pwret = getpwanam(s);
+ if (pwret != NULL && pwret->pwa_passwd != NULL)
+ {
+ pstrcpy(ret->pw_passwd, pwret->pwa_passwd);
+ }
}
#endif
- }
-
- return ret;
+ return ret;
}
-
/****************************************************************************
a wrapper for getpwnam() that tries with all lower and all upper case
if the initial name fails. Also tried with first letter capitalised