frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
xserver/dix
History
Enrico Weigelt, metux IT consult 65c0c35197 (!1967) dix: move props into WindowRec and fix potential NULL deref 
The pointer to the window properties is currently inside the WindowOptional
structure, which may or may not exist at any given time. Thus, before accessing
those fields, at least need to check whether it exists, potentially need to
create it first.

Since a pointer is small (in relation to WindowRec) and windows having properties
is a pretty common, we can make our life much simpler here by moving the pointer
directly into WindowRec, so we don't need extra WindowOptionalRec allocation.

This also fixes an analyzer warning on potential NULL dereference issue:

| ../dix/property.c: In function ‘dixChangeWindowProperty’:
|../dix/property.c:343:37: warning: dereference of NULL ‘*pWin.optional’ [CWE-476] [-Wanalyzer-null-dereference]
|  343 |         pProp->next = pWin->optional->userProps;
|      |                       ~~~~~~~~~~~~~~^~~~~~~~~~~

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
 		2025-05-22 17:34:41 +02:00
..
.gitignore Fix spelling/wording issues 2020-07-05 13:07:33 -07:00
BuiltInAtoms R6.6 is the Xorg base-line 2003-11-14 15:54:54 +00:00
atom.c (!2003) dix: replace AtomError() by better messages 2025-05-22 17:34:23 +02:00
callback_priv.h dix: fix duplicate typedef of CallbackListPtr 2024-09-01 22:32:49 +00:00
closestr.h dix: move closestr.h into dix directory 2024-04-16 01:42:39 +00:00
color.c dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
colormap.c rename old symbol PANORAMIX to XINERAMA 2025-02-06 15:51:27 +00:00
colormap_priv.h dix: move colormap flags into colormap_priv.h and rename them 2024-09-01 22:06:50 +00:00
cursor.c os: move BUG_*() macros to own private header 2025-02-17 19:32:48 +00:00
cursor_priv.h dix: unexport GetSpritePosition() 2024-09-02 16:43:29 +00:00
devices.c dix: Dequeue pending events on frozen device on removal 2025-02-25 11:43:01 +01:00
dispatch.c (!2002) dix: drop unused SetVendorRelease() 2025-05-22 17:34:34 +02:00
dispatch.h dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
display.c dix: add getter for display name 2025-02-11 19:13:01 +01:00
dix_priv.h drop not needed includes of geext.h 2025-02-24 20:19:55 +00:00
dixfonts.c (!1967) dix: protect doPolyText() from potential NULL dereference 2025-05-22 17:34:41 +02:00
dixgrabs_priv.h dix: unexport non-public functions from dixgrabs.h and document prototypes 2024-10-10 13:50:57 +00:00
dixstruct_priv.h include: move private defs to dixstruct_priv.h 2024-04-30 00:47:38 +00:00
dixutils.c dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
enterleave.c dix: fix erroneous BUG_RETURN check 2025-03-21 11:42:41 +10:00
enterleave.h dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
eventconvert.c dix: limit checks to MAX_VALUATORS when generating Xi events 2024-10-28 05:38:25 +00:00
eventconvert.h dix: unexport eventconvert.h functions 2024-03-11 12:26:44 +01:00
events.c (!1989) panoramix: replace FOR_NSCREENS() by FOR_NSCREENS_BACKWARD(i) 2025-05-22 17:34:35 +02:00
exevents_priv.h (!1986) XI: unexport XIPropToInt() and XIPropToFloat() 2025-05-22 17:34:36 +02:00
extension.c dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
gc.c dix: use dixDestroyPixmap() instead of direct driver call 2025-02-12 17:48:30 +01:00
gc_priv.h dix: unexport SetClipRects() 2024-04-15 23:10:31 +00:00
generate-atoms (!2003) dix: replace AtomError() by better messages 2025-05-22 17:34:23 +02:00
gestures.c os: move BUG_*() macros to own private header 2025-02-17 19:32:48 +00:00
getevents.c os: move BUG_*() macros to own private header 2025-02-17 19:32:48 +00:00
globals.c dix: add getter for display name 2025-02-11 19:13:01 +01:00
glyphcurs.c dix: use dixDestroyPixmap() instead of direct driver call 2025-02-12 17:48:30 +01:00
grabs.c dix: unexport non-public functions from dixgrabs.h and document prototypes 2024-10-10 13:50:57 +00:00
input_priv.h dix: fix warning on redefinition of typedefs 2025-05-18 17:35:31 +00:00
inpututils.c os: move BUG_*() macros to own private header 2025-02-17 19:32:48 +00:00
main.c dix: keep a ref to the rootCursor 2025-02-25 11:43:01 +01:00
meson.build dix: add getter for display name 2025-02-11 19:13:01 +01:00
pixmap.c dix: use dixDestroyPixmap() instead of direct driver call 2025-02-12 17:48:30 +01:00
privates.c dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
property.c (!1967) dix: move props into WindowRec and fix potential NULL deref 2025-05-22 17:34:41 +02:00
property_priv.h dix: unexport DeleteAllWindowProperties() 2024-09-02 17:50:47 +00:00
protocol.txt drop remains of DMX 2024-03-05 16:57:52 +01:00
ptrveloc.c (!1986) XI: unexport XIPropToInt() and XIPropToFloat() 2025-05-22 17:34:36 +02:00
ptrveloc_priv.h dix: fix duplicate typedef of MotionTracker and *MotionTrackerPtr 2024-09-01 22:32:49 +00:00
region.c dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
registry.c dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
registry_priv.h dix: unexport XREGISTRY_UNKNOWN define 2024-03-03 23:20:06 +00:00
resource.c rename old symbol PANORAMIX to XINERAMA 2025-02-06 15:51:27 +00:00
screenint_priv.h dix: fix duplicate typedef of *ScreenPtr 2024-09-01 22:32:49 +00:00
selection.c dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
stubmain.c Allow DDX to provide a main() 2013-07-23 23:56:58 +01:00
swaprep.c dix: drop unnecessary check on HAVE_DIX_CONFIG_H 2024-10-10 13:38:31 +00:00
swapreq.c misc.h: drop LengthRestB() macro 2025-02-17 19:25:14 +00:00
tables.c dix: tables.c should include header that defines InitialVector 2025-04-19 17:02:15 +00:00
touch.c os: move BUG_*() macros to own private header 2025-02-17 19:32:48 +00:00
window.c (!1967) dix: move props into WindowRec and fix potential NULL deref 2025-05-22 17:34:41 +02:00