diff options
| author | thestig <thestig@chromium.org> | 2016-06-22 07:03:23 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-06-22 07:03:23 -0700 |
| commit | 29ce9238c6a05abb1de5e4175c97e4fcadd5640f (patch) | |
| tree | ddb81bfe0829948ca70d8b3f6b56bf68c6c9346b /testing/embedder_test.cpp | |
| parent | 907a5223ea5abd09878de20cc74c59ebd0d6c3c8 (diff) | |
| download | pdfium-29ce9238c6a05abb1de5e4175c97e4fcadd5640f.tar.xz | |
Fix memory leaks in EmbedderTest::OpenDocument().
EmbedderTest::OpenDocument() does not gracefully handle the case of
being called twice in a given test case. So avoid doing that.
Review-Url: https://codereview.chromium.org/2088093002
Diffstat (limited to 'testing/embedder_test.cpp')
| -rw-r--r-- | testing/embedder_test.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/testing/embedder_test.cpp b/testing/embedder_test.cpp index 59190a7751..9ef1e06dca 100644 --- a/testing/embedder_test.cpp +++ b/testing/embedder_test.cpp @@ -120,6 +120,7 @@ bool EmbedderTest::OpenDocument(const std::string& filename, if (!file_contents_) return false; + EXPECT_TRUE(!loader_); loader_ = new TestLoader(file_contents_.get(), file_length_); file_access_.m_FileLen = static_cast<unsigned long>(file_length_); file_access_.m_GetBlock = TestLoader::GetBlock; |