Fix memory leak from ssl cert in c client

apache · Oct 30, 2024 · 264cec2 · 264cec2
1 parent 6d71307
commit 264cec2
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/zookeeper-client/zookeeper-client-c/src/zookeeper.c b/zookeeper-client/zookeeper-client-c/src/zookeeper.c
@@ -692,6 +692,7 @@ static void destroy(zhandle_t *zh)
 #ifdef HAVE_OPENSSL_H
     if (zh->fd->cert) {
         free(zh->fd->cert->certstr);
+        free(zh->fd->cert->ca);
         free(zh->fd->cert);
         zh->fd->cert = NULL;
     }
@@ -3873,8 +3874,6 @@ int zookeeper_close(zhandle_t *zh)
     LOG_INFO(LOGCALLBACK(zh), "Freeing zookeeper resources for sessionId=%#llx\n", zh->client_id.client_id);
     destroy(zh);
     adaptor_destroy(zh);
-    free(zh->fd->cert->certstr);
-    free(zh->fd->cert->ca);
     free(zh->fd);
     free(zh);
 #ifdef _WIN32