<html> <head> <base href="https://bugs.documentfoundation.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - REPORTBUILDER - Trying to edit chart leads to crash of whole LO" href="https://bugs.documentfoundation.org/show_bug.cgi?id=117159#c12">Comment # 12</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - REPORTBUILDER - Trying to edit chart leads to crash of whole LO" href="https://bugs.documentfoundation.org/show_bug.cgi?id=117159">bug 117159</a> from <a class="email" href="mailto:mikekaganski@hotmail.com" title="Mike Kaganski <mikekaganski@hotmail.com>"> Mike Kaganski</a> <pre>The problem is that chart::InternalDataProvider does not implement css::beans::XPropertySet, which makes PropertyHandler::inspect (called from DataProviderHandler::inspect) to assign nullptr to the m_xComponent, and then dereferencing it inside DataProviderHandler::getPropertyValue segfaults. The code in FormComponentPropertyHandler methods (e.g., impl_getPropertyValue_throw) seems to dereference m_xComponent unconditionally - that seems odd. But I have no idea which should be the fix - if the interface is needed (and then it should throw when the interface is null) or the dereference should be guarded by checks (because it may be null for a good reason) (and I am not working on this - just placing it here for whoever has the idea).</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>