commit 1307cd42d2be1623f2f284e3c321e72be577946c
author Tim Duesterhus <tim@bastelstu.be> Sat Apr 22 17:47:35 2023 +0200
committer Willy Tarreau <w@1wt.eu> Sun Apr 23 00:28:25 2023 +0200
tree 23097dd43784fcb46c5337cb990d2d3c16ed1c2b
parent fe83f589068d8cd90ea63fabadd60408ae80cce1

CLEANUP: Stop checking the pointer before calling `ring_free()`

Changes performed with this Coccinelle patch:

    @@
    expression e;
    @@

    - if (e != NULL) {
    	ring_free(e);
    - }

    @@
    expression e;
    @@

    - if (e) {
    	ring_free(e);
    - }

    @@
    expression e;
    @@

    - if (e)
    	ring_free(e);

    @@
    expression e;
    @@

    - if (e != NULL)
    	ring_free(e);

src/sink.c

diff --git a/src/sink.c b/src/sink.c
index 55b6ad1..af0e891 100644
--- a/src/sink.c
+++ b/src/sink.c
@@ -1164,8 +1164,7 @@
 		free(sft);
 
 	if (sink) {
-		if (sink->ctx.ring)
-			ring_free(sink->ctx.ring);
+		ring_free(sink->ctx.ring);
 
 		LIST_DELETE(&sink->sink_list);
 		free(sink->name);