[ZOOKEEPER-3944] zookeeper c api sasl client memory leak

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.6.2
• Fix Version/s: 3.7.0
• Component/s: c client
• Assignee: Unassigned
• Reporter: Xiaotian Qi
• Created: 24/Sep/20 11:43
• Updated: 28/Mar/21 08:55
• Resolved: 28/Sep/20 07:27

Description

I find this memory leak through sanitizer.

Here within zookeeper_close() it destroy the client and reset it to NULL. This client is created using calloc()

In zoo_sasl_client_destroy(), the client is not freed, neither within zookeeper_close(). 

I would suggest add a free() before setting it to NULL in zookeeper_close