Revert "gensec: Fix a memory corruption in gensec_use_kerberos_mechs"
authorAndrew Bartlett <abartlet@samba.org>
Thu, 9 Feb 2012 21:07:21 +0000 (08:07 +1100)
committerAndrew Bartlett <abartlet@samba.org>
Fri, 10 Feb 2012 09:49:24 +0000 (20:49 +1100)
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

auth/gensec/gensec_start.c

index ab092a7bb17b5355fbf34b90a175bb6aa6071f54..08b2fb68cb14b01e64e6a81c5480a3ea8acdba05 100644 (file)
@@ -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;
        }