]> granicus.if.org Git - python/commit
_hotshot hotshot_profiler(): If write_header() returned
authorTim Peters <tim.peters@gmail.com>
Tue, 7 Mar 2006 23:53:32 +0000 (23:53 +0000)
committerTim Peters <tim.peters@gmail.com>
Tue, 7 Mar 2006 23:53:32 +0000 (23:53 +0000)
commitdf44ab7b1cfb97b29712f40db422f27b2d8d1838
tree0c564fe1d04c4d95733766167aa3ab603b06ab2f
parent516999e6e2b1d89bc29a8930d651e4cddf968bfe
_hotshot hotshot_profiler():  If write_header() returned
an error code, this let `self` leak.  This is a disaster
on Windows, since `self` already points to a newly-opened
file object, and it was impossible for Python code to
close the thing since the only reference to it was in a
blob of leaked C memory.

test_hotshot test_bad_sys_path():  This new test provoked
the C bug above.  This test passed, but left an open
"@test" file behind, which caused a massive cascade of
bogus test failures in later, unrelated tests on Windows.
Changed the test code to remove the @test file it leaves
behind, which relies on the change above to close that
file first.
Lib/test/test_hotshot.py
Modules/_hotshot.c