summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2025-02-24present: need to include geext.hEnrico Weigelt, metux IT consult1-0/+2
It's calling GERegisterExtension(), which is defined in geext.h Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1810>
2025-02-24misc.h: move out MAXEXTENSIONS to geext.cEnrico Weigelt, metux IT consult2-1/+2
The only user of it is geext.c, so no need to have that define in a public include file. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1810>
2025-02-24panoramix: don't install panoramiX.h and panoramiXsrv.hEnrico Weigelt, metux IT consult2-5/+0
Not used by any drivers/modules, so no need to keep them public. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1815>
2025-02-24dbe: fix byte swapping in SProcDbeSwapBuffers()Enrico Weigelt, metux IT consult1-1/+1
The loop forgot to move the SwapInfo pointer, so the same list entry gets swapped over and over again, while the remaining ones get ignored. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1822>
2025-02-23misc.h: move out checked_int64_(add|subtract)Enrico Weigelt, metux IT consult2-29/+29
These inline functions are used only in exactly one place, so no need to keep them in a public header. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1808>
2025-02-23present: fix prototype for present_select_input()Enrico Weigelt, metux IT consult1-1/+1
Wants XID instead of CARD32 for 2nd parameter. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1809>
2025-02-23Xext: geext: move geeint.h stuff into geext.cEnrico Weigelt, metux IT consult4-54/+9
Everything in here only used by geext.c, so no need to keep a separate header file for that. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext.h: fix missing include of Xfuncproto.hEnrico Weigelt, metux IT consult1-0/+2
Needed for _X_EXPORT. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext.h: don't include <dix-config.h>Enrico Weigelt, metux IT consult1-5/+0
Really no need to ever include it from here. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: move struct _GEExtension into geext.cEnrico Weigelt, metux IT consult2-6/+6
Not used anywhere outside geext implementation itself, so shouldn't be public at all. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: unexport GEExtensions[]Enrico Weigelt, metux IT consult2-4/+1
This field isn't used by anybody outside, nor should it be accessed directly. So better make it local. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: drop unused GEInitEvent()Enrico Weigelt, metux IT consult2-13/+0
This function isn't called by anybody, so no need to keep it. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: drop unused GEV() macroEnrico Weigelt, metux IT consult1-3/+0
Not used anywhere, so no need to keep it around any longer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: drop unused GEEXT() macroEnrico Weigelt, metux IT consult1-2/+0
Not used by anybody, so no need to keep it around any longer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: drop unused GEEXTIDX() macroEnrico Weigelt, metux IT consult1-3/+0
Not used anywhere, so no need to keep it around any longer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: drop unused GEMaskIsSet() macroEnrico Weigelt, metux IT consult1-5/+0
Not used anywhere, so no need to keep it around any longer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: drop unused GECLIENT() macroEnrico Weigelt, metux IT consult1-4/+0
Not used anywhere, so no need to keep it around any longer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: drop unused GEIsType() macroEnrico Weigelt, metux IT consult1-5/+0
Not used anywhere, so no need to keep it around any longer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23Xext: geext: drop unused GEEventFill() macroEnrico Weigelt, metux IT consult1-4/+0
It's not used anywhere, so no need to keep it around any longer. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1811>
2025-02-23os: drop unused TimerCheck()Enrico Weigelt, metux IT consult3-13/+2
This isn't used anywhere, so can be dropped. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1789>
2025-02-23os: unexport SetCriticalOutputPending()Enrico Weigelt, metux IT consult5-2/+4
Not used by any drivers/modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1814>
2025-02-23xwin: don't need NULL check before free()Enrico Weigelt, metux IT consult1-2/+1
free() is safe against NULL arguments. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1818>
2025-02-23glamor: don't need NULL check before free()Enrico Weigelt, metux IT consult1-8/+4
free() is safe against NULL arguments. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1818>
2025-02-23Xext: geext: fix regression in SProcGEQueryVersion()Enrico Weigelt, metux IT consult1-1/+1
Commit 36dd28129b5aa490aa6cbb7748a306193134339c had an ugly typo (probably broke on rebase - shame on myself) causing SProcGEQueryVersion() calling itself recursively. Obviously, ProcGEQueryVersion() needs to be called instead. Fixes: 36dd28129b5aa490aa6cbb7748a306193134339c Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1817>
2025-02-23present: need to include <X11/Xfuncproto.h>Enrico Weigelt, metux IT consult2-2/+5
Some headers making use of X_EXPORT macro, so Xfuncproto.h should be included explicitly, instead of silently relying any other include already doing that. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1813>
2025-02-23present: need to include dix-config.hEnrico Weigelt, metux IT consult8-0/+8
dix-config.h always needs to be included first, otherwise things can get messed up in really obscure ways, eg. certain types silently changing in size and causing mysterious crashes. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1813>
2025-02-20meson.build: enable VLA warningEnrico Weigelt, metux IT consult1-0/+2
variable length arrays can be dangerous, so better don't use them at all. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1819>
2025-02-20test: sync: don't use VLAEnrico Weigelt, metux IT consult1-7/+7
The array size is fixed anyways, so we can use a symbol instead of variable. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1819>
2025-02-20xfree86: modesetting: don't use VLAEnrico Weigelt, metux IT consult1-3/+8
even through this specific case is correct and safe, it's safer to remove all VLA usages and forbid them completely by compiler flag. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1819>
2025-02-19dri2: Protect against dri2ClientPrivate assertion failuresDoug Brown1-2/+12
If DRI2ScreenInit hasn't been called yet, DRI2Authenticate and DRI2CreateDrawable2 cause the X server to crash. This has been observed to happen on multiple modern Linux distros in various conditions, including QEMU and VMware VMs. Make these functions more robust in order to prevent the crash. This patch was originally provided by Bernhard Übelacker and expanded upon by Mark Wagner. Signed-off-by: Doug Brown <doug@schmorgal.com> Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1053 Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1534 Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1608>
2025-02-18misc.h: unexport compare_version()Enrico Weigelt, metux IT consult11-25/+33
Not used by any external modules, so no need to keep it in public header. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1807>
2025-02-18os: unexport CloseWellKnownConnections()Enrico Weigelt, metux IT consult6-2/+5
Not used by any modules, so no need to keep it public. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport ResetWellKnownSockets()Enrico Weigelt, metux IT consult2-2/+1
Not used by any modules (and not supposed to be), so should not be exported at all. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport CreateWellKnownSockets()Enrico Weigelt, metux IT consult2-2/+1
Not used by any modules (and not supposed to be), so should not be exported at all. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport NotifyParentProcess()Enrico Weigelt, metux IT consult2-2/+1
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport ResetOsBuffers()Enrico Weigelt, metux IT consult2-2/+1
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport FlushIfCriticalOutputPending()Enrico Weigelt, metux IT consult2-2/+1
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport FlushAllOutput()Enrico Weigelt, metux IT consult3-2/+2
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport ResetCurrentRequest()Enrico Weigelt, metux IT consult6-2/+13
Not used by any external modules, so no need to keep it in public API. Since it's used by DRI, still needs the _X_EXPORT flag, as long as DRI is a dynamically loaded module. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport InsertFakeRequest()Enrico Weigelt, metux IT consult2-4/+1
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport WriteFdToClient()Enrico Weigelt, metux IT consult5-2/+4
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport ReadRequestFromClient()Enrico Weigelt, metux IT consult3-3/+3
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport ListenOnOpenFD()Enrico Weigelt, metux IT consult3-2/+2
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport AddClientOnOpenFD()Enrico Weigelt, metux IT consult3-2/+2
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport ClientAuthorized()Enrico Weigelt, metux IT consult2-6/+5
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport AuthorizationIDOfClient()Enrico Weigelt, metux IT consult4-2/+3
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18os: unexport ClientIsLocal()Enrico Weigelt, metux IT consult4-4/+7
Not used by any modules, so no need to keep it exported. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1790>
2025-02-18ci: update freebsd builder imageEnrico Weigelt, metux IT consult1-3/+4
FreeBSD upstream removed image for 14.0 image, so we need to update. Also increasing image size, since the new images are too small. (needs update of ci-templates) Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1784>
2025-02-17xfree86: doc: update docs on XF86ModuleDataEnrico Weigelt, metux IT consult1-1/+1
Even though the order of these fields shouldn't change anytime soon, it's still better programming style to name'em explicitly. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1788>
2025-02-17xfree86: inputtest: use explicit field initializers for XF86ModuleDataEnrico Weigelt, metux IT consult1-13/+11
Even though the order of these fields shouldn't change anytime soon, it's still better programming style to name'em explicitly. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1788>