Unlikely to practically happen, but still safer to just check for sure. A simple zero-value test doesn't cost us much (on modern CPUs perhaps not even a full clock cycle). | ../composite/compalloc.c: In function ‘compRedirectWindow’: | ../composite/compalloc.c:167:35: warning: dereference of NULL ‘pClient’ [CWE-476] [-Wanalyzer-null-dereference] | 167 | ccw->id = FakeClientID(pClient->index); | | ~~~~~~~^~~~~~~ | ../composite/compalloc.c: In function ‘compUnredirectWindow’: | ../composite/compalloc.c:331:75: warning: dereference of NULL ‘pClient’ [CWE-476] [-Wanalyzer-null-dereference] | 331 | if (ccw->update == update && dixClientIdForXID(ccw->id) == pClient->index) { | | ~~~~~~~^~~~~~~ Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> |
||
---|---|---|
.. | ||
compalloc.c | ||
compext.c | ||
compinit.c | ||
compint.h | ||
compositeext.h | ||
compositeext_priv.h | ||
compoverlay.c | ||
compwindow.c | ||
meson.build |