office-gobmx/include/tools/weakbase.h
Tomaž Vajngerl 8ca89dc44a fix reseting WeakReference when moving a ref to another ref.
In ViewIteratorImpl::Reverse we std::move a reference, to a local
variable, however the reference was not reset correctly, which
caused a crash.

The issue is that a mpWeakConnection in WeakReference always
needs to be non-null as shown in reset(reference_type* pReference)
method. In the move constructor of WeakReference, we just std::move
the reference like:

mpWeakConnection = std::move(rWeakRef.mpWeakConnection);

This means rWeakRef.mpWeakConnection will be reset to null, which
is not what is expected, so what is missing is to instantiate
rWeakRef.mpWeakConnection by calling reset(nullptr) or using the
added reset() method.

This also adds a test for LOKit PDF search as this is the case
where the crash has been reproduced. It happens because of a
call to:

... std::move(maPosition.mxObject);

in the method ViewIteratorImpl::Reverse()

Change-Id: I43692554ba4f6231d00091269b7c902ab5cbc11f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95995
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2020-06-10 15:58:16 +02:00

159 lines
5.4 KiB
C++

/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
/*
* This file is part of the LibreOffice project.
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/.
*
* This file incorporates work covered by the following license notice:
*
* Licensed to the Apache Software Foundation (ASF) under one or more
* contributor license agreements. See the NOTICE file distributed
* with this work for additional information regarding copyright
* ownership. The ASF licenses this file to you under the Apache
* License, Version 2.0 (the "License"); you may not use this file
* except in compliance with the License. You may obtain a copy of
* the License at http://www.apache.org/licenses/LICENSE-2.0 .
*/
#ifndef INCLUDED_TOOLS_WEAKBASE_H
#define INCLUDED_TOOLS_WEAKBASE_H
#include <sal/types.h>
#include <rtl/ref.hxx>
#include <tools/toolsdllapi.h>
/** the template classes in this header are helper to implement weak references
to implementation objects that are not refcounted.
THIS IS NOT THREADSAFE
Use this only to have 'safe' pointers to implementation objects that you
don't own but that you reference with a pointer.
Example:
class ImplClass : public tools::WeakBase< ImplClass >
{
~ImplClass() { clearWeek(); } // not needed but safer, see method description
...
};
class UserClass
{
tools::WeakReference< ImplClass > mxWeakRef;
UserClass( ImplClass* pOjbect ) : mxWeakRef( pObject ) {}
DoSomething()
{
if( mxWeakRef.is() )
mxWeakRef->DoSomethingMore();
}
};
*/
namespace tools
{
class WeakBase;
/** private connection helper, do not use directly */
struct WeakConnection
{
sal_Int32 mnRefCount;
WeakBase* mpReference;
WeakConnection() : mnRefCount( 0 ), mpReference( nullptr ) {};
WeakConnection( WeakBase* pReference ) : mnRefCount( 0 ), mpReference( pReference ) {};
void acquire() { mnRefCount++; }
void release() { mnRefCount--; if( mnRefCount == 0 ) delete this; }
};
/** template implementation to hold a weak reference to an instance of type reference_type */
template <class reference_type>
class SAL_WARN_UNUSED WeakReference
{
public:
/** constructs an empty reference */
inline WeakReference();
/** constructs a reference with a pointer to a class derived from WeakBase */
inline WeakReference( reference_type* pReference );
/** constructs a reference from another reference */
inline WeakReference( const WeakReference< reference_type >& rWeakRef );
/** move a reference from another reference */
inline WeakReference( WeakReference< reference_type >&& rWeakRef );
/** returns true if the reference object is not null and still alive */
inline bool is() const;
/** returns true if the reference object is not null and still alive */
operator bool() const { return is(); }
/** returns the pointer to the reference object or null */
inline reference_type * get() const;
/** sets this reference to the given object or null */
inline void reset( reference_type* pReference );
/** resets this reference to null */
inline void reset();
/** returns the pointer to the reference object or null */
inline reference_type * operator->() const;
/** returns a ref to the reference object */
inline reference_type& operator*() const;
/** returns true if this instance references pReferenceObject */
inline bool operator== (const reference_type * pReferenceObject) const;
/** returns true if this instance and the given weakref reference the same object */
inline bool operator== (const WeakReference<reference_type> & handle) const;
/** only needed for using this class with stl containers */
inline bool operator!= (const WeakReference<reference_type> & handle) const;
/** only needed for using this class with stl containers */
inline bool operator< (const WeakReference<reference_type> & handle) const;
/** only needed for using this class with stl containers */
inline bool operator> (const WeakReference<reference_type> & handle) const;
/** the assignment operator */
inline WeakReference<reference_type>& operator= (const WeakReference<reference_type> & handle);
/** the move assignment operator */
inline WeakReference<reference_type>& operator= (WeakReference<reference_type> && handle);
private:
rtl::Reference<WeakConnection> mpWeakConnection;
};
/** derive your implementation classes from this class if you want them to support weak references */
class TOOLS_DLLPUBLIC WeakBase
{
template<typename T> friend class WeakReference;
public:
WeakBase() {}
virtual ~WeakBase();
/** clears the reference pointer in all living weak references for this instance.
Further created weak references will also be invalid.
You should call this method in the d'tor of your derived classes for an early
invalidate of all living weak references while your object is already inside
it d'tor.
*/
inline void clearWeak();
private:
inline WeakConnection* getWeakConnection();
rtl::Reference<WeakConnection> mpWeakConnection;
};
}
#endif
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */