From 77d8ed02c7e97471ceccee5abbabeb2fdea413c7 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Tue, 30 Jan 2018 17:48:32 +0000 Subject: Revert "Use UnownedPtr instead of T* in MaybeOwned." This reverts commit e563e8352139e4852a955e319023b09f2844aee9. Reason for revert: Original change's description: > Use UnownedPtr instead of T* in MaybeOwned. > > Always check the liftime in the unowned case. Doing so unearthed > the following issues: > > Transient lifetime issue in jbig2_image when doing realloc(). > Stale (but unused) dictionary pointer in CPDF_Image. > Destruction order in error branch in cpdf_dibsource.cpp > > Change-Id: I12b758aafeefedc7abe1e8b21a18db959929e95f > Reviewed-on: https://pdfium-review.googlesource.com/24552 > Commit-Queue: Tom Sepez > Reviewed-by: dsinclair TBR=thestig@chromium.org,tsepez@chromium.org,dsinclair@chromium.org Change-Id: I3c56ee6ab502da90e3adb7507dbc8cc92f090140 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://pdfium-review.googlesource.com/24670 Reviewed-by: Tom Sepez Commit-Queue: Tom Sepez --- core/fxcrt/maybe_owned.h | 22 ++++++---------------- 1 file changed, 6 insertions(+), 16 deletions(-) (limited to 'core/fxcrt/maybe_owned.h') diff --git a/core/fxcrt/maybe_owned.h b/core/fxcrt/maybe_owned.h index 130d2bdc3c..11dd68642d 100644 --- a/core/fxcrt/maybe_owned.h +++ b/core/fxcrt/maybe_owned.h @@ -11,7 +11,6 @@ #include "core/fxcrt/fx_memory.h" #include "core/fxcrt/fx_system.h" -#include "core/fxcrt/unowned_ptr.h" namespace fxcrt { @@ -34,34 +33,25 @@ class MaybeOwned { } void Reset(std::unique_ptr ptr) { - m_pObj = ptr.get(); m_pOwnedObj = std::move(ptr); + m_pObj = m_pOwnedObj.get(); } void Reset(T* ptr = nullptr) { - m_pObj = ptr; m_pOwnedObj.reset(); + m_pObj = ptr; } - T* Get() const { return m_pObj.Get(); } bool IsOwned() const { return !!m_pOwnedObj; } - - // Downgrades to unowned, caller takes ownership. + T* Get() const { return m_pObj; } std::unique_ptr Release() { ASSERT(IsOwned()); return std::move(m_pOwnedObj); } - // Downgrades to empty, caller takes ownership. - std::unique_ptr ReleaseAndClear() { - ASSERT(IsOwned()); - m_pObj = nullptr; - return std::move(m_pOwnedObj); - } - MaybeOwned& operator=(const MaybeOwned& that) = delete; MaybeOwned& operator=(MaybeOwned&& that) { - m_pObj = that.m_pObj; m_pOwnedObj = std::move(that.m_pOwnedObj); + m_pObj = that.m_pObj; that.m_pObj = nullptr; return *this; } @@ -88,11 +78,11 @@ class MaybeOwned { explicit operator bool() const { return !!m_pObj; } T& operator*() const { return *m_pObj; } - T* operator->() const { return m_pObj.Get(); } + T* operator->() const { return m_pObj; } private: std::unique_ptr m_pOwnedObj; - UnownedPtr m_pObj; + T* m_pObj; }; } // namespace fxcrt -- cgit v1.2.3