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:42:12 UTC

git commit: TS-1475: UpTest class doesn't take ownership of fd so it doesn't get closed. Coverity #1022145

Repository: trafficserver
Updated Branches:
  refs/heads/master df080dcea -> 6c95b89f0


TS-1475: UpTest class doesn't take ownership of fd so it doesn't get closed. Coverity #1022145


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/6c95b89f
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/6c95b89f
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/6c95b89f

Branch: refs/heads/master
Commit: 6c95b89f007c569354686c3f85ef41f2225c5bd7
Parents: df080dc
Author: Brian Geffon <br...@apache.org>
Authored: Mon Jul 14 13:42:05 2014 -0700
Committer: Brian Geffon <br...@apache.org>
Committed: Mon Jul 14 13:42:05 2014 -0700

----------------------------------------------------------------------
 proxy/http/HttpUpdateTester.cc | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/6c95b89f/proxy/http/HttpUpdateTester.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpUpdateTester.cc b/proxy/http/HttpUpdateTester.cc
index 50f3e5b..f935cee 100644
--- a/proxy/http/HttpUpdateTester.cc
+++ b/proxy/http/HttpUpdateTester.cc
@@ -122,5 +122,6 @@ init_http_update_test()
   }
 
   UpTest *u = new UpTest(f, new_ProxyMutex());
+  fclose(f); // UpTest doesn't take ownership of f.
   eventProcessor.schedule_imm(u);
 }