[PR PATCH] [Updated] samba-tool domain trust: fix trust compatibility to Windows Server 1709 and FreeIPA

[Alexander Bokovoy]

On ti, 27 helmi 2018, Stefan Metzmacher wrote:
> Hi Alexander,
> 
> > I've created https://bugzilla.samba.org/show_bug.cgi?id=13308 for this
> > and will review it now.
> >
> > Am 26.02.2018 um 14:39 schrieb Alexander Bokovoy via samba-technical:
> >> On ma, 26 helmi 2018, Github bot account via samba-technical wrote:
> >>> There is an updated pull request by abbra against master on the Samba Samba Github repository
> >>>
> >>> https://github.com/abbra/samba smb2-trust
> >>> https://github.com/samba-team/samba/pull/134
> 
> >>> Submitting via github to get a test build run.
> >> Travis CI build succeeded, so please review.
> 
> The Travis CI build doesn't run much tests, so this was completely
> untested.
> 
> I'll push it with the attached modification:
You picked older version.

These are the changes that are in the pull request already. I pushed
them second time after Travis CI actually failed.

> diff --git a/python/samba/netcmd/domain.py b/python/samba/netcmd/domain.py
> index fd6170e..4ce97e5 100644
> --- a/python/samba/netcmd/domain.py
> +++ b/python/samba/netcmd/domain.py
> @@ -1733,14 +1733,14 @@ class DomainTrustCommand(Command):
>          self.local_creds = local_creds
>          return self.local_server
> 
> -    def _get_connection(self, connect_func, lp, creds, bindings)
> +    def _get_connection(self, connect_func, lp, creds, bindings):
>          """ _get_connection: try to generate a valid connection
>                               using multiple bindings to allow
>                               protocol fallbacks
>          """
>          bnd = None
>          last_exc = None
> -        for binding in self.local_bindings:
> +        for binding in bindings:
>              try:
>                  bnd = connect_func(binding, lp, creds)
>                  if bnd is not None:
> 
> 
> metze
> 
> 
> 
> 
> 




-- 
/ Alexander Bokovoy