talloc: ASAN fix for test_rusty
authorSwen Schillig <swen@linux.ibm.com>
Thu, 15 Aug 2019 12:39:58 +0000 (14:39 +0200)
committerAndrew Bartlett <abartlet@samba.org>
Wed, 21 Aug 2019 07:20:39 +0000 (07:20 +0000)
Direct leak of 100 byte(s) in 1 object(s) allocated from:
     #0 0x7f06393dfc08 in __interceptor_malloc (/lib64/libasan.so.5+0xefc08)
     #1 0x7f06392d1af3 in __talloc_with_prefix ../../talloc.c:782
     #2 0x7f06392d1af3 in __talloc ../../talloc.c:824
     #3 0x7f06392d1af3 in __talloc_strlendup ../../talloc.c:2455
     #4 0x7f06392d1af3 in talloc_strdup ../../talloc.c:2471
     #5 0x40b4f0 in test_rusty ../../testsuite.c:1543
     #6 0x40b4f0 in torture_local_talloc ../../testsuite.c:2146
     #7 0x402603 in main ../../testsuite_main.c:32
     #8 0x7f063908a412 in __libc_start_main (/lib64/libc.so.6+0x24412)

Signed-off-by: Swen Schillig <swen@linux.ibm.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Matthias Dieter Wallnöfer <mdw@samba.org>
lib/talloc/testsuite.c

index d936d18c2ae434686a29e58bff308afc1bc51133..ffede68f52a1d3ffb535ea7bab2825483ec2b470 100644 (file)
@@ -1540,7 +1540,7 @@ static bool test_free_ref_null_context(void)
 static bool test_rusty(void)
 {
        void *root;
-       const char *p1;
+       char *p1;
 
        talloc_enable_null_tracking();
        root = talloc_new(NULL);
@@ -1549,6 +1549,8 @@ static bool test_rusty(void)
        talloc_report_full(root, stdout);
        talloc_free(root);
        CHECK_BLOCKS("null_context", NULL, 2);
+       talloc_free(p1); /* make ASAN happy */
+
        return true;
 }