idmap_script: add missing "IDTOSID" argument to the script command line.
authorBjörn Baumbach <bb@sernet.de>
Fri, 26 Aug 2016 15:16:51 +0000 (17:16 +0200)
committerStefan Metzmacher <metze@samba.org>
Tue, 6 Sep 2016 17:32:57 +0000 (19:32 +0200)
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

source3/winbindd/idmap_script.c

index f98602195749bdc4d937234126d2b0c05eaec549..75eabdf1c007933e73206401b7bf480a09432edf 100644 (file)
@@ -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);