From a8315807e76625a03c8272f42cdc7218608a3047 Mon Sep 17 00:00:00 2001 From: Andy Green Date: Wed, 19 Aug 2020 06:46:19 +0100 Subject: [PATCH] coverity: 10417: move goto inside preprocessor conditional that needs it Otherwise coverity sees it with !defined(LWS_ROLE_WS) sitting there doing nothing --- lib/roles/h1/ops-h1.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/roles/h1/ops-h1.c b/lib/roles/h1/ops-h1.c index 1b510b371..9073652e9 100644 --- a/lib/roles/h1/ops-h1.c +++ b/lib/roles/h1/ops-h1.c @@ -1021,12 +1021,13 @@ rops_client_bind_h1(struct lws *wsi, const struct lws_client_connect_info *i) #if defined(LWS_ROLE_WS) if (lws_create_client_ws_object(i, wsi)) goto fail_wsi; + + goto bind_h1; #else lwsl_err("%s: ws role not configured\n", __func__); goto fail_wsi; #endif - goto bind_h1; } /* if a recognized http method, bind to it */