[master] 5129b3c81 We already own the priv pointer, so no need to take it.

Poul-Henning Kamp phk at FreeBSD.org
Thu Jan 7 14:10:07 UTC 2021


commit 5129b3c81dc88e814ed84f65c9fd312b32b7ed3f
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date:   Thu Jan 7 14:09:19 2021 +0000

    We already own the priv pointer, so no need to take it.
    
    (Spotted, in a round-about way by: Gcc on SunOS)

diff --git a/bin/varnishd/cache/cache_tcp_pool.c b/bin/varnishd/cache/cache_tcp_pool.c
index 31423253f..02db6c6f3 100644
--- a/bin/varnishd/cache/cache_tcp_pool.c
+++ b/bin/varnishd/cache/cache_tcp_pool.c
@@ -607,7 +607,7 @@ vtp_free(void *priv)
 {
 	struct tcp_pool *tp;
 
-	TAKE_OBJ_NOTNULL(tp, &priv, TCP_POOL_MAGIC);
+	CAST_OBJ_NOTNULL(tp, priv, TCP_POOL_MAGIC);
 	free(tp->ip4);
 	free(tp->ip6);
 	free(tp->uds);


More information about the varnish-commit mailing list