CLEANUP: stats: shut up a wrong null-deref warning from gcc 9.2

As reported in bug #447, gcc 9.2 invents impossible code paths and then
complains that we don't check for our pointers to be NULL... This code
path is not critical, better add the test to shut it up than try to
help it being less creative.

This code hasn't changed for a while, so it could help distros to
backport this to older releases.

diff --git a/src/stats.c b/src/stats.c
index 6330512ca88f0f5a2fec6ec5a13085e2d75a5e9d..f8a15bd547ad675ba2f280941fd3daa0e3aafc92 100644 (file)
--- a/src/stats.c
+++ b/src/stats.c
@@ -2411,7 +2411,7 @@ static void stats_dump_html_head(struct appctx *appctx, struct uri_auth *uri)
                      "-->\n"
                      "</style></head>\n",
                      (appctx->ctx.stats.flags & STAT_SHNODE) ? " on " : "",
-                     (appctx->ctx.stats.flags & STAT_SHNODE) ? (uri->node ? uri->node : global.node) : ""
+                     (appctx->ctx.stats.flags & STAT_SHNODE) ? (uri && uri->node ? uri->node : global.node) : ""
                      );
 }