diff options
author | Lei Zhang <thestig@chromium.org> | 2016-01-07 00:42:45 -0800 |
---|---|---|
committer | Lei Zhang <thestig@chromium.org> | 2016-01-07 00:42:45 -0800 |
commit | 0213958bf66e839f132d9dea531d2627091c3cbe (patch) | |
tree | 5dbb6b4a781b24506cdd3e937eba17620570cc8e | |
parent | cbed99098ad4cdc3f9f260cc1e8521f67c12d35d (diff) | |
download | pdfium-chromium/2619.tar.xz |
Fix a GetFileContents() call that should have been in commit aa326bd6.chromium/2619chromium/2618chromium/2617chromium/2616
TBR=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1568763003 .
-rw-r--r-- | testing/test_support.cpp | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/testing/test_support.cpp b/testing/test_support.cpp index ce143ae8e9..96e01c5b01 100644 --- a/testing/test_support.cpp +++ b/testing/test_support.cpp @@ -45,11 +45,12 @@ bool GetExternalData(const std::string& exe_path, std::string full_path = GetFullPathForSnapshotFile(exe_path, bin_dir, filename); size_t data_length = 0; - char* data_buffer = GetFileContents(full_path.c_str(), &data_length); - if (!data_buffer) { + std::unique_ptr<char, pdfium::FreeDeleter> data_buffer = + GetFileContents(full_path.c_str(), &data_length); + if (!data_buffer) return false; - } - result_data->data = const_cast<const char*>(data_buffer); + + result_data->data = data_buffer.release(); result_data->raw_size = data_length; return true; } |