diff options
| author | Michael Adam <obnox@samba.org> | 2009-02-02 00:35:43 +0100 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2009-02-02 00:53:04 +0100 |
| commit | 8c57f34b99b9bd587e1c2be0905a8c41eebbd568 (patch) | |
| tree | 761587a06af871c438e79ffff1a10e3d7603c40a /source3/winbindd | |
| parent | d936be23cd9d04adc2dca27cfc7772b53383f298 (diff) | |
| download | samba-8c57f34b99b9bd587e1c2be0905a8c41eebbd568.tar.gz samba-8c57f34b99b9bd587e1c2be0905a8c41eebbd568.tar.bz2 samba-8c57f34b99b9bd587e1c2be0905a8c41eebbd568.zip | |
s3:winbind_user: move initialization of domain up in winbindd_fill_pwent()
and streamline logic some
Michael
Diffstat (limited to 'source3/winbindd')
| -rw-r--r-- | source3/winbindd/winbindd_user.c | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/source3/winbindd/winbindd_user.c b/source3/winbindd/winbindd_user.c index 5356e16a74..2e326430fc 100644 --- a/source3/winbindd/winbindd_user.c +++ b/source3/winbindd/winbindd_user.c @@ -80,6 +80,13 @@ static bool winbindd_fill_pwent(TALLOC_CTX *ctx, char *dom_name, char *user_name if (!pw || !dom_name || !user_name) return False; + domain = find_domain_from_name_noinit(dom_name); + if (domain == NULL) { + DEBUG(5,("winbindd_fill_pwent: Failed to find domain for %s. " + "Disabling name alias support\n", dom_name)); + nt_status = NT_STATUS_NO_SUCH_DOMAIN; + } + /* Resolve the uid number */ if (!NT_STATUS_IS_OK(idmap_sid_to_uid(dom_name, user_sid, @@ -98,19 +105,10 @@ static bool winbindd_fill_pwent(TALLOC_CTX *ctx, char *dom_name, char *user_name return False; } - strlower_m(user_name); - /* Username */ - domain = find_domain_from_name_noinit(dom_name); - if (domain) { - nt_status = normalize_name_map(ctx, domain, user_name, - &mapped_name); - } else { - DEBUG(5,("winbindd_fill_pwent: Failed to find domain for %s. " - "Disabling name alias support\n", dom_name)); - nt_status = NT_STATUS_NO_SUCH_DOMAIN; - } + strlower_m(user_name); + nt_status = normalize_name_map(ctx, domain, user_name, &mapped_name); /* Basic removal of whitespace */ if (NT_STATUS_IS_OK(nt_status)) { |