diff options
| author | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-05 17:08:46 +0000 |
|---|---|---|
| committer | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-05 17:08:46 +0000 |
| commit | fcc681407908a0f160a4861fe046dc02fb80a3e3 (patch) | |
| tree | 56e88a00acc5ed78a098035f9c9d086f1d5a7d5f /net/ftp | |
| parent | 28ad92a4ab702510bc6287f407548d8b6e3a1cd5 (diff) | |
| download | chromium_src-fcc681407908a0f160a4861fe046dc02fb80a3e3.zip chromium_src-fcc681407908a0f160a4861fe046dc02fb80a3e3.tar.gz chromium_src-fcc681407908a0f160a4861fe046dc02fb80a3e3.tar.bz2 | |
Coverity: Fix leaking UCharsetDetector in ftp code.
CID=7193
BUG=none
TEST=using the additional test data, tools/valgrind/chrome_tests.sh -t net --gtest_filter="FtpDirectoryListingBufferTest.*"
Review URL: http://codereview.chromium.org/355037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31099 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/ftp')
| -rw-r--r-- | net/ftp/ftp_directory_listing_buffer.cc | 1 | ||||
| -rw-r--r-- | net/ftp/ftp_directory_listing_buffer_unittest.cc | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/net/ftp/ftp_directory_listing_buffer.cc b/net/ftp/ftp_directory_listing_buffer.cc index 030da83..d7833e8 100644 --- a/net/ftp/ftp_directory_listing_buffer.cc +++ b/net/ftp/ftp_directory_listing_buffer.cc @@ -27,6 +27,7 @@ std::string DetectEncoding(const std::string& text) { &status); const UCharsetMatch* match = ucsdet_detect(detector, &status); const char* encoding = ucsdet_getName(match, &status); + ucsdet_close(detector); // Should we check the quality of the match? A rather arbitrary number is // assigned by ICU and it's hard to come up with a lower limit. if (U_FAILURE(status)) diff --git a/net/ftp/ftp_directory_listing_buffer_unittest.cc b/net/ftp/ftp_directory_listing_buffer_unittest.cc index f8f1a51..ae448fb 100644 --- a/net/ftp/ftp_directory_listing_buffer_unittest.cc +++ b/net/ftp/ftp_directory_listing_buffer_unittest.cc @@ -17,6 +17,7 @@ namespace { TEST(FtpDirectoryListingBufferTest, Parse) { const char* test_files[] = { "dir-listing-ls-1", + "dir-listing-ls-1-utf8", "dir-listing-ls-2", "dir-listing-vms-1", "dir-listing-vms-2", |