Attachment 8443 Details for Bug 9570 – git formated patch for master

[patch] git formated patch for master

0001-passdb-fix-NT_STATUS_NO_SUCH_GROUP.patch (text/plain), 1.50 KB, created by Arvid Requate on 2013-01-17 16:11:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Arvid Requate

Created: 2013-01-17 16:11:51 UTC

Size: 1.50 KB

patch

obsolete

>From 52fef2bcf261e9aa79a4cd82541eefd147584593 Mon Sep 17 00:00:00 2001
>From: Arvid Requate <requate@univention.de>
>Date: Thu, 17 Jan 2013 16:44:28 +0100
>Subject: [PATCH] passdb: fix NT_STATUS_NO_SUCH_GROUP
>
>Share options like "force group" and "valid users = @group1"
>triggered a NT_STATUS_NO_SUCH_GROUP. While the group was found in
>the SAM backend, its objectclass was not retrived.
>
>This fix also revealed a talloc access after free in the group
>branch of pdb_samba_dsdb_getgrfilter.
>---
> source3/passdb/pdb_samba_dsdb.c |    4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/source3/passdb/pdb_samba_dsdb.c b/source3/passdb/pdb_samba_dsdb.c
>index 0ff2e0a..2e61e0d 100644
>--- a/source3/passdb/pdb_samba_dsdb.c
>+++ b/source3/passdb/pdb_samba_dsdb.c
>@@ -861,7 +861,7 @@ static NTSTATUS pdb_samba_dsdb_getgrfilter(struct pdb_methods *m, GROUP_MAP *map
> {
> 	struct pdb_samba_dsdb_state *state = talloc_get_type_abort(
> 		m->private_data, struct pdb_samba_dsdb_state);
>-	const char *attrs[] = { "objectSid", "description", "samAccountName", "groupType",
>+	const char *attrs[] = { "objectClass", "objectSid", "description", "samAccountName", "groupType",
> 				NULL };
> 	struct ldb_message *msg;
> 	va_list ap;
>@@ -928,7 +928,7 @@ static NTSTATUS pdb_samba_dsdb_getgrfilter(struct pdb_methods *m, GROUP_MAP *map
> 		id_maps[1] = NULL;
> 
> 		status = idmap_sids_to_xids(state->idmap_ctx, tmp_ctx, id_maps);
>-		talloc_free(tmp_ctx);
>+
> 		if (!NT_STATUS_IS_OK(status)) {
> 			talloc_free(tmp_ctx);
> 			return status;
>-- 
>1.7.10.4
>

Flags:

metze: review+

Actions: View

Attachments on bug 9570: 8443 | 10231