summaryrefslogtreecommitdiff
path: root/testing/test_support.cpp
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2016-01-07 00:42:45 -0800
committerLei Zhang <thestig@chromium.org>2016-01-07 00:42:45 -0800
commit0213958bf66e839f132d9dea531d2627091c3cbe (patch)
tree5dbb6b4a781b24506cdd3e937eba17620570cc8e /testing/test_support.cpp
parentcbed99098ad4cdc3f9f260cc1e8521f67c12d35d (diff)
downloadpdfium-0213958bf66e839f132d9dea531d2627091c3cbe.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 .
Diffstat (limited to 'testing/test_support.cpp')
-rw-r--r--testing/test_support.cpp9
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;
}