RE: bug 74702 - Issue with bool OutputDevice::IsNativeControlSupported(ControlType, ControlPart)

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