[PATCH] libads: abstract out SASL wrapping code

[Andreas Schneider]

On Monday, 15 May 2017 20:47:13 CEST Alexander Bokovoy via samba-technical 
wrote:
> On ti, 16 touko 2017, Andrew Bartlett wrote:
> > On Mon, 2017-05-15 at 17:28 +0300, Alexander Bokovoy via samba-
> > 
> > technical wrote:
> > > > Notice NT_STATUS_CONNECTION_DISCONNECT message garbled?
> > > > 
> > > > I'm still trying to understand what broke -- on client side we seem
> > > > never get back (my debug statements never get printed) after
> > > > successful
> > > > SASL GSS-SPNEGO bind.
> > > 
> > > Attached patch passes samba.tests.pam_winbind test.
> > 
> > Thanks for all your hard work on this.  What was the problem in the
> > LDAP server in the end?
> 
> I haven't found that yet as I fixed one wrong pointer pass in my code
> and went away with that. However, if you look at the output in my
> previous email, it looks like the error string is garbled at the end, so
> there is something off-by-one somewhere.
> 
> I also get crash in GUID_buf_string() because I run 'make test' with
> high enough log level and source3/locking/share_mode_lock.c:846 causes
> an NDR print out of a share mode lock entry for log level 11 or above.
> As result, there seem to be a garbled GUID.
> 
> The test is samba3.smbtorture_s3.crypt_client(nt4_dc).TORTURE(nt4_dc).
> 
> 
> #5  0x00007fb4ab130aa1 in sig_fault (sig=11) at ../lib/util/fault.c:94
> No locals.
> #6  <signal handler called>
> No symbol table info available.
> #7  0x00007fb4a3b5e174 in GUID_buf_string (guid=0x55d21b7dd178,
> dst=0x7ffcd1993960) at ../librpc/ndr/uuid.c:335 No locals.
> #8  0x00007fb4a3b5e122 in GUID_string (mem_ctx=0x559a1b884160,
> guid=0x55d21b7dd178) at ../librpc/ndr/uuid.c:314 buf = {buf =
> "`\324m\247\264\177\000\000`W\210\033\232U\000\000ma touko 15 20.59.13 20"}
> #9  0x00007fb4a3b623ef in ndr_print_GUID (ndr=0x559a1b884160,
> name=0x7fb4aa0e80b4 "client_guid", guid=0x55d21b7dd178) at
> ../librpc/ndr/ndr_misc.c:29 No locals.
> #10 0x00007fb4aa09272f in ndr_print_share_mode_lease (ndr=0x559a1b884160,
> name=0x7fb4aa0e82fb "lease", r=0x55d21b7dd178) at
> default/source3/librpc/gen_ndr/ndr_open_files.c:69 _flags_save_STRUCT = 0
> #11 0x00007fb4aa09331c in ndr_print_share_mode_entry (ndr=0x559a1b884160,
> name=0x7fb4aa0e8c56 "share_modes", r=0x559a1b830270) at
> default/source3/librpc/gen_ndr/ndr_open_files.c:176 No locals.
> #12 0x00007fb4aa0960dd in ndr_print_share_mode_data (ndr=0x559a1b884160,
> name=0x7fb4aaea411d "d", r=0x559a1b858e60) at
> default/source3/librpc/gen_ndr/ndr_open_files.c:527 cntr_share_modes_0 = 0
>         cntr_leases_0 = 0
>         cntr_delete_tokens_0 = 0
> #13 0x00007fb4a3b5f2fe in ndr_print_debug (fn=0x7fb4aa095eb4
> <ndr_print_share_mode_data>, name=0x7fb4aaea411d "d", ptr=0x559a1b858e60)
> at ../librpc/ndr/ndr.c:420 ndr = 0x559a1b884160
>         __FUNCTION__ = "ndr_print_debug"
> #14 0x00007fb4aad8c995 in share_mode_traverse_fn (rec=0x7ffcd1993ba0,
> _state=0x7ffcd1993ec0) at ../source3/locking/share_mode_lock.c:846 state =
> 0x7ffcd1993ec0
>         i = 1
>         key = {dptr = 0x559a1b865ff0 <incomplete sequence \375>, dsize = 24}
> value = {dptr = 0x559a1b86600c "\324\f)M\244\333o\301", dsize = 348} blob =
> {data = 0x559a1b86600c "\324\f)M\244\333o\301", length = 348} ndr_err =
> NDR_ERR_SUCCESS
>         d = 0x559a1b858e60
>         fid = {devid = 64770, inode = 11919960, extid = 0}
>         ret = 32764
>         __FUNCTION__ = "share_mode_traverse_fn"

We traverse the database here and we do have a lease without a client_guid. I 
do not find the place where this database entry is written. So we need to find 
the place which doesn't add the client_guid.




-- 
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             asn at samba.org
www.samba.org