From 6a9bd051c439ef66cf00795b3829ca3ceece0497 Mon Sep 17 00:00:00 2001 From: Garrett D'Amore Date: Sun, 3 Jan 2021 10:58:38 -0800 Subject: fixes #1401 valgrind reports leaks in all tests This arranges for nng_fini to be called via atexit in the test version of the library. It also cleans up some of the actual tests to reduce extraneous (and in some cases incorrect) calls to nng_fini. --- tests/reqctx.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'tests/reqctx.c') diff --git a/tests/reqctx.c b/tests/reqctx.c index e937b995..c727b91c 100644 --- a/tests/reqctx.c +++ b/tests/reqctx.c @@ -1,5 +1,5 @@ // -// Copyright 2018 Staysail Systems, Inc. +// Copyright 2021 Staysail Systems, Inc. // Copyright 2018 Capitar IT Group BV // // This software is supplied under the terms of the MIT License, a @@ -28,10 +28,10 @@ static struct { } rep_state; void -rep_cb(void *notused) +rep_cb(void *unused) { int rv; - (void) notused; + (void) unused; nng_mtx_lock(rep_state.mtx); if (rep_state.state == START) { @@ -96,8 +96,6 @@ TestMain("REQ concurrent contexts", { memset(recv_order, 0, NCTX * sizeof(int)); - atexit(nng_fini); - Convey("We can use REQ contexts concurrently", { nng_socket req; -- cgit v1.2.3-70-g09d2