[MCOL-881] Mix of new/free Created: 2017-08-17 Updated: 2019-07-10 Resolved: 2019-07-10 |
|
| Status: | Closed |
| Project: | MariaDB ColumnStore |
| Component/s: | MariaDB Server |
| Affects Version/s: | None |
| Fix Version/s: | Icebox |
| Type: | Bug | Priority: | Minor |
| Reporter: | Andrew Hutchings (Inactive) | Assignee: | Unassigned |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Description |
|
Spotted in valgrind in 1.1, not sure if 1.0 is affected yet. ha_calpont_impl.cpp:4211 runs a new to generate cal_conn_info. A free is run on it in sql/sql_class.h:2955 |
| Comments |
| Comment by David Hall (Inactive) [ 2017-08-17 ] |
|
The comment at the free in sql/sql_class.h explains why it's done. In the past, there was no delete or free on the object leaving a memory leak. The free was added as a kludge to get rid of the memory leak with ideas on how to fix it correctly. |
| Comment by Andrew Hutchings (Inactive) [ 2019-07-10 ] |
|
Going away in 1.4 |