[REVIEWED][3-5] misc fixes for -3-5

Michael Meeks michael.meeks at suse.com
Mon Feb 20 09:13:11 PST 2012


Hi Michael,

On Sat, 2012-02-18 at 00:19 +0100, Michael Stahl wrote:
> some misc commits, mostly crashes fixed while loading ~2000 presumed
> ODT documents (looking at actual patches reveals expecting people to set
> bug attachment mime types correctly is unrealistic), proposed for
> libreoffice-3-5:

	Nice work ! :-)

> http://cgit.freedesktop.org/libreoffice/core/commit/?id=cd24d2524f2c2da51f982015f1b027003b192808
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=43cb934dda9af45a83748c775f274bbd0fdcd67a

	Lovely, pushed.

> boost crash (don't know if that happens in product builds as well):
> this basically syncs the binfilter code with core:
> 
> http://cgit.freedesktop.org/libreoffice/binfilter/commit/?id=508835a5ea8d385b3a740b769d75cdd4c3362809

	picked.

> gdb thingy:
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=89b68f682e40f1bdc6c967bf38308504fae70d82

	ditto - no harm for the product to be able to debug it better :-)

> assertions triggered by silly code:
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=6c3e8f9d19a0392a817c1b5692421ed0972a3b7e
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=c1f723fd53e8c41b80285d11a149e0044f580e2f

	Fun; pushed.

> pyuno valgrind:
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=29177b1d04b845e8e6bf8edc8beb34b76a95d9c3

	Urk, that looks really flaky, the code is somewhat amazing there -
surely we're not creating a new thread for each python object that is
unref'd ? Indeed the premise seems to be that python should be using a
recursive mutex but that's madness surely ;-) ... Do we not attach and
detach the python mutex as/when we call in and out of the python
interpreter ourselves ? or ... well nasty anyhow. Patch doesn't look as
if it can do any harm though, so I pushed it ;-)

> crash in Impress accessibility code:
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=cdfa3bd5e4c62dc88158a4c6c4908404baf1f529
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=e72e8dcd332474321439eb266528eb7e5fc8809f

	Neither fills me with joy - but ... ;-) I'm still reeling from the
python code so ... pushed these too ;-)

	Thanks for the patch set !

		Michael.

-- 
michael.meeks at suse.com  <><, Pseudo Engineer, itinerant idiot



More information about the LibreOffice mailing list