bae6cbc8ca
include: move private defs to dixstruct_priv.h
...
Public server module API shouldn't be clobbered with private definitions,
thus move them out to dixstruct_priv.h
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1289 >
2024-04-30 00:47:38 +00:00
620b678cfe
os: secure-rpc: make build option tristate
...
Add support `auto` mode, which only enables secure-rpc when
dependencies are met.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1441 >
2024-04-17 23:38:25 +00:00
339717bac6
os: secure-rpc: check struct authdes_cred
...
Some platforms (eg. NetBSD) don't have DES support in rpc anymore, so we
should check, in order to prevent weird, hard to understand build breaks.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1441 >
2024-04-17 23:38:25 +00:00
d2d3f4a700
os: define SECURE_RPC locally instead of global config header
...
Since this define is only used inside os subdir, move the definition there
and out of the global config header. This also makes some further steps,
e.g. checking whether it's supported, a litle bit easier.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1441 >
2024-04-17 23:38:25 +00:00
c17c527b9a
os: unexport ForceClockId()
...
This function is only intended for DDX'es (currently just Xwayland) that
need to force using another clock. Really shouldn't be called by drivers
or extensions.
Fixes: a779fda224https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1333 >
2024-04-17 16:20:54 -07:00
a0b69624f7
os: unexport ListenToAllClients()
...
It's just called by DIX, not by any drivers/modules, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1387 >
2024-04-16 18:23:27 -07:00
d18ef33099
os: unexport OnlyListenToOneClient()
...
Not used by any drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1385 >
2024-04-16 18:13:11 -07:00
4c1795701c
os: unexport MakeClientGrabPervious() and MakeClientGrabImpervious()
...
Not used by any drivers, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1384 >
2024-04-16 17:52:14 -07:00
be4c8444eb
os: unexport Os*() functions
...
These aren't called (and suited for being called) by drivers,
thus drop them from the public module API.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1381 >
2024-04-16 14:20:30 +02:00
3aa62ca6f0
os: fix missing include of misc.h in busfault.h
...
busfault.h is using FALSE, but didn't include misc.h, so it really
depends on include order whether it works.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1381 >
2024-04-16 14:19:59 +02:00
76d01e9bf6
os: unexport OsVendorVErrorFProc pointer
...
This pointer allows a DDX to install it's own error print handler. It's really
only intended for DDXes, thus no need to have it exported to modules.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1369 >
2024-04-15 23:59:38 +00:00
1999785fa9
os: move over osLookupColor to dix
...
This function is only used by DIX and not os-specific at all.
So move it over to DIX and give it a more fitting name.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1373 >
2024-04-15 16:19:30 -07:00
6cb599f59c
os: unexport OsLookupColor()
...
It's an internal function, only used by DIX, not by drivers and shouldn't
have been exported in the first place.
Fixes: 49f77fff14https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1373 >
2024-04-15 16:19:06 -07:00
3b7a63e6ae
os: unexport xthread_sigmask
...
This function isn't used by drivers and there's currently no need to do so,
thus keep it out of the public module API.
Fixes: 30ac756798https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1368 >
2024-04-15 22:49:02 +00:00
cf5fab0bbe
os: unexport AccessUsingXdmcp()
...
This function is only used internally for xdmcp auth handling, not used
in modules/drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
dd21a80219
os: unexport ChangeAccessControl()
...
this function isn't used by modules/drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
3f00c1b6cf
os: unexport host control functions
...
these aren't used by modules/drivers, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
5a9b885118
os: unexport authorization management functions
...
Those aren't used by modules, thus no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
bed778ee60
os: unexport LocalAccessScopeUser()
...
this function is only used by wayland ddx, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
e621081fa3
os: unexport EnableLocalUser() and DisableLocalUser()
...
these aren't used by module, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:55:22 -07:00
54441ff98a
os: unexport EnableLocalAccess() and DisableLocalAccess()
...
these aren't used by modules, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:44:36 -07:00
b00fdd482c
os: unexport EnableLocalHost() and DisableLocalHost()
...
These aren't used by modules, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:44:36 -07:00
5f19eab1ee
os: unexport local client creds handling
...
these functions aren't used by modules, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:44:34 -07:00
cbc9f557e5
os: unexport set_font_authorizations()
...
This function is only used by DIX, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1344 >
2024-04-15 14:42:45 -07:00
389b528203
os: unexport command line args handling functions
...
These functions shouldn't be called by drivers or extensions, thus
shouldn't be exported. Also moving it to separate header, so the
already huge ones aren't cluttered with even more things.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1334 >
2024-04-15 21:21:40 +00:00
dae76bbaaf
include: move busfault.h out of public include dir
...
this header is isn't public, so it's more appropriate, where the actual
code is, which is the os/ directory.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1337 >
2024-04-15 21:10:15 +00:00
3843a643e0
os: unexport audit functions
...
These aren't used by any drivers, so no need to keep them around in the
public driver API.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1326 >
2024-04-15 19:54:34 +00:00
eac10cf1fa
os: connection: drop obsolete define Pid_t
...
Clean up a yet some more stale leftovers.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1260 >
2024-04-15 18:02:11 +00:00
537293ff4e
os: move SELinux enforcement state to the extension
...
Those fields are better off in the extension instead of the OS layer.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1328 >
2024-03-25 19:40:04 +00:00
8dc82a13a9
os: rpc: fix type mismatch
...
fix warning:
> ../os/rpcauth.c:159:16: warning: result of comparison of constant -1 with expression of type 'XID' (aka 'unsigned int') is always false [-Wtautological-constant-out-of-range-compare]
> if (rpc_id == ~0L)
> ~~~~~~ ^ ~~~
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1431 >
2024-03-21 22:57:40 +00:00
dfa606d5a4
os: fix missing include of <errno.h>
...
It's much cleaner to always include directly what one needs,
instead of relying on very indirect including.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1435 >
2024-03-21 17:32:30 +01:00
924939c886
Revert "Fix missing includes of <errno.h>"
...
Removing errno from xf86_OSlib.h breaks the xf86-input-mouse driver
build. And xf86_OSlib.h itself relies on errno anyway in the SYSCALL
macro provided by this header.
This reverts commit f6a367102chttps://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1423 >
2024-03-19 00:33:26 +00:00
ccfa7e9f2e
Fix missing include of <sys/wait.h>
...
Instead of relying on indirect includes, it's much cleaner if everybody
includes directly what he needs.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1419 >
2024-03-18 23:02:45 +00:00
f6a367102c
Fix missing includes of <errno.h>
...
It's much cleaner to always include directly what one needs,
instead of relying on very indirect including.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1416 >
2024-03-18 22:58:32 +00:00
b335a0fa20
dix: unexport CloseDownClient()
...
This function isn't used by drivers, so no need to export it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1383 >
2024-03-13 00:47:36 +00:00
02ba47f09d
os: move out Format*() functions
...
Move out the Format*() functions to separate source and header.
These aren't exported, thus no module API change.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1312 >
2024-03-09 17:34:51 +00:00
040e41c7e9
dix: unexport global variables
...
Those aren't used by drivers, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1375 >
2024-03-09 17:23:43 +00:00
30b403b0f0
os: unexport Fopen(), Fclose(), Popen(), Pclose(), System()
...
These functions are just used for reading auth file or calling xkbcomp while
dropping privileges, in case the Xserver is started as unprivileged user
with suid-root. Thus, shouldn't be used (and aren't used) by drivers.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1370 >
2024-03-09 17:18:46 +00:00
15d3c1a6f1
os: move os_move_fd() out of public API
...
This function isn't used by any driver and doesn't seem to be useful for them,
thus move it out of the public module API, in order to tidy it up a bit.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1366 >
2024-03-09 17:12:46 +00:00
1bdbe1cb3f
os: fix missing X11/Xdefs.h include in os/osdep.h
...
osdep.h needs Bool type, which is defined X11/Xdefs.h.
For now it works, since Xdefs usually is already included somewhere
else, but that's an unreliable programming styles which quickly hits
us when changing include order.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1366 >
2024-03-09 17:12:46 +00:00
934b771f7a
os: unexport AutoResetServer()
...
This is an internal function, not used and not supposed to be used by
any drivers, so shouldn't have been exported in the first place.
Fixes: 49f77fff14https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1372 >
2024-03-07 23:10:17 +00:00
b1ce5543f1
dix: unexport workqueue functions
...
These aren't used by drivers, so no need to export them.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1376 >
2024-03-07 22:57:53 +00:00
c3255fbc1c
os: drop unneeded DDXOSVERRORF conditional
...
This conditional practically only controls whether we have an pointer,
where DDX can plug in it's own VErrorF() handler (currently only xwin
doing that). The cost of having it even when DDX doesn't use it, is
really negligible: it's just one pointer and an extra non-null check
on it per VErrorF() call - a very cold path. Strangely, xwin has extra
Getting rid of this unnecessary complexity that really hasn't any
practical gain.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1329 >
2024-03-03 23:15:14 +00:00
40c5d39c55
include: move xsha1.h to os/
...
This header is never exported and belongs to OS layer, thus no need to have it
in include/ directory, where all the public ones are - better off under os/.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1350 >
2024-03-03 23:11:05 +00:00
77f9792911
os: simplify win32 uname()
...
Just define struct utsname and a tiny uname() function instead of
cluttering the code with ifdef's.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1295 >
2024-03-03 23:02:54 +00:00
7bd19a9580
xwin: replace ZeroMemory()
...
replace Windows specific ZeroMemory (macro just calling memset())
by static initialization, calloc() and memset().
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1295 >
2024-03-03 23:02:54 +00:00
de34a20dc3
os: fix mising prototype / include on WIN32 builds
...
[90/383] Compiling C object os/liblibxlibc.a.p/strcasestr.c.obj
359../os/strcasestr.c:45:1: warning: no previous prototype for ‘xstrcasestr’ [-Wmissing-prototypes]
360 45 | xstrcasestr(const char *s, const char *find)
361 | ^~~~~~~~~~~
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1295 >
2024-03-03 23:02:54 +00:00
0706c03f9d
os: fix unused variable on WIN32 build
...
../os/access.c:443:18: warning: unused variable ‘n’ [-Wunused-variable]
420 443 | register int n;
421 | ^
422
[120/383] Compiling C object os/liblibxserver_os.a.p/connection.c.obj
431../os/connection.c:137:14: warning: ‘ParentProcess’ defined but not used [-Wunused-variable]
432 137 | static Pid_t ParentProcess;
433 | ^~~~~~~~~~~~~
../os/connection.c:132:13: warning: ‘RunFromSmartParent’ defined but not used [-Wunused-variable]
435 132 | static Bool RunFromSmartParent; /* send SIGUSR1 to parent process */
436 | ^~~~~~~~~~~~~~~~~~
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1295 >
2024-03-03 23:02:54 +00:00
43a6d4eb61
os: fix unused variable on non-IPv6 build
...
[119/383] Compiling C object os/liblibxserver_os.a.p/access.c.obj
415../os/access.c: In function ‘DefineSelf’:
416../os/access.c:468:26: warning: unused variable ‘inet6addr’ [-Wunused-variable]
417 468 | struct sockaddr_in6 *inet6addr;
418 | ^~~~~~~~~
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1295 >
2024-03-03 23:02:54 +00:00
5d2f675ca8
os: utils: drop obsolete System() on non-win32 targets
...
This function is only used on WIN32 targets, so we can drop the *nix
implementation.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1295 >
2024-03-03 23:02:54 +00:00
Enrico Weigelt, metux IT consult
Peter Hutterer