diff options
author | tsepez <tsepez@chromium.org> | 2016-11-28 13:59:14 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-11-28 13:59:14 -0800 |
commit | 405ac0f09e1622d7ff3cf60314d290851ac9f7fd (patch) | |
tree | b4fc6bddcbdde7ff3f709be189c8caaca8252af1 /core/fpdfapi/parser/cpdf_stream_acc.cpp | |
parent | 3c0f366daafed42495b22ecbdda8f21f3f939d9b (diff) | |
download | pdfium-405ac0f09e1622d7ff3cf60314d290851ac9f7fd.tar.xz |
Use CFX_MaybeOwned<> in fpdf_edit_create.cppchromium/2936
Fix missing second template parameter in cfx_maybe_owned.h
Review-Url: https://codereview.chromium.org/2522313002
Diffstat (limited to 'core/fpdfapi/parser/cpdf_stream_acc.cpp')
-rw-r--r-- | core/fpdfapi/parser/cpdf_stream_acc.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/core/fpdfapi/parser/cpdf_stream_acc.cpp b/core/fpdfapi/parser/cpdf_stream_acc.cpp index 01d8e148df..423de7c571 100644 --- a/core/fpdfapi/parser/cpdf_stream_acc.cpp +++ b/core/fpdfapi/parser/cpdf_stream_acc.cpp @@ -74,14 +74,14 @@ uint32_t CPDF_StreamAcc::GetSize() const { return m_pStream ? m_pStream->GetRawSize() : 0; } -uint8_t* CPDF_StreamAcc::DetachData() { +std::unique_ptr<uint8_t, FxFreeDeleter> CPDF_StreamAcc::DetachData() { if (m_bNewBuf) { - uint8_t* p = m_pData; + std::unique_ptr<uint8_t, FxFreeDeleter> p(m_pData); m_pData = nullptr; m_dwSize = 0; return p; } - uint8_t* p = FX_Alloc(uint8_t, m_dwSize); - FXSYS_memcpy(p, m_pData, m_dwSize); + std::unique_ptr<uint8_t, FxFreeDeleter> p(FX_Alloc(uint8_t, m_dwSize)); + FXSYS_memcpy(p.get(), m_pData, m_dwSize); return p; } |