[SCM] Samba Shared Repository - branch master updated
Jeremy Allison
jra at samba.org
Wed Oct 14 12:30:04 MDT 2009
The branch, master has been updated
via ce4542f... Final part of fix for bug 6793 - winbindd crash with "INTERNAL ERROR: Signal 6" Don't use mapped_user uninitialized. Jeremy.
via c6fc461... s3:winbind: Fix a double-free
via db29d3e... s3:winbind: Fix bug 6793 -- segfault in winbindd_pam_auth
from 67b544b... s3-build: we need to have talloc 2.0.1 when building with external talloc.
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master
- Log -----------------------------------------------------------------
commit ce4542fbdeeffb20f1f93105974863aa42981e53
Author: Jeremy Allison <jra at samba.org>
Date: Wed Oct 14 11:11:26 2009 -0700
Final part of fix for bug 6793 - winbindd crash with "INTERNAL ERROR: Signal 6"
Don't use mapped_user uninitialized.
Jeremy.
commit c6fc461e71cd1837a58ad5dcad3d3f952390e5cf
Author: Volker Lendecke <vl at samba.org>
Date: Tue Oct 13 20:56:28 2009 +0200
s3:winbind: Fix a double-free
Part of a fix for bug #6793.
commit db29d3eb400bee78347963c8ccd057f564383e36
Author: Volker Lendecke <vl at samba.org>
Date: Wed Oct 14 11:14:57 2009 -0700
s3:winbind: Fix bug 6793 -- segfault in winbindd_pam_auth
-----------------------------------------------------------------------
Summary of changes:
source3/winbindd/idmap_adex/provider_unified.c | 1 -
source3/winbindd/winbindd_pam.c | 15 ++++++++-------
2 files changed, 8 insertions(+), 8 deletions(-)
Changeset truncated at 500 lines:
diff --git a/source3/winbindd/idmap_adex/provider_unified.c b/source3/winbindd/idmap_adex/provider_unified.c
index b6652f4..af33405 100644
--- a/source3/winbindd/idmap_adex/provider_unified.c
+++ b/source3/winbindd/idmap_adex/provider_unified.c
@@ -440,7 +440,6 @@ done:
}
talloc_destroy(frame);
- TALLOC_FREE(entry_dn);
return nt_status;
}
diff --git a/source3/winbindd/winbindd_pam.c b/source3/winbindd/winbindd_pam.c
index edbaa55..c9fd227 100644
--- a/source3/winbindd/winbindd_pam.c
+++ b/source3/winbindd/winbindd_pam.c
@@ -797,8 +797,8 @@ NTSTATUS append_auth_data(struct winbindd_cli_state *state,
void winbindd_pam_auth(struct winbindd_cli_state *state)
{
struct winbindd_domain *domain;
- fstring name_domain, name_user;
- char *mapped_user = NULL;
+ fstring name_domain, name_user, mapped_user;
+ char *mapped = NULL;
NTSTATUS result;
NTSTATUS name_map_status = NT_STATUS_UNSUCCESSFUL;
@@ -822,15 +822,16 @@ void winbindd_pam_auth(struct winbindd_cli_state *state)
name_map_status = normalize_name_unmap(state->mem_ctx,
state->request->data.auth.user,
- &mapped_user);
+ &mapped);
/* If the name normalization didnt' actually do anything,
just use the original name */
- if (!NT_STATUS_IS_OK(name_map_status) &&
- !NT_STATUS_EQUAL(name_map_status, NT_STATUS_FILE_RENAMED))
- {
- mapped_user = state->request->data.auth.user;
+ if (NT_STATUS_IS_OK(name_map_status)
+ ||NT_STATUS_EQUAL(name_map_status, NT_STATUS_FILE_RENAMED)) {
+ fstrcpy(mapped_user, mapped);
+ } else {
+ fstrcpy(mapped_user, state->request.data.auth.user);
}
if (!canonicalize_username(mapped_user, name_domain, name_user)) {
--
Samba Shared Repository
More information about the samba-cvs
mailing list