[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-07-12 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

Armin Le Grand (CIB)  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |armin.le.gr...@me.com
   |desktop.org |
 Status|NEW |ASSIGNED

--- Comment #10 from Armin Le Grand (CIB)  ---
@Miklos: okay, added https://gerrit.libreoffice.org/#/c/57319/
Any comments...? People to add...?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-07-05 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

Xisco Faulí  changed:

   What|Removed |Added

Crash report or||["std::deque
   ||>,std::allocator > >
   ||>::push_back(std::unique_pt
   ||r > &&)"]

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-07-04 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

--- Comment #9 from Armin Le Grand (CIB)  ---
@Miklos: All correct - except me being a specialist for VCL ;-) Todo for
tomorrow...

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-07-04 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

--- Comment #8 from Miklos Vajna  ---
Armin, I'm afraid others already consider you an expert in vcl topics. ;-)

What about just asking on the dev mailing list, which has more subscribers than
this bug? Best if you push your proof of concept patch to gerrit, so there is
something concrete to discuss.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-07-04 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

Armin Le Grand (CIB)  changed:

   What|Removed |Added

 CC||vmik...@collabora.co.uk

--- Comment #7 from Armin Le Grand (CIB)  ---
Second solution works too (adding

mpControlData->mpReferenceDevice->isDisposed() ? _rTargetDevice :
*mpControlData->mpReferenceDevice

as 2nd parameter to ControlTextRenderer contructor.
But at every repaint this gets repeated and the
mpControlData->mpReferenceDevice stays disposed - also after dialog is
closed...

-> We somehow have a living Control with disposed
mpControlData->mpReferenceDevice. I do not know enough abou the current Window
implementation to say if this is okay or very bad.

@Miklos: Do you have an idea who might nowadays know enough about
Windows/Control vcl stuff to take a look at this one...?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-07-04 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

--- Comment #6 from Armin Le Grand (CIB)  ---
One working solution is to add

mpControlData->mpReferenceDevice->isDisposed()

to Control::DrawControlText last 'if' to avoid a ControlTextRenderer being
constructed. I will also try to construct a ControlTextRenderer by using
_rTargetDevice if mpReferenceDevice is disposed.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-07-04 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

--- Comment #5 from Armin Le Grand (CIB)  ---
mpOutDevStateStack is a std::unique_ptr and gets initialized
in OutputDevice::OutputDevice (mpOutDevStateStack.reset(new OutDevStateStack)).
It only gets reset (mpOutDevStateStack.reset()) in OutputDevice::dispose(),
thus I guess this must happen somehow.
When breaking in Control::DrawControlText at the end where ControlTextRenderer
gets constructed, mpControlData->mpReferenceDevice has a device, but it's
VclReferenceBase says indeed -> {mnRefCnt=2 mbDisposed=true }.
That mpControlData->mpReferenceDevice comes from Control directly, so that may
also be disposed already. Question is if this can be detected? There is already
Control::ImplCallEventListenersAndHandler where xThis->IsDisposed() is used
with 'VclPtr xThis(this)' (whyever).

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-07-04 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

--- Comment #4 from Armin Le Grand (CIB)  ---
Stack says:

vcllo.dll!std::_Container_base12::_Orphan_all() Line 217 (c:\Program Files
(x86)\Microsoft Visual Studio 14.0\VC\include\xutility:217)
vcllo.dll!std::_Deque_alloc
>,std::allocator >
> > >::_Orphan_all() Line 877 (c:\Program Files (x86)\Microsoft Visual Studio
14.0\VC\include\deque:877)
vcllo.dll!std::deque
>,std::allocator >
> >::push_back(std::unique_ptr >
&& _Val) Line 1160 (c:\Program Files (x86)\Microsoft Visual Studio
14.0\VC\include\deque:1160)
vcllo.dll!OutDevStateStack::push_back(OutDevState * p) Line 25
(c:\lo\work01\vcl\source\outdev\outdevstatestack.cxx:25)
vcllo.dll!OutputDevice::Push(PushFlags nFlags) Line 117
(c:\lo\work01\vcl\source\outdev\outdevstate.cxx:117)
vcllo.dll!vcl::ReferenceDeviceTextLayout::ReferenceDeviceTextLayout(const
Control & _rControl, OutputDevice & _rTargetDevice, OutputDevice &
_rReferenceDevice) Line 140 (c:\lo\work01\vcl\source\gdi\textlayout.cxx:140)
vcllo.dll!vcl::ControlTextRenderer::ControlTextRenderer(const Control &
_rControl, OutputDevice & _rTargetDevice, OutputDevice & _rReferenceDevice)
Line 336 (c:\lo\work01\vcl\source\gdi\textlayout.cxx:336)
vcllo.dll!Control::DrawControlText(OutputDevice & _rTargetDevice, const
tools::Rectangle & rRect, const rtl::OUString & _rStr, DrawTextFlags _nStyle,
std::vector > * _pVector,
rtl::OUString * _pDisplayText, const Size * i_pDeviceSize) Line 436
(c:\lo\work01\vcl\source\control\ctrl.cxx:436)
vcllo.dll!Button::ImplDrawAlignedImage(OutputDevice * pDev, Point & rPos, Size
& rSize, unsigned long nImageSep, DrawFlags nDrawFlags, DrawTextFlags
nTextStyle, tools::Rectangle * pSymbolRect, bool bAddImageSep) Line 271
(c:\lo\work01\vcl\source\control\button.cxx:271)
vcllo.dll!RadioButton::ImplDraw(OutputDevice * pDev, DrawFlags nDrawFlags,
const Point & rPos, const Size & rSize, const Size & rImageSize,
tools::Rectangle & rStateRect, tools::Rectangle & rMouseRect) Line 2066
(c:\lo\work01\vcl\source\control\button.cxx:2066)
vcllo.dll!RadioButton::ImplDrawRadioButton(OutputDevice & rRenderContext) Line
2165 (c:\lo\work01\vcl\source\control\button.cxx:2165)
vcllo.dll!RadioButton::Paint(OutputDevice & rRenderContext, const
tools::Rectangle & __formal) Line 2427
(c:\lo\work01\vcl\source\control\button.cxx:2427)
vcllo.dll!vcl::Window::ImplPaintToDevice(OutputDevice * i_pTargetOutDev, const
Point & i_rPos) Line 1448 (c:\lo\work01\vcl\source\window\paint.cxx:1448)
vcllo.dll!vcl::Window::PaintToDevice(OutputDevice * pDev, const Point & rPos,
const Size & __formal) Line 1521
(c:\lo\work01\vcl\source\window\paint.cxx:1521)
tklo.dll!VCLXWindow::draw(long nX, long nY) Line 2333
(c:\lo\work01\toolkit\source\awt\vclxwindow.cxx:2333)
tklo.dll!UnoControl::draw(long x, long y) Line 1014
(c:\lo\work01\toolkit\source\controls\unocontrol.cxx:1014)
drawinglayerlo.dll!drawinglayer::processor2d::VclPixelProcessor2D::processBasePrimitive2D(const
drawinglayer::primitive2d::BasePrimitive2D & rCandidate) Line 672
(c:\lo\work01\drawinglayer\source\processor2d\vclpixelprocessor2d.cxx:672)
drawinglayerlo.dll!drawinglayer::processor2d::BaseProcessor2D::process(const
drawinglayer::primitive2d::Primitive2DContainer & rSource) Line 72
(c:\lo\work01\drawinglayer\source\processor2d\baseprocessor2d.cxx:72)
drawinglayerlo.dll!drawinglayer::processor2d::BaseProcessor2D::process(const
drawinglayer::primitive2d::BasePrimitive2D & rCandidate) Line 49
(c:\lo\work01\drawinglayer\source\processor2d\baseprocessor2d.cxx:49)
drawinglayerlo.dll!drawinglayer::processor2d::VclPixelProcessor2D::processBasePrimitive2D(const
drawinglayer::primitive2d::BasePrimitive2D & rCandidate) Line 913
(c:\lo\work01\drawinglayer\source\processor2d\vclpixelprocessor2d.cxx:913)

Rendering of a RadioButton (probably unrelated to my changes, but you never
know).
Fact is that in OutputDevice::Push the mpOutDevStateStack is 'empty' ->
mpOutDevStateStack->push_back( pState ) crashes. I have no idea what
mpOutDevStateStack is or how it is managed. Taking a look...

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-06-30 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

Julien Nabet  changed:

   What|Removed |Added

 CC||serval2...@yahoo.fr

--- Comment #3 from Julien Nabet  ---
Created attachment 143227
  --> https://bugs.documentfoundation.org/attachment.cgi?id=143227=edit
Valgrind trace

On pc Debian x86-64 with master sources updated yesterday, it crashes my whole
laptop.
I could retrieve some info by using valgrind.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-06-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

Xisco Faulí  changed:

   What|Removed |Added

   Keywords||haveBacktrace

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-06-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

--- Comment #2 from Xisco Faulí  ---
Created attachment 143109
  --> https://bugs.documentfoundation.org/attachment.cgi?id=143109=edit
gdb backtrace

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 118377] CRASH: Writer crashes inserting a section

2018-06-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118377

Xisco Faulí  changed:

   What|Removed |Added

   Severity|normal  |critical
 Blocks||116979
 CC||armin.le.gr...@me.com,
   ||xiscofa...@libreoffice.org
   Keywords||bibisected, bisected,
   ||regression
 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEW
   Priority|medium  |highest

--- Comment #1 from Xisco Faulí  ---
Regression introduced by:

author  Armin Le Grand 2018-03-01 15:54:32
+0100
committer   Armin Le Grand   2018-04-06 22:29:02
+0200
commit  6c14c27c75a03e2363f2b363ddf0a6f2f46cfa91 (patch)
treee66f50adb222dbc1490b4df2d3c63541dad999ac
parent  e1b247a843c5eb850fe0079819239d9883412d38 (diff)
SOSAW080: Added first bunch of basic changes to helpers

Bisected with: bibisect-linux64-6.1

Adding Cc: to Armin Le Grand


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=116979
[Bug 116979] [META] Regression introduced by Aw080
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs