diff options
author | Alexander Polcyn <apolcyn@google.com> | 2018-04-19 10:19:18 -0700 |
---|---|---|
committer | Alexander Polcyn <apolcyn@google.com> | 2018-04-19 10:20:20 -0700 |
commit | 046ad6b13f5eaba4b833abb0ecb49e2c6b75dd46 (patch) | |
tree | 9a03cccda90de15f74f48867c8dc2743ae0b6770 | |
parent | 6f3cadb687f7d9e3da3d49b18c3f68463352db6b (diff) |
Fix bug in addr sorting shutdonw
-rw-r--r-- | test/cpp/naming/address_sorting_test.cc | 10 | ||||
-rw-r--r-- | third_party/address_sorting/address_sorting.c | 1 |
2 files changed, 10 insertions, 1 deletions
diff --git a/test/cpp/naming/address_sorting_test.cc b/test/cpp/naming/address_sorting_test.cc index df93ff6f8d..a423733caf 100644 --- a/test/cpp/naming/address_sorting_test.cc +++ b/test/cpp/naming/address_sorting_test.cc @@ -722,16 +722,24 @@ TEST(AddressSortingTest, TestStableSortV4CompatAndSiteLocalAddresses) { } int main(int argc, char** argv) { - const char* resolver = gpr_getenv("GRPC_DNS_RESOLVER"); + char* resolver = gpr_getenv("GRPC_DNS_RESOLVER"); if (resolver == nullptr || strlen(resolver) == 0) { gpr_setenv("GRPC_DNS_RESOLVER", "ares"); } else if (strcmp("ares", resolver)) { gpr_log(GPR_INFO, "GRPC_DNS_RESOLVER != ares: %s.", resolver); } + gpr_free(resolver); grpc_test_init(argc, argv); ::testing::InitGoogleTest(&argc, argv); grpc_init(); auto result = RUN_ALL_TESTS(); grpc_shutdown(); + // Test sequential and nested inits and shutdowns. + grpc_init(); + grpc_init(); + grpc_shutdown(); + grpc_shutdown(); + grpc_init(); + grpc_shutdown(); return result; } diff --git a/third_party/address_sorting/address_sorting.c b/third_party/address_sorting/address_sorting.c index ec46099bec..d62aca3424 100644 --- a/third_party/address_sorting/address_sorting.c +++ b/third_party/address_sorting/address_sorting.c @@ -366,4 +366,5 @@ void address_sorting_shutdown() { abort(); } g_current_source_addr_factory->vtable->destroy(g_current_source_addr_factory); + g_current_source_addr_factory = NULL; } |