]> git.ipfire.org Git - thirdparty/krb5.git/commitdiff
Fix memory leaks in test programs
authorGreg Hudson <ghudson@mit.edu>
Sun, 12 Mar 2017 16:29:50 +0000 (12:29 -0400)
committerGreg Hudson <ghudson@mit.edu>
Mon, 13 Mar 2017 16:54:37 +0000 (12:54 -0400)
Eliminate memory leaks detected by asan in test programs (and
introduced since commit 4947c270032691d556140b290e1b10846b692968), to
make it easier to find more serious leaks.

src/kdc/t_replay.c
src/tests/icinterleave.c

index 87dbbc642031fc1c445a7eef1ae554c3a9295c81..fe49560edad244a04f329bc21e425619c27c3e8f 100644 (file)
@@ -620,6 +620,8 @@ test_kdc_check_lookaside_hit(void **state)
     assert_true(data_eq(rep, *result_data));
     assert_int_equal(hits, 1);
     assert_int_equal(e->num_hits, 1);
+
+    krb5_free_data(context, result_data);
 }
 
 static void
@@ -697,6 +699,8 @@ test_kdc_check_lookaside_hit_multiple(void **state)
     assert_int_equal(e1->num_hits, 1);
     assert_int_equal(e2->num_hits, 0);
 
+    krb5_free_data(context, result_data);
+
     /* Set result_data so we can verify that it is reset to NULL. */
     result_data = &req1;
     result = kdc_check_lookaside(context, &req2, &result_data);
@@ -730,6 +734,8 @@ test_kdc_check_lookaside_hit_hash_collision(void **state)
     assert_int_equal(e1->num_hits, 1);
     assert_int_equal(e2->num_hits, 0);
 
+    krb5_free_data(context, result_data);
+
     /* Set result_data so we can verify that it is reset to NULL. */
     result_data = &req1;
     result = kdc_check_lookaside(context, &req2, &result_data);
index d76ecf361655ac6339e2939414d5b510598d0f72..a1bdd3547f08f75194ce27d891a73429bddb2192 100644 (file)
@@ -119,6 +119,10 @@ main(int argc, char **argv)
         }
     }
 
+    for (i = 0; i < nclients; i++)
+        krb5_free_data_contents(ctx, &reps[i]);
+    free(reps);
+    free(iccs);
     krb5_free_context(ctx);
     return 0;
 }