You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2020/04/20 20:44:07 UTC

[trafficserver] branch 9.0.x updated: Fixes minor memory leak in configure_net

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new fff8c18  Fixes minor memory leak in configure_net
fff8c18 is described below

commit fff8c180ae9942aadddefc2d884201f1b0197a56
Author: Randall Meyer <rr...@apple.com>
AuthorDate: Mon Apr 20 10:23:31 2020 -0700

    Fixes minor memory leak in configure_net
    
    This was introduced in 58b154bb6f253ac8660bd778dd2bf43cb7b60687
    
    (cherry picked from commit 18066c24fa50792a6738f88ac8e47edd805d0388)
---
 iocore/net/Net.cc | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/iocore/net/Net.cc b/iocore/net/Net.cc
index c8af271..f5bbb88 100644
--- a/iocore/net/Net.cc
+++ b/iocore/net/Net.cc
@@ -65,10 +65,15 @@ configure_net()
   REC_ReadConfigStringAlloc(ccp, "proxy.config.net.tcp_congestion_control_in");
   if (ccp && *ccp != '\0') {
     net_ccp_in = ccp;
+  } else {
+    ats_free(ccp);
   }
+
   REC_ReadConfigStringAlloc(ccp, "proxy.config.net.tcp_congestion_control_out");
   if (ccp && *ccp != '\0') {
     net_ccp_out = ccp;
+  } else {
+    ats_free(ccp);
   }
 }