CLEANUP: muxes: Remove useless if condition in show_fd function

In H1, H2 and FCGI muxes, in the show_fd function, there is duplicated test on
the stream's subs field.

This patch fixes the issue #1142. It may be backported as far as 2.2.

diff --git a/src/mux_fcgi.c b/src/mux_fcgi.c
index 494517b4430a28211e2c2a2ed05382e6ba58d935..70b3d227275c4fc367afee093e239f2be079137c 100644 (file)
--- a/src/mux_fcgi.c
+++ b/src/mux_fcgi.c
@@ -4126,14 +4126,12 @@ static int fcgi_show_fd(struct buffer *msg, struct connection *conn)
                                      fstrm->cs->flags, fstrm->cs->data);
                chunk_appendf(&trash, " .subs=%p", fstrm->subs);
                if (fstrm->subs) {
-                       if (fstrm->subs) {
-                               chunk_appendf(&trash, "(ev=%d tl=%p", fstrm->subs->events, fstrm->subs->tasklet);
-                               chunk_appendf(&trash, " tl.calls=%d tl.ctx=%p tl.fct=",
-                                             fstrm->subs->tasklet->calls,
-                                             fstrm->subs->tasklet->context);
-                               resolve_sym_name(&trash, NULL, fstrm->subs->tasklet->process);
-                               chunk_appendf(&trash, ")");
-                       }
+                       chunk_appendf(&trash, "(ev=%d tl=%p", fstrm->subs->events, fstrm->subs->tasklet);
+                       chunk_appendf(&trash, " tl.calls=%d tl.ctx=%p tl.fct=",
+                                     fstrm->subs->tasklet->calls,
+                                     fstrm->subs->tasklet->context);
+                       resolve_sym_name(&trash, NULL, fstrm->subs->tasklet->process);
+                       chunk_appendf(&trash, ")");
                }
        }
        return 0;

diff --git a/src/mux_h1.c b/src/mux_h1.c
index 1bfdab61c4298ad371decedb78a59e45bd6a48af..5ff3d54514726f554eb9fc4fcf928490b9ddbc96 100644 (file)
--- a/src/mux_h1.c
+++ b/src/mux_h1.c
@@ -3540,16 +3540,14 @@ static int h1_show_fd(struct buffer *msg, struct connection *conn)
 
                chunk_appendf(&trash, " .subs=%p", h1s->subs);
                if (h1s->subs) {
-                       if (h1s->subs) {
-                               chunk_appendf(&trash, "(ev=%d tl=%p", h1s->subs->events, h1s->subs->tasklet);
-                               chunk_appendf(&trash, " tl.calls=%d tl.ctx=%p tl.fct=",
-                                             h1s->subs->tasklet->calls,
-                                             h1s->subs->tasklet->context);
-                               if (h1s->subs->tasklet->calls >= 1000000)
-                                       ret = 1;
-                               resolve_sym_name(&trash, NULL, h1s->subs->tasklet->process);
-                               chunk_appendf(&trash, ")");
-                       }
+                       chunk_appendf(&trash, "(ev=%d tl=%p", h1s->subs->events, h1s->subs->tasklet);
+                       chunk_appendf(&trash, " tl.calls=%d tl.ctx=%p tl.fct=",
+                                     h1s->subs->tasklet->calls,
+                                     h1s->subs->tasklet->context);
+                       if (h1s->subs->tasklet->calls >= 1000000)
+                               ret = 1;
+                       resolve_sym_name(&trash, NULL, h1s->subs->tasklet->process);
+                       chunk_appendf(&trash, ")");
                }
        }
        return ret;

diff --git a/src/mux_h2.c b/src/mux_h2.c
index 09b42684f6cd8c7400b3825a3d5410df3d2ca411..1d1c6bb9f9f1b83e37798dd7d589e3bf3d7b7482 100644 (file)
--- a/src/mux_h2.c
+++ b/src/mux_h2.c
@@ -6495,16 +6495,14 @@ static int h2_show_fd(struct buffer *msg, struct connection *conn)
 
                chunk_appendf(&trash, " .subs=%p", h2s->subs);
                if (h2s->subs) {
-                       if (h2s->subs) {
-                               chunk_appendf(&trash, "(ev=%d tl=%p", h2s->subs->events, h2s->subs->tasklet);
-                               chunk_appendf(&trash, " tl.calls=%d tl.ctx=%p tl.fct=",
-                                             h2s->subs->tasklet->calls,
-                                             h2s->subs->tasklet->context);
-                               if (h2s->subs->tasklet->calls >= 1000000)
-                                       ret = 1;
-                               resolve_sym_name(&trash, NULL, h2s->subs->tasklet->process);
-                               chunk_appendf(&trash, ")");
-                       }
+                       chunk_appendf(&trash, "(ev=%d tl=%p", h2s->subs->events, h2s->subs->tasklet);
+                       chunk_appendf(&trash, " tl.calls=%d tl.ctx=%p tl.fct=",
+                                     h2s->subs->tasklet->calls,
+                                     h2s->subs->tasklet->context);
+                       if (h2s->subs->tasklet->calls >= 1000000)
+                               ret = 1;
+                       resolve_sym_name(&trash, NULL, h2s->subs->tasklet->process);
+                       chunk_appendf(&trash, ")");
                }
        }
        return ret;