cfilters: make Curl_conn_connect always assign 'done'

It could return error without assigning it, and we have a caller in
multi.c that assumes it gets set.

Spotted by CodeSonar
Closes #13884

diff --git a/lib/cfilters.c b/lib/cfilters.c
index a1e405bc5e88bb07ce7ac5259aa51b2c3bfbbd21..60767aa38902d6702edc9e6eff017d68695fd919 100644 (file)
--- a/lib/cfilters.c
+++ b/lib/cfilters.c
@@ -345,8 +345,10 @@ CURLcode Curl_conn_connect(struct Curl_easy *data,
 
   cf = data->conn->cfilter[sockindex];
   DEBUGASSERT(cf);
-  if(!cf)
+  if(!cf) {
+    *done = FALSE;
     return CURLE_FAILED_INIT;
+  }
 
   *done = cf->connected;
   if(!*done) {