[PATCH] This fixes a regression with 'net ads' and kerberos method set to create keytab
Jeremy Allison
jra at samba.org
Tue Mar 14 16:52:08 UTC 2017
On Tue, Mar 14, 2017 at 06:49:10AM +1300, Andrew Bartlett via samba-technical wrote:
> On Mon, 2017-03-13 at 18:13 +0100, Andreas Schneider via samba-
> technical wrote:
> > Hello,
> >
> > I've opened
> >
> > https://bugzilla.redhat.com/show_bug.cgi?id=1430755
> >
> > which our QA found. Attached is the patch to address the issue and
> > I've addedÂ
> > a testcase so this will not happen in future.
> >
> >
> > Review and push appreciated!
>
> The only patch I'm cautious on is the subunit.sh patch. Can you
> explain it a bit more? I'm hesitant because so much other finely
> balanced code rests on that macro...
>
> Other than that: Reviewed-by: Andrew Bartlett <abartlet at samba.org>
The subunit.sh one is correct. The difference between "$*" and "$@"
is explained here:
https://stackoverflow.com/questions/2761723/what-is-the-difference-between-and-in-shell-scripts
essentially - with "$@" the arguments: 1 2 '3 4'
get correctly expanded as:
1
2
3 4
with "$*", the last argument gets incorrectly split as it
contains spaces.
1
2
3
4
Reviewed-by: Jeremy Allison <jra at samba.org>
More information about the samba-technical
mailing list