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 | |
| parent | 919440ff0a8ca05a394c316405e1dfb869ffd488 (diff) | |
| download | samba-901e3b7246de9bdc07e2b3d88f55917bf2a37377.tar.gz samba-901e3b7246de9bdc07e2b3d88f55917bf2a37377.tar.bz2 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
| -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 ab092a7bb1..08b2fb68cb 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; } |