[DRILL-5368] Memory leak in C++ server metadata handler

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.10.0
• Fix Version/s: 1.11.0
• Component/s: Client - C++
• Labels: ready-to-commit
• Assignee: Laurent Goujon
• Reporter: Laurent Goujon
• Created: 20/Mar/17 17:54
• Updated: 03/Apr/17 05:17
• Resolved: 03/Apr/17 05:17

Description

When receiving server metadata response, a protobuf ServerMetaResp object is dynamically allocated but never freed.

Since for this handler, there's no need to keep the instance attached to the handler (content is copied over by the MetaData class), a reference is enough and allocation can be done on the stack.

Issue Links

• links to: GitHub Pull Request #790