You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/05/12 18:23:35 UTC

[GitHub] [trafficserver] randall opened a new pull request, #8843: Fix leaks in ConfigManager::configName (#8269)

randall opened a new pull request, #8843:
URL: https://github.com/apache/trafficserver/pull/8843

   This fixes an ASan reported leak of ConfigManager::configName. It used
   to be strdup'd but not freed in the destructor. This simply changes it
   to a std::string.
   
   ASan also reported a leak in AddConfigFilesHere which is fixed with an
   ats_free as well.
   
   (cherry picked from commit ee820c73dbe8beef6d44df3e32ffb11438017460)
   
   Conflicts:
       src/traffic_manager/AddConfigFilesHere.cc


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] bryancall merged pull request #8843: Fix leaks in ConfigManager::configName (#8269)

Posted by GitBox <gi...@apache.org>.

bryancall merged PR #8843:
URL: https://github.com/apache/trafficserver/pull/8843


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] bneradt commented on pull request #8843: Fix leaks in ConfigManager::configName (#8269)

Posted by GitBox <gi...@apache.org>.

bneradt commented on PR #8843:
URL: https://github.com/apache/trafficserver/pull/8843#issuecomment-1125569069

   Looks good. The fedora/rocky builds should start passing once we pull in various 9.1.x unit test/build PRs for those later compilers.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org