]> git.ipfire.org Git - thirdparty/krb5.git/commitdiff
Fix kdb_ldap_create_principal cleanup
authorGreg Hudson <ghudson@mit.edu>
Thu, 28 Mar 2013 19:59:07 +0000 (15:59 -0400)
committerGreg Hudson <ghudson@mit.edu>
Thu, 28 Mar 2013 19:59:07 +0000 (15:59 -0400)
entry must be initialized before all code which can jump to cleanup.

src/plugins/kdb/ldap/ldap_util/kdb5_ldap_realm.c

index 1050fcd73ed5d84facf7e74e26a0d446b61064df..7494928647867a9b4bb790072137841520bf5662 100644 (file)
@@ -1344,6 +1344,8 @@ kdb_ldap_create_principal(krb5_context context, krb5_principal princ,
     krb5_timestamp now;
     krb5_actkvno_node     actkvno;
 
+    memset(&entry, 0, sizeof(entry));
+
     if ((pblock == NULL) || (context == NULL)) {
         retval = EINVAL;
         goto cleanup;
@@ -1355,8 +1357,6 @@ kdb_ldap_create_principal(krb5_context context, krb5_principal princ,
         goto cleanup;
     }
 
-    memset(&entry, 0, sizeof(entry));
-
     tl_data = malloc(sizeof(*tl_data));
     if (tl_data == NULL) {
         retval = ENOMEM;