diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2012-02-10 08:07:21 +1100 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2012-02-10 20:49:24 +1100 |
| commit | 901e3b7246de9bdc07e2b3d88f55917bf2a37377 (patch) | |
| tree | ffe385955e25d5f9f14c4845316e429c4b0bc7b8 /auth | |
| parent | 919440ff0a8ca05a394c316405e1dfb869ffd488 (diff) | |
| download | samba-901e3b7246de9bdc07e2b3d88f55917bf2a37377.tar.gz | |
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')
| -rw-r--r-- | auth/gensec/gensec_start.c | 3 |
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; } |