You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by br...@apache.org on 2014/07/14 22:34:58 UTC
git commit: TS-1475: Fix memory leak in error case. Coverity #1022140
Repository: trafficserver
Updated Branches:
refs/heads/master a0e322c5f -> 56c5bd90d
TS-1475: Fix memory leak in error case. Coverity #1022140
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/56c5bd90
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/56c5bd90
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/56c5bd90
Branch: refs/heads/master
Commit: 56c5bd90d8d35fb486764cd031d6ddee9a424741
Parents: a0e322c
Author: Brian Geffon <br...@apache.org>
Authored: Mon Jul 14 13:34:39 2014 -0700
Committer: Brian Geffon <br...@apache.org>
Committed: Mon Jul 14 13:34:51 2014 -0700
----------------------------------------------------------------------
proxy/logging/LogAccess.cc | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/56c5bd90/proxy/logging/LogAccess.cc
----------------------------------------------------------------------
diff --git a/proxy/logging/LogAccess.cc b/proxy/logging/LogAccess.cc
index 5009113..c0f81c0 100644
--- a/proxy/logging/LogAccess.cc
+++ b/proxy/logging/LogAccess.cc
@@ -1371,6 +1371,8 @@ resolve_logfield_string(LogAccess *context, const char *format_str)
//
if (!n_fields) {
Debug("log-resolve", "No fields found; returning copy of format_str");
+ ats_free(printf_str);
+ ats_free(fields_str);
return ats_strdup(format_str);
}