summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2004-06-24 00:08:15 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 12:56:46 -0500
commitcf5fdae640e3678a68279b728d19a8aefb1b09ec (patch)
treea7eb0d93d60911ba83dd1cf80d002b44697ff818
parentf89a67e3452b0613f659c4ba26c6ed79843c33de (diff)
downloadsamba-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.c9
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;