Segfault in smbtorture if [/pipe/lsarpc] in binding string

[Stefan (metze) Metzmacher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Andrew Bartlett schrieb:
| While chasing down a problem for VL, I found that we have an
| uninitialised value in binding->authservice if the pipe is specified.
|
| It appears that we only fill in binding->authservice if we need to find
| the default pipe by looking up the IDL.
|
| What is the correct place to hook in this automatic lookup, and should
| we allow the service to be configured in the binding string somehow?
|
| ==12626== Conditional jump or move depends on uninitialised value(s)
| ==12626==    at 0x823F0CE: dcerpc_bind_auth_password (dcerpc_auth.c:193)
| ==12626==    by 0x824110C: dcerpc_pipe_auth (dcerpc_util.c:907)
| ==12626==    by 0x824145F: dcerpc_pipe_connect_ncacn_np
| (dcerpc_util.c:999)
| ==12626==    by 0x8241A00: dcerpc_pipe_connect_b (dcerpc_util.c:1183)
| ==12626==    by 0x8241B5A: dcerpc_pipe_connect (dcerpc_util.c:1231)
| ==12626==    by 0x805BCAE: torture_rpc_connection (torture.c:140)
| ==12626==    by 0x82A847B: torture_rpc_lsa (lsa.c:1615)
| ==12626==    by 0x806201D: run_test (torture.c:2472)
| ==12626==    by 0x80634F0: main (torture.c:2751)

First we should just initialize it to NULL, as the gensec backend will use "host" by default then

Jelmer,
can you maybe look for a place where this could be configured.



- --
metze

Stefan Metzmacher <metze at samba.org> www.samba.org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3-nr1 (Windows XP)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFCMqVqm70gjA5TCD8RAuQuAJ9FmluL+SXfL3YgmB7Wn5MCk111CgCeN3mS
ZPZlcMtEUDl8zF3TJOUGt9Y=
=lqxO
-----END PGP SIGNATURE-----