cppcheck sept 18 redux

Ben Klein shacklein at gmail.com
Mon Sep 21 08:29:21 CDT 2009


2009/9/21 chris ahrendt <celticht32 at yahoo.com>:
> False positive ticket created for:
>
> [/home/cahrendt/wine-git/dlls/wineps.drv/init.c:270]: (error) Possible
> null pointer dereference: dmW - otherwise it is redundant to check if
> dmW is null at line 272
>
> what about the others? or the suggestion of adding an additional int to
> the array?

The code is not using the ca[5] local variable, but the ca[1] defined
in the cs_t struct. The question remains, how exactly does
FIELD_OFFSET work, and does it end up dereferencing ca[5]?
Furthermore, since this is in a test, is this expected in the first
place? :)



More information about the wine-devel mailing list