From 775f7bce09426516a813bbbbd7af749dd4eea59e Mon Sep 17 00:00:00 2001 From: Andy Green Date: Mon, 19 Mar 2018 08:05:37 +0800 Subject: [PATCH] coverity 188313: client: avoid possibility of strlen on NULL cce --- lib/client/client.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/client/client.c b/lib/client/client.c index 04c9201b..25e345c2 100644 --- a/lib/client/client.c +++ b/lib/client/client.c @@ -1171,11 +1171,15 @@ bail3: close_reason = LWS_CLOSE_STATUS_NOSTATUS; bail2: - if (wsi->protocol) + if (wsi->protocol) { + n = 0; + if (cce) + n = strlen(cce); wsi->protocol->callback(wsi, LWS_CALLBACK_CLIENT_CONNECTION_ERROR, wsi->user_space, (void *)cce, - (unsigned int)strlen(cce)); + (unsigned int)n); + } wsi->already_did_cce = 1; lwsl_info("closing connection due to bail2 connection error\n");