diff options
author | Stefan Metzmacher <metze@samba.org> | 2004-06-24 00:08:15 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:56:46 -0500 |
commit | cf5fdae640e3678a68279b728d19a8aefb1b09ec (patch) | |
tree | a7eb0d93d60911ba83dd1cf80d002b44697ff818 | |
parent | f89a67e3452b0613f659c4ba26c6ed79843c33de (diff) | |
download | samba-cf5fdae640e3678a68279b728d19a8aefb1b09ec.tar.gz samba-cf5fdae640e3678a68279b728d19a8aefb1b09ec.tar.bz2 samba-cf5fdae640e3678a68279b728d19a8aefb1b09ec.zip |
r1234: valgrind found this uninitialised var...
But I don't know how to fix this correct,
so maybe this needs to be fixed
(tridge: can you please look at this)
metze
(This used to be commit b8b4d0d5bf037c79102709ea995ad8b8d6a9caff)
-rw-r--r-- | source4/rpc_server/netlogon/dcerpc_netlogon.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/source4/rpc_server/netlogon/dcerpc_netlogon.c b/source4/rpc_server/netlogon/dcerpc_netlogon.c index c3b1bfc516..ee3970d0f2 100644 --- a/source4/rpc_server/netlogon/dcerpc_netlogon.c +++ b/source4/rpc_server/netlogon/dcerpc_netlogon.c @@ -294,7 +294,14 @@ static NTSTATUS netr_ServerAuthenticate(struct dcesrv_call_state *dce_call, TALL struct netr_ServerAuthenticate *r) { struct netr_ServerAuthenticate3 r3; - uint32 negotiate_flags, rid; + uint32_t rid; + /* TODO: + * negotiate_flags is used as an [in] parameter + * so it need to be initialised. + * + * (I think ... = 0; seems wrong here --metze) + */ + uint32 negotiate_flags = 0; r3.in.server_name = r->in.server_name; r3.in.account_name = r->in.account_name; |