summaryrefslogtreecommitdiffstats
path: root/auth/gensec
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2012-02-10 08:07:21 +1100
committerAndrew Bartlett <abartlet@samba.org>2012-02-10 20:49:24 +1100
commit901e3b7246de9bdc07e2b3d88f55917bf2a37377 (patch)
treeffe385955e25d5f9f14c4845316e429c4b0bc7b8 /auth/gensec
parent919440ff0a8ca05a394c316405e1dfb869ffd488 (diff)
downloadsamba-901e3b7246de9bdc07e2b3d88f55917bf2a37377.tar.gz
samba-901e3b7246de9bdc07e2b3d88f55917bf2a37377.tar.xz
samba-901e3b7246de9bdc07e2b3d88f55917bf2a37377.zip
Revert "gensec: Fix a memory corruption in gensec_use_kerberos_mechs"
This reverts commit 744ed53a62037a659133ccd4de2065491208ae7d. The real bug here is that the second half of the outer loop should not have been run once we found spnego. Andrew Bartlett
Diffstat (limited to 'auth/gensec')
-rw-r--r--auth/gensec/gensec_start.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/auth/gensec/gensec_start.c b/auth/gensec/gensec_start.c
index ab092a7bb17..08b2fb68cb1 100644
--- a/auth/gensec/gensec_start.c
+++ b/auth/gensec/gensec_start.c
@@ -75,8 +75,7 @@ _PUBLIC_ struct gensec_security_ops **gensec_use_kerberos_mechs(TALLOC_CTX *mem_
/* noop */
}
- new_gensec_list = talloc_array(mem_ctx, struct gensec_security_ops *,
- num_mechs_in*2 + 1);
+ new_gensec_list = talloc_array(mem_ctx, struct gensec_security_ops *, num_mechs_in + 1);
if (!new_gensec_list) {
return NULL;
}