[Libreoffice-bugs] [Bug 47068] EDITING: Report-Builder crashes when moving a graphic over another field

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=47068

Caolán McNamara <caolanm at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sbergman at redhat.com

--- Comment #9 from Caolán McNamara <caolanm at redhat.com> 2012-04-03 08:04:21 PDT ---
stoc/source/inspect/introspection.cxx the lifecycle of ImplIntrospectionAdapter
and ImplIntrospectionAccess looks troublesome to me.

Seeing as ImplIntrospectionAdapter has a mpAccess of ImplIntrospectionAccess*
presumably the ImplIntrospectionAdapter should add a reference to its mpAccess
ImplIntrospectionAccess so that it can enforce the existance of mpAccess so
that in


reportdesign/source/core/sdr/UndoEnv.cxx

                    if ( m_pImpl->m_xIntrospection.is() )
                    {
                        Reference< XIntrospectionAccess > xIntrospection(
                            m_pImpl->m_xIntrospection->inspect( makeAny(
_rEvent.Source ) ),
                            UNO_SET_THROW
                        );
                        rObjectInfo.xPropertyIntrospection.set(
xIntrospection->queryAdapter( XPropertySet::static_type() ), UNO_QUERY_THROW );
                    }

the result of queryAdapter, which still references xIntrospection as its
mpParent, will retain a reference to it so that it doesn't point to junk, as
happens at the moment with xIntrospection goes out of scope.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.