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

--- Comment #10 from Noel Power <[email protected]> 2012-03-01 03:53:54 PST ---
==23747==  Address 0x1f6d1de8 is 440 bytes inside a block of size 2,856 free'd
==23747==    at 0x4C2562E: operator delete(void*) (in
/usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==23747==    by 0x25FEA3B9: svxform::XFormsPage::~XFormsPage()
(datanavi.cxx:360)
==23747==    by 0x25FF4395:
svxform::DataNavigatorWindow::MenuSelectHdl(MenuButton*) (datanavi.cxx:1884)
==23747==    by 0x25FF3046:
svxform::DataNavigatorWindow::LinkStubMenuSelectHdl(void*, void*)
(datanavi.cxx:1650)
==23747==    by 0x9223DC7: Link::Call(void*) const (link.hxx:140)
==23747==    by 0x92B673D: MenuButton::Select() (menubtn.cxx:218)
==23747==    by 0x92B5FFC: MenuButton::ImplExecuteMenu() (menubtn.cxx:86)
==23747==    by 0x92B65F3: MenuButton::MouseButtonDown(MouseEvent const&)
(menubtn.cxx:186)


more detail seems to have been introduced by commit
f7f47366ea78a71853fcaca2fa402bef667d9cce

I see no reason why we should delete a page we are using ( especially without
actually deleting it from the tab container thing, I suggest reverting this
part 


@@ -1885,6 +1885,7 @@ namespace svxform
                             m_aTabCtrl.SetPageText( nId, sNewName );
                             bIsDocModified = true;
                         }
+                        delete(pPage);
                     }
                     break;
                 }

at least

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to