Skip to content

fix: clear up memory leaks

Iurii Zhesson requested to merge slb/yz/fix-memory-leaks into master

Type of change

  • Bug Fix
  • Feature
  • Pipeline
  • Test

Does this introduce a change in the core logic?

  • No
  • Yes

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Does this follow conventional commits spec?

  • No
  • Yes

Have you set the target Milestone?

  • No
  • Yes

Have you set the no-detached-pipeline label?

  • No
  • Yes

Updates description?

Clear up various memory leaks: bare 'new' allocations without corresponding 'delete' or followed by potential exception; lost Derived class deleter on Base shared_ptr assignment from raw pointer to Derived; 'curl_slist_append' missing corresponding 'curl_slist_free_all'. Don't suppress leaks detected. Use common hidden job to declare sanitizer options environment variables in one place. Improve ABI check script and output readability. Remove unneeded includes, redundant, unused and commented out code, fix typos and grammar, format files involved.

Merge request reports