Attachment 12449 Details for Bug 12194 – Patch for v4-5-test

[patch] Patch for v4-5-test

tmp45.diff.txt (text/plain), 1.59 KB, created by Stefan Metzmacher on 2016-09-07 06:54:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Stefan Metzmacher

Created: 2016-09-07 06:54:38 UTC

Size: 1.59 KB

patch

obsolete

>From 7b3ca198f1563264a7d7cfc52cdc532e159078a1 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Bj=C3=B6rn=20Baumbach?= <bb@sernet.de>
>Date: Fri, 26 Aug 2016 17:16:51 +0200
>Subject: [PATCH] idmap_script: add missing "IDTOSID" argument to the script
> command line.
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>According to the documentation the commands should look like
>"IDTOSID UID xxxx" instead of "UID xxxx".
>
>This fixes changes of commit b4239ca096738f553b0f9d7fa6aaa4219b72ef7f:
>    idmap_script: Parallelize script calls
>
>Bug: https://bugzilla.samba.org/show_bug.cgi?id=12194
>
>Signed-off-by: BjÃ¶rn Baumbach <bb@sernet.de>
>Reviewed-by: Stefan Metzmacher <metze@samba.org>
>Reviewed-by: Michael Adam <obnox@samba.org>
>
>Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
>Autobuild-Date(master): Tue Sep  6 19:32:57 CEST 2016 on sn-devel-144
>
>(cherry picked from commit 4f654acad01dbb1b17e933a855bb53ea6a7bb34d)
>---
> source3/winbindd/idmap_script.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/source3/winbindd/idmap_script.c b/source3/winbindd/idmap_script.c
>index f986021..75eabdf 100644
>--- a/source3/winbindd/idmap_script.c
>+++ b/source3/winbindd/idmap_script.c
>@@ -101,7 +101,7 @@ static struct tevent_req *idmap_script_xid2sid_send(
> 		    return tevent_req_post(req, ev);
> 	}
> 
>-	state->syscmd = talloc_asprintf(state, "%s %cID %lu", script, key,
>+	state->syscmd = talloc_asprintf(state, "%s IDTOSID %cID %lu", script, key,
> 					(unsigned long)xid.id);
> 	if (tevent_req_nomem(state->syscmd, req)) {
> 		return tevent_req_post(req, ev);
>-- 
>1.9.1
>

Flags:

obnox: review+
asn: review+

Actions: View

Attachments on bug 12194: 12449