vhost: track vh-protocol protocol init

We may paint vhosts with plugin protocols, but that doesn't mean
that the protocol is enabled on the vhost.  In turn that means if
we didn't do PROTOCOL_INIT, we should not do PROTOCOL_DESTROY on
the vhost-protocol combo.

Author: lws-team

lib/core-net/private-lib-core-net.h

@@ -534,7 +534,7 @@ struct lws_vhost {
 #if defined(LWS_WITH_TLS_SESSIONS)
 	uint32_t		tls_session_cache_max;
 #endif
-
+	uint32_t		protocol_init; /* bitmap indicating if protocol initialized */
 #if defined(LWS_WITH_SECURE_STREAMS_STATIC_POLICY_ONLY) || defined(LWS_WITH_SECURE_STREAMS_CPP)
 	int8_t			ss_refcount;
 	/**< refcount of number of ss connections with streamtypes using this

lib/core-net/vhost.c

@@ -466,6 +466,7 @@ lws_protocol_init_vhost(struct lws_vhost *vh, int *any)
 		) {
 			lwsl_vhost_info(vh, "init %s.%s", vh->name,
 					vh->protocols[n].name);
+			vh->protocol_init |= 1 << n;
 			if (vh->protocols[n].callback((struct lws *)lwsa,
 				LWS_CALLBACK_PROTOCOL_INIT, NULL,
 #if !defined(LWS_WITH_PLUGINS)
@@ -1492,7 +1493,7 @@ __lws_vhost_destroy2(struct lws_vhost *vh)
 
 			lwsl_vhost_debug(vh, "protocol destroy");
 
-			if (protocol->callback)
+			if (protocol->callback && (vh->protocol_init & (1 << n)))
 				protocol->callback(&wsi, LWS_CALLBACK_PROTOCOL_DESTROY,
 					   NULL, NULL, 0);
 			protocol++;