Mutual exclusion problem in mysql_remap plugin

[klebertarcisio]

Hi Peeps,

Look this previous discussion in: #6627

I am commenting about the malloc of line 188. Is there a need to call the free function?
The do_mysql_remap function already has a local query variable.
What would be the best solution? Delete the my_data struct? How would it be replaced?
The iniparser_freedict method frees several resources. Would it be interesting to release the data as well?
Sorry for the amount of questions. I'm trying to understand the big picture.

trafficserver/plugins/experimental/mysql_remap/mysql_remap.cc

Line 188 in 21f6a70

my_data *data = static_cast<my_data *>(malloc(1 * sizeof(my_data)));

Regards

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.

[bryancall]

The plan is the deprecate the plugin for ATS 10.x. Do you use this plugin and do you want to support it?

[bryancall]

The mysql_remap plugin was removed from ATS in 2022 (#9257; it had been deprecated in 9.0.0), so the code this issue references no longer ships. Closing as obsolete.

Update: the mysql_remap plugin was removed as of the 10.0.0 release (current: 10.1.2).