Re: [PATCH for-9.2 6/9] tests/unit/crypto-tls-x509-helpers: deinit privkey in test_tls_cleanup

[Fabiano Rosas]

Peter Maydell <peter.maydell@linaro.org> writes:

> We create a gnutls_x509_privkey_t in test_tls_init(), but forget
> to deinit it in test_tls_cleanup(), resulting in leaks
> reported in hte migration test such as:
>
> Indirect leak of 8 byte(s) in 1 object(s) allocated from:
>     #0 0x55fa6d11c12e in malloc 
> (/mnt/nvmedisk/linaro/qemu-from-laptop/qemu/build/asan/tests/qtest/migration-test+0x22f12e)
>  (BuildId: 852a267993587f557f50e5715f352f43720077ba)
>     #1 0x7f073982685d in __gmp_default_allocate 
> (/lib/x86_64-linux-gnu/libgmp.so.10+0xa85d) (BuildId: 
> f110719303ddbea25a5e89ff730fec520eed67b0)
>     #2 0x7f0739836193 in __gmpz_realloc 
> (/lib/x86_64-linux-gnu/libgmp.so.10+0x1a193) (BuildId: 
> f110719303ddbea25a5e89ff730fec520eed67b0)
>     #3 0x7f0739836594 in __gmpz_import 
> (/lib/x86_64-linux-gnu/libgmp.so.10+0x1a594) (BuildId: 
> f110719303ddbea25a5e89ff730fec520eed67b0)
>     #4 0x7f07398a91ed in nettle_mpz_set_str_256_u 
> (/lib/x86_64-linux-gnu/libhogweed.so.6+0xb1ed) (BuildId: 
> 3cc4a3474de72db89e9dcc93bfb95fe377f48c37)
>     #5 0x7f073a146a5a  (/lib/x86_64-linux-gnu/libgnutls.so.30+0x131a5a) 
> (BuildId: 97b8f99f392f1fd37b969a7164bcea884e23649b)
>     #6 0x7f073a07192c  (/lib/x86_64-linux-gnu/libgnutls.so.30+0x5c92c) 
> (BuildId: 97b8f99f392f1fd37b969a7164bcea884e23649b)
>     #7 0x7f073a078333  (/lib/x86_64-linux-gnu/libgnutls.so.30+0x63333) 
> (BuildId: 97b8f99f392f1fd37b969a7164bcea884e23649b)
>     #8 0x7f073a0e8353  (/lib/x86_64-linux-gnu/libgnutls.so.30+0xd3353) 
> (BuildId: 97b8f99f392f1fd37b969a7164bcea884e23649b)
>     #9 0x7f073a0ef0ac in gnutls_x509_privkey_import 
> (/lib/x86_64-linux-gnu/libgnutls.so.30+0xda0ac) (BuildId: 
> 97b8f99f392f1fd37b969a7164bcea884e23649b)
>     #10 0x55fa6d2547e3 in test_tls_load_key 
> tests/unit/crypto-tls-x509-helpers.c:99:11
>     #11 0x55fa6d25460c in test_tls_init 
> tests/unit/crypto-tls-x509-helpers.c:128:15
>     #12 0x55fa6d2495c4 in test_migrate_tls_x509_start_common 
> tests/qtest/migration-test.c:1044:5
>     #13 0x55fa6d24c23a in test_migrate_tls_x509_start_reject_anon_client 
> tests/qtest/migration-test.c:1216:12
>     #14 0x55fa6d23fb40 in test_precopy_common 
> tests/qtest/migration-test.c:1789:21
>     #15 0x55fa6d236b7c in test_precopy_tcp_tls_x509_reject_anon_client 
> tests/qtest/migration-test.c:2614:5
>
> (Oddly, there is no reported leak in the x509 unit tests, even though
> those also use test_tls_init() and test_tls_cleanup().)
>
> Deinit the privkey in test_tls_cleanup().
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Reviewed-by: Fabiano Rosas <farosas@suse.de>