| File: | home/maarten/src/libreoffice/core/sot/source/base/object.cxx |
| Warning: | line 47, column 9 Use of memory after it is freed |
Press '?' to see keyboard shortcuts
Keyboard shortcuts:
| 1 | /* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */ | |||
| 2 | /* | |||
| 3 | * This file is part of the LibreOffice project. | |||
| 4 | * | |||
| 5 | * This Source Code Form is subject to the terms of the Mozilla Public | |||
| 6 | * License, v. 2.0. If a copy of the MPL was not distributed with this | |||
| 7 | * file, You can obtain one at http://mozilla.org/MPL/2.0/. | |||
| 8 | * | |||
| 9 | * This file incorporates work covered by the following license notice: | |||
| 10 | * | |||
| 11 | * Licensed to the Apache Software Foundation (ASF) under one or more | |||
| 12 | * contributor license agreements. See the NOTICE file distributed | |||
| 13 | * with this work for additional information regarding copyright | |||
| 14 | * ownership. The ASF licenses this file to you under the Apache | |||
| 15 | * License, Version 2.0 (the "License"); you may not use this file | |||
| 16 | * except in compliance with the License. You may obtain a copy of | |||
| 17 | * the License at http://www.apache.org/licenses/LICENSE-2.0 . | |||
| 18 | */ | |||
| 19 | ||||
| 20 | #include <sot/object.hxx> | |||
| 21 | ||||
| 22 | SotObject::SotObject() | |||
| 23 | : nOwnerLockCount(0) | |||
| 24 | , bInClose(false) | |||
| 25 | { | |||
| 26 | } | |||
| 27 | ||||
| 28 | SotObject::~SotObject() = default; | |||
| 29 | ||||
| 30 | /** When the OwnerLock is decremented to zero, the DoClose method is called. | |||
| 31 | * This happens independently of the lock or RefCount. If the OwnerLock | |||
| 32 | * counter != zero, no DoClose is called by <SotObject::FuzzyLock>. | |||
| 33 | * | |||
| 34 | * bLock - true, lock. false, unlock. | |||
| 35 | */ | |||
| 36 | void SotObject::OwnerLock(bool bLock) | |||
| 37 | { | |||
| 38 | if (bLock) | |||
| ||||
| 39 | { | |||
| 40 | nOwnerLockCount++; | |||
| 41 | AddFirstRef(); | |||
| 42 | } | |||
| 43 | else if (nOwnerLockCount) | |||
| 44 | { | |||
| 45 | if (0 == --nOwnerLockCount) | |||
| 46 | DoClose(); | |||
| 47 | ReleaseRef(); | |||
| ||||
| 48 | } | |||
| 49 | } | |||
| 50 | ||||
| 51 | bool SotObject::DoClose() | |||
| 52 | { | |||
| 53 | bool bRet = false; | |||
| 54 | if (!bInClose) | |||
| 55 | { | |||
| 56 | tools::SvRef<SotObject> xHoldAlive(this); | |||
| 57 | bInClose = true; | |||
| 58 | bRet = Close(); | |||
| 59 | bInClose = false; | |||
| 60 | } | |||
| 61 | return bRet; | |||
| 62 | } | |||
| 63 | ||||
| 64 | bool SotObject::Close() | |||
| 65 | { | |||
| 66 | return true; | |||
| 67 | } | |||
| 68 | ||||
| 69 | /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ |
| 1 | /* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */ |
| 2 | /* |
| 3 | * This file is part of the LibreOffice project. |
| 4 | * |
| 5 | * This Source Code Form is subject to the terms of the Mozilla Public |
| 6 | * License, v. 2.0. If a copy of the MPL was not distributed with this |
| 7 | * file, You can obtain one at http://mozilla.org/MPL/2.0/. |
| 8 | * |
| 9 | * This file incorporates work covered by the following license notice: |
| 10 | * |
| 11 | * Licensed to the Apache Software Foundation (ASF) under one or more |
| 12 | * contributor license agreements. See the NOTICE file distributed |
| 13 | * with this work for additional information regarding copyright |
| 14 | * ownership. The ASF licenses this file to you under the Apache |
| 15 | * License, Version 2.0 (the "License"); you may not use this file |
| 16 | * except in compliance with the License. You may obtain a copy of |
| 17 | * the License at http://www.apache.org/licenses/LICENSE-2.0 . |
| 18 | */ |
| 19 | #ifndef INCLUDED_TOOLS_REF_HXX |
| 20 | #define INCLUDED_TOOLS_REF_HXX |
| 21 | |
| 22 | #include <sal/config.h> |
| 23 | #include <cassert> |
| 24 | #include <tools/toolsdllapi.h> |
| 25 | #include <utility> |
| 26 | |
| 27 | /** |
| 28 | This implements similar functionality to boost::intrusive_ptr |
| 29 | */ |
| 30 | |
| 31 | namespace tools { |
| 32 | |
| 33 | /** T must be a class that extends SvRefBase */ |
| 34 | template<typename T> class SAL_DLLPUBLIC_RTTI__attribute__ ((type_visibility("default"))) SvRef final { |
| 35 | public: |
| 36 | SvRef(): pObj(nullptr) {} |
| 37 | |
| 38 | SvRef(SvRef&& rObj) noexcept |
| 39 | { |
| 40 | pObj = rObj.pObj; |
| 41 | rObj.pObj = nullptr; |
| 42 | } |
| 43 | |
| 44 | SvRef(SvRef const & rObj): pObj(rObj.pObj) |
| 45 | { |
| 46 | if (pObj != nullptr) pObj->AddNextRef(); |
| 47 | } |
| 48 | |
| 49 | SvRef(T * pObjP): pObj(pObjP) |
| 50 | { |
| 51 | if (pObj != nullptr) pObj->AddFirstRef(); |
| 52 | } |
| 53 | |
| 54 | ~SvRef() |
| 55 | { |
| 56 | if (pObj != nullptr) pObj->ReleaseRef(); |
| 57 | } |
| 58 | |
| 59 | void clear() |
| 60 | { |
| 61 | if (pObj != nullptr) { |
| 62 | T * pRefObj = pObj; |
| 63 | pObj = nullptr; |
| 64 | pRefObj->ReleaseRef(); |
| 65 | } |
| 66 | } |
| 67 | |
| 68 | SvRef & operator =(SvRef const & rObj) |
| 69 | { |
| 70 | if (rObj.pObj != nullptr) { |
| 71 | rObj.pObj->AddNextRef(); |
| 72 | } |
| 73 | T * pRefObj = pObj; |
| 74 | pObj = rObj.pObj; |
| 75 | if (pRefObj != nullptr) { |
| 76 | pRefObj->ReleaseRef(); |
| 77 | } |
| 78 | return *this; |
| 79 | } |
| 80 | |
| 81 | SvRef & operator =(SvRef && rObj) |
| 82 | { |
| 83 | if (pObj != nullptr) { |
| 84 | pObj->ReleaseRef(); |
| 85 | } |
| 86 | pObj = rObj.pObj; |
| 87 | rObj.pObj = nullptr; |
| 88 | return *this; |
| 89 | } |
| 90 | |
| 91 | bool is() const { return pObj != nullptr; } |
| 92 | |
| 93 | explicit operator bool() const { return is(); } |
| 94 | |
| 95 | T * get() const { return pObj; } |
| 96 | |
| 97 | T * operator ->() const { assert(pObj != nullptr)(static_cast <bool> (pObj != nullptr) ? void (0) : __assert_fail ("pObj != nullptr", "/home/maarten/src/libreoffice/core/include/tools/ref.hxx" , 97, __extension__ __PRETTY_FUNCTION__)); return pObj; } |
| 98 | |
| 99 | T & operator *() const { assert(pObj != nullptr)(static_cast <bool> (pObj != nullptr) ? void (0) : __assert_fail ("pObj != nullptr", "/home/maarten/src/libreoffice/core/include/tools/ref.hxx" , 99, __extension__ __PRETTY_FUNCTION__)); return *pObj; } |
| 100 | |
| 101 | bool operator ==(const SvRef<T> &rhs) const { return pObj == rhs.pObj; } |
| 102 | bool operator !=(const SvRef<T> &rhs) const { return !(*this == rhs); } |
| 103 | |
| 104 | private: |
| 105 | T * pObj; |
| 106 | }; |
| 107 | |
| 108 | /** |
| 109 | * This implements similar functionality to std::make_shared. |
| 110 | */ |
| 111 | template<typename T, typename... Args> |
| 112 | SvRef<T> make_ref(Args&& ... args) |
| 113 | { |
| 114 | return SvRef<T>(new T(std::forward<Args>(args)...)); |
| 115 | } |
| 116 | |
| 117 | } |
| 118 | |
| 119 | /** Classes that want to be referenced-counted via SvRef<T>, should extend this base class */ |
| 120 | class TOOLS_DLLPUBLIC__attribute__ ((visibility("default"))) SvRefBase |
| 121 | { |
| 122 | // work around a clang 3.5 optimization bug: if the bNoDelete is *first* |
| 123 | // it mis-compiles "if (--nRefCount == 0)" and never deletes any object |
| 124 | unsigned int nRefCount : 31; |
| 125 | // the only reason this is not bool is because MSVC cannot handle mixed type bitfields |
| 126 | unsigned int bNoDelete : 1; |
| 127 | |
| 128 | protected: |
| 129 | virtual ~SvRefBase() COVERITY_NOEXCEPT_FALSE; |
| 130 | |
| 131 | public: |
| 132 | SvRefBase() : nRefCount(0), bNoDelete(1) {} |
| 133 | SvRefBase(const SvRefBase &) : nRefCount(0), bNoDelete(1) {} |
| 134 | |
| 135 | SvRefBase & operator=(const SvRefBase &) { return *this; } |
| 136 | |
| 137 | void RestoreNoDelete() |
| 138 | { bNoDelete = 1; } |
| 139 | |
| 140 | void AddNextRef() |
| 141 | { |
| 142 | assert( nRefCount < (1 << 30) && "Do not add refs to dead objects" )(static_cast <bool> (nRefCount < (1 << 30) && "Do not add refs to dead objects") ? void (0) : __assert_fail ("nRefCount < (1 << 30) && \"Do not add refs to dead objects\"" , "/home/maarten/src/libreoffice/core/include/tools/ref.hxx", 142, __extension__ __PRETTY_FUNCTION__)); |
| 143 | ++nRefCount; |
| 144 | } |
| 145 | |
| 146 | void AddFirstRef() |
| 147 | { |
| 148 | assert( nRefCount < (1 << 30) && "Do not add refs to dead objects" )(static_cast <bool> (nRefCount < (1 << 30) && "Do not add refs to dead objects") ? void (0) : __assert_fail ("nRefCount < (1 << 30) && \"Do not add refs to dead objects\"" , "/home/maarten/src/libreoffice/core/include/tools/ref.hxx", 148, __extension__ __PRETTY_FUNCTION__)); |
| 149 | if( bNoDelete ) |
| 150 | bNoDelete = 0; |
| 151 | ++nRefCount; |
| 152 | } |
| 153 | |
| 154 | void ReleaseRef() |
| 155 | { |
| 156 | assert( nRefCount >= 1)(static_cast <bool> (nRefCount >= 1) ? void (0) : __assert_fail ("nRefCount >= 1", "/home/maarten/src/libreoffice/core/include/tools/ref.hxx" , 156, __extension__ __PRETTY_FUNCTION__)); |
| 157 | if( --nRefCount == 0 && !bNoDelete) |
| 158 | { |
| 159 | // I'm not sure about the original purpose of this line, but right now |
| 160 | // it serves the purpose that anything that attempts to do an AddRef() |
| 161 | // after an object is deleted will trip an assert. |
| 162 | nRefCount = 1 << 30; |
| 163 | delete this; |
| 164 | } |
| 165 | } |
| 166 | |
| 167 | unsigned int GetRefCount() const |
| 168 | { return nRefCount; } |
| 169 | }; |
| 170 | |
| 171 | template<typename T> |
| 172 | class SvCompatWeakBase; |
| 173 | |
| 174 | /** SvCompatWeakHdl acts as an intermediary between SvCompatWeakRef<T> and T. |
| 175 | */ |
| 176 | template<typename T> |
| 177 | class SvCompatWeakHdl final : public SvRefBase |
| 178 | { |
| 179 | friend class SvCompatWeakBase<T>; |
| 180 | T* _pObj; |
| 181 | |
| 182 | SvCompatWeakHdl( T* pObj ) : _pObj( pObj ) {} |
| 183 | |
| 184 | public: |
| 185 | void ResetWeakBase( ) { _pObj = nullptr; } |
| 186 | T* GetObj() { return _pObj; } |
| 187 | }; |
| 188 | |
| 189 | /** We only have one place that extends this, in include/sfx2/frame.hxx, class SfxFrame. |
| 190 | Its function is to notify the SvCompatWeakHdl when an SfxFrame object is deleted. |
| 191 | */ |
| 192 | template<typename T> |
| 193 | class SvCompatWeakBase |
| 194 | { |
| 195 | tools::SvRef< SvCompatWeakHdl<T> > _xHdl; |
| 196 | |
| 197 | public: |
| 198 | /** Does not use initializer due to compiler warnings, |
| 199 | because the lifetime of the _xHdl object can exceed the lifetime of this class. |
| 200 | */ |
| 201 | SvCompatWeakBase( T* pObj ) { _xHdl = new SvCompatWeakHdl<T>( pObj ); } |
| 202 | |
| 203 | ~SvCompatWeakBase() { _xHdl->ResetWeakBase(); } |
| 204 | |
| 205 | SvCompatWeakHdl<T>* GetHdl() { return _xHdl.get(); } |
| 206 | }; |
| 207 | |
| 208 | /** We only have one weak reference in LO, in include/sfx2/frame.hxx, class SfxFrameWeak. |
| 209 | */ |
| 210 | template<typename T> |
| 211 | class SAL_WARN_UNUSED__attribute__((warn_unused)) SvCompatWeakRef |
| 212 | { |
| 213 | tools::SvRef< SvCompatWeakHdl<T> > _xHdl; |
| 214 | public: |
| 215 | SvCompatWeakRef( ) {} |
| 216 | SvCompatWeakRef( T* pObj ) |
| 217 | { if( pObj ) _xHdl = pObj->GetHdl(); } |
| 218 | #if defined(__COVERITY__) |
| 219 | ~SvCompatWeakRef() COVERITY_NOEXCEPT_FALSE {} |
| 220 | #endif |
| 221 | SvCompatWeakRef& operator = ( T * pObj ) |
| 222 | { _xHdl = pObj ? pObj->GetHdl() : nullptr; return *this; } |
| 223 | bool is() const |
| 224 | { return _xHdl.is() && _xHdl->GetObj(); } |
| 225 | explicit operator bool() const { return is(); } |
| 226 | T* operator -> () const |
| 227 | { return _xHdl.is() ? _xHdl->GetObj() : nullptr; } |
| 228 | operator T* () const |
| 229 | { return _xHdl.is() ? _xHdl->GetObj() : nullptr; } |
| 230 | }; |
| 231 | |
| 232 | #endif |
| 233 | |
| 234 | /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ |