Author: davide Date: Mon Feb 12 10:06:34 2018 New Revision: 324919 URL: http://llvm.org/viewvc/llvm-project?rev=324919&view=rev Log: [testsuite] Reintroduce test to check leaking.
It wasn't python leaking, it was lldb. Thanks to Pavel for the explanation. Pointy-hat to me. Added: lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c Added: lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile?rev=324919&view=auto ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile (added) +++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile Mon Feb 12 10:06:34 2018 @@ -0,0 +1,5 @@ +LEVEL = ../../make + +C_SOURCES := main.c + +include $(LEVEL)/Makefile.rules Added: lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py?rev=324919&view=auto ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py (added) +++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py Mon Feb 12 10:06:34 2018 @@ -0,0 +1,108 @@ +""" +Test whether a process started by lldb has no extra file descriptors open. +""" + +from __future__ import print_function + + +import os +import lldb +from lldbsuite.test import lldbutil +from lldbsuite.test.lldbtest import * +from lldbsuite.test.decorators import * + + +def python_leaky_fd_version(test): + import sys + # Python random module leaks file descriptors on some versions. + if sys.version_info >= (2, 7, 8) and sys.version_info < (2, 7, 10): + return "Python random module leaks file descriptors in this python version" + return None + + +class AvoidsFdLeakTestCase(TestBase): + + NO_DEBUG_INFO_TESTCASE = True + + mydir = TestBase.compute_mydir(__file__) + + @expectedFailure(python_leaky_fd_version, "bugs.freebsd.org/197376") + @expectedFailureAll( + oslist=['freebsd'], + bugnumber="llvm.org/pr25624 still failing with Python 2.7.10") + # The check for descriptor leakage needs to be implemented differently + # here. + @skipIfWindows + @skipIfTargetAndroid() # Android have some other file descriptors open by the shell + @skipIfDarwinEmbedded # <rdar://problem/33888742> # debugserver on ios has an extra fd open on launch + def test_fd_leak_basic(self): + self.do_test([]) + + @expectedFailure(python_leaky_fd_version, "bugs.freebsd.org/197376") + @expectedFailureAll( + oslist=['freebsd'], + bugnumber="llvm.org/pr25624 still failing with Python 2.7.10") + # The check for descriptor leakage needs to be implemented differently + # here. + @skipIfWindows + @skipIfTargetAndroid() # Android have some other file descriptors open by the shell + @skipIfDarwinEmbedded # <rdar://problem/33888742> # debugserver on ios has an extra fd open on launch + def test_fd_leak_log(self): + self.do_test(["log enable -f '/dev/null' lldb commands"]) + + def do_test(self, commands): + self.build() + exe = self.getBuildArtifact("a.out") + + for c in commands: + self.runCmd(c) + + target = self.dbg.CreateTarget(exe) + + process = target.LaunchSimple( + None, None, self.get_process_working_directory()) + self.assertTrue(process, PROCESS_IS_VALID) + + self.assertTrue( + process.GetState() == lldb.eStateExited, + "Process should have exited.") + self.assertTrue( + process.GetExitStatus() == 0, + "Process returned non-zero status. Were incorrect file descriptors passed?") + + @expectedFailure(python_leaky_fd_version, "bugs.freebsd.org/197376") + @expectedFailureAll( + oslist=['freebsd'], + bugnumber="llvm.org/pr25624 still failing with Python 2.7.10") + # The check for descriptor leakage needs to be implemented differently + # here. + @skipIfWindows + @skipIfTargetAndroid() # Android have some other file descriptors open by the shell + @skipIfDarwinEmbedded # <rdar://problem/33888742> # debugserver on ios has an extra fd open on launch + def test_fd_leak_multitarget(self): + self.build() + exe = self.getBuildArtifact("a.out") + + target = self.dbg.CreateTarget(exe) + breakpoint = target.BreakpointCreateBySourceRegex( + 'Set breakpoint here', lldb.SBFileSpec("main.c", False)) + self.assertTrue(breakpoint, VALID_BREAKPOINT) + + process1 = target.LaunchSimple( + None, None, self.get_process_working_directory()) + self.assertTrue(process1, PROCESS_IS_VALID) + self.assertTrue( + process1.GetState() == lldb.eStateStopped, + "Process should have been stopped.") + + target2 = self.dbg.CreateTarget(exe) + process2 = target2.LaunchSimple( + None, None, self.get_process_working_directory()) + self.assertTrue(process2, PROCESS_IS_VALID) + + self.assertTrue( + process2.GetState() == lldb.eStateExited, + "Process should have exited.") + self.assertTrue( + process2.GetExitStatus() == 0, + "Process returned non-zero status. Were incorrect file descriptors passed?") Added: lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c?rev=324919&view=auto ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c (added) +++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c Mon Feb 12 10:06:34 2018 @@ -0,0 +1,28 @@ +#include <sys/types.h> +#include <sys/stat.h> +#include <unistd.h> +#include <errno.h> +#include <stdio.h> + +int +main (int argc, char const **argv) +{ + struct stat buf; + int i, rv = 0; // Set breakpoint here. + + // Make sure stdin/stdout/stderr exist. + for (i = 0; i <= 2; ++i) { + if (fstat(i, &buf) != 0) + return 1; + } + + // Make sure no other file descriptors are open. + for (i = 3; i <= 256; ++i) { + if (fstat(i, &buf) == 0 || errno != EBADF) { + fprintf(stderr, "File descriptor %d is open.\n", i); + rv = 2; + } + } + + return rv; +} _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits