errata

________________________________
De : Adrien Ollier
Envoyé : lundi 15 avril 2019 09:33
À : libreoffice@lists.freedesktop.org
Objet : bug 74702 - Issue with bool 
OutputDevice::IsNativeControlSupported(ControlType, ControlPart)

Hello everybody,

working on bug 
#74702<https://bugs.documentfoundation.org/show_bug.cgi?id=74702> led me to 
read file core/vcl/source/outdev/nativecontrols.cxx.

I think there is an issue here:

[cid:679fb8e4-9553-43c2-8c2b-0c2e2a238558]

If mpGraphics == nullptr and AcquireGraphics() == false, then the second if 
does not return false and we execute the instruction of the return statement 
but this will lead to a crash (because mpGraphics is false nullptr in this 
scenario).

If the second if is correct, then the last instruction sould be:
​return mpGraphics ? mpGraphics->IsSupported(nType, nPart) : false;

What do you think about that ? Do you agree with me ?

Adrien Ollier
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to