[Libreoffice] [PATCH] possible null-dereferencing found by cppcheck

Caolán McNamara caolanm at redhat.com
Tue Jan 18 06:55:06 PST 2011


On Tue, 2011-01-18 at 07:45 -0700, Tor Lillqvist wrote:
> >> Or am I missing something... 
> > Yes, we have a miserable trap of fall-through:
> 
> No, I did see the fall-through. But if pDlg is NULL (a bit earlier), we
>  don't enter the block that contains the switch with the fall-through,
>  where the other (hopefully only other possible) case does set pArgs

Those abstract dialogs should never fail in practice, nevertheless I've
added an early return for that case.

C.



More information about the LibreOffice mailing list