diff options
author | Tom Sepez <tsepez@chromium.org> | 2018-05-16 16:32:43 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-05-16 16:32:43 +0000 |
commit | 977f9370ea6c8da403ddd73f8c13317520b5bf69 (patch) | |
tree | 019875bbc48a4c720bf8bfc939203e71dc95787c /core/fpdfapi/edit/cpdf_encryptor.h | |
parent | 5cf305791117afc774bd3da5122a9d8b1cd1878d (diff) | |
download | pdfium-977f9370ea6c8da403ddd73f8c13317520b5bf69.tar.xz |
Use pdfium::span<> in CPDF_Encryptor
Provides more lifetime/bounds checking "for free".
Change-Id: Ifaf7a95c0f6f288777cd6ef35996e01b3dc4bc63
Reviewed-on: https://pdfium-review.googlesource.com/32594
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'core/fpdfapi/edit/cpdf_encryptor.h')
-rw-r--r-- | core/fpdfapi/edit/cpdf_encryptor.h | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/core/fpdfapi/edit/cpdf_encryptor.h b/core/fpdfapi/edit/cpdf_encryptor.h index a62a4ebf22..67fd5c4f9b 100644 --- a/core/fpdfapi/edit/cpdf_encryptor.h +++ b/core/fpdfapi/edit/cpdf_encryptor.h @@ -9,9 +9,10 @@ #include <stdint.h> -#include <memory> +#include <vector> #include "core/fxcrt/fx_memory.h" +#include "third_party/base/span.h" class CPDF_CryptoHandler; @@ -19,17 +20,14 @@ class CPDF_Encryptor { public: CPDF_Encryptor(CPDF_CryptoHandler* pHandler, int objnum, - const uint8_t* src_data, - uint32_t src_size); + pdfium::span<const uint8_t> src_data); ~CPDF_Encryptor(); - uint32_t GetSize() const { return m_dwSize; } - const uint8_t* GetData() const { return m_pData; } + pdfium::span<const uint8_t> GetSpan() const { return m_Span; } private: - const uint8_t* m_pData = nullptr; - std::unique_ptr<uint8_t, FxFreeDeleter> m_pNewBuf; - uint32_t m_dwSize = 0; + std::vector<uint8_t> m_NewBuf; + pdfium::span<const uint8_t> m_Span; }; #endif // CORE_FPDFAPI_EDIT_CPDF_ENCRYPTOR_H_ |