aboutsummaryrefslogtreecommitdiff
path: root/nx-X11/programs/Xserver/hw/nxagent/Rootless.c
Commit message (Collapse)AuthorAgeFilesLines
* fix formatting and spelling in various filesUlrich Sibiller2021-01-161-1/+1
|
* Rootless.c: prevent PVS errorUlrich Sibiller2021-01-151-1/+2
| | | | "V547 Expression 'wmHints.input == 1' is always true."
* Rootless.c: do not import private propertiesUlrich Sibiller2020-05-071-0/+36
| | | | | | | | | | In rootless mode some properties are private (or internal or adminitrative). They are only required for the windows on the real X server side that represent nxagent's windows. Those properties should never be cloned from there to the the nxagent windows so we filter them. Fixes ArcticaProject/nx-libs#920
* Rootless.c: improve TEST outputUlrich Sibiller2020-05-071-12/+16
| | | | more output, correct format specifiers
* Rootless.c: add some commentsUlrich Sibiller2020-05-071-0/+11
|
* Rootless.c: fix common realloc mistakeUlrich Sibiller2020-05-071-2/+1
| | | | | | | As reported by static analyzer: (error) Common realloc mistake: 'ptr' nulled but not freed upon failure Attributes ArcticaProject/nx-libs#905
* nxagent: make all traps BooleansUlrich Sibiller2020-05-071-2/+2
| | | | and use True/False instead of 1/0
* Rootless.c: undo overzealous scope improvementUlrich Sibiller2020-05-071-3/+3
| | | | It is better to create the values array once and not on every iteration..
* Rootless.c: fix wring datatype being usedUlrich Sibiller2020-01-081-2/+2
| | | | | | | | | Using XlibAtom here is wrong. On a 64bit system the size of this type is 8 bytes (64bit). At this stage we do not read Xlib datatypes but wire datatypes. Over the wire an Atom is always 32bit, so we need to use CARD32 here. Fixes ArcticaProject/nx-libs#894
* Rootless.c: fix wordingUlrich Sibiller2020-01-061-1/+1
|
* nxagent: Use XlibAtom data type where appropriateUlrich Sibiller2020-01-051-7/+7
| | | | | | | | XlibAtom should be used wherever remote atoms are handled. This is important as Xlib data types and server data types of the same name do not always have identical sizes. See also https://lists.freedesktop.org/archives/xorg-devel/2015-August/047245.html
* Rootless.c: rewrite loop to allow for scope improvementsUlrich Sibiller2019-11-151-17/+10
| | | | | make i a local loop variable everywhere by changing the while loop to a for loop.
* Rootless.c: fix linefeeds when printing listsUlrich Sibiller2019-11-151-5/+5
|
* Rootless.c: silence compiler warning about wrong parmeter sizesUlrich Sibiller2019-11-151-5/+3
|
* Rootless.c: disable unused functionUlrich Sibiller2019-11-151-0/+2
|
* Rootless.c: silence compiler warnings about wrong fromat specifiersUlrich Sibiller2019-11-151-15/+15
|
* Rootless.c: Fatal error if malloc failsUlrich Sibiller2019-11-151-1/+5
|
* Rootless.c: cleanup function prototypesUlrich Sibiller2019-11-151-10/+8
|
* Rootless.c: reformat commentsUlrich Sibiller2019-11-151-34/+26
|
* Rootless.c: scope improvementsUlrich Sibiller2019-11-151-58/+38
|
* Rootless.c: use __func__ instead of hardcoded function nameUlrich Sibiller2019-11-151-78/+74
|
* Rootless.c: use designated initializersUlrich Sibiller2019-11-151-15/+9
| | | | This fixes a possible unitialized struct variable.
* Rootless.c: use SAFE_free and SAFE_XFreeUlrich Sibiller2019-08-061-9/+6
|
* various scope improvementsUlrich Sibiller2019-06-191-26/+12
|
* Rootless.c: improve TEST/WARN/DEBUG outoutUlrich Sibiller2019-06-191-36/+36
| | | | by adding [] around values as almost everywhere
* Revert "nxagent: rework Bool handling"Ulrich Sibiller2019-02-151-2/+2
| | | | | | | | | | | | This reverts commit 16cd2bbe1c4425e3fa557f9ca0723aa94a50b071. It turned out that I had missed some of the Booleans being actually tristate variables. I think I can fix this (they do not need to be tristate) but I revert this for now to get back to a working state (there are reports about non-working fullscreen mode and reconnect problems). Refers to ArcticaProject/nx-libs#772
* nxagent: rework Bool handlingUlrich Sibiller2019-02-111-2/+2
| | | | drop "== False", "== 0", "== True" and "== 0" for nxagentOptions and Traps
* hw/nxagent/: Use <function>(void) rather than <function>().Mike Gabriel2018-02-261-4/+4
|
* Rootless.c: Fix wrong function name in WARNING outputUlrich Sibiller2018-02-051-1/+1
|
* Make sure XQueryTree results are freedUlrich Sibiller2017-12-071-1/+1
| | | | Fixes ArcticaProject/nx-libs#585
* Events.c: move variable to inner scopeUlrich Sibiller2017-12-071-2/+1
|
* Rootless.c: improve debugging outputUlrich Sibiller2017-03-101-4/+4
|
* Rootless.c: improve malloc failure handlingUlrich Sibiller2017-03-101-2/+34
|
* Rootless.c: fix possible memleakUlrich Sibiller2017-03-101-0/+1
|
* Rootless.c: Fix wrongly negated comparisonsUlrich Sibiller2017-03-101-2/+2
|
* nxagent: move validateString as inline func to Utils.hUlrich Sibiller2017-03-031-0/+1
|
* Xserver/dix/atom.c (et al.): Constify atom name strings.Mike Gabriel2017-03-021-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Inspired by X.org commits: commit 08093c25a91c07ab8af7cece9bba738b827cfd1b Author: Alan Coopersmith <alan.coopersmith@oracle.com> Date: Mon Oct 24 23:16:30 2011 -0700 Convert some malloc + strncpy pairs into strndup calls Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com> commit 816b79dd061e9839cec94a4986a7820b70ca8a7f Author: Mikhail Gusarov <dottedmag@dottedmag.net> Date: Thu May 13 03:45:21 2010 +0700 Remove useless casts Signed-off-by: Mikhail Gusarov <dottedmag@dottedmag.net> Reviewed-by: Keith Packard <keithp@keithp.com> This PR ships a tiny change in MakeAtom, that we adopted. We did not adopt the full commit. commit 5623c27700b7b23a8dbbd8c8f45e5d4fa0c667e3 Author: Alan Coopersmith <alan.coopersmith@sun.com> Date: Mon Feb 2 19:25:14 2009 -0800 Constify atom name strings Changes MakeAtom to take a const char * and NameForAtom to return them, since many callers pass pointers to constant strings stored in read-only ELF sections. Updates in-tree callers as necessary to clear const mismatch warnings introduced by this change. Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com> Acked-by: Peter Hutterer <peter.hutterer@who-t.net> Backported-to-NX-by: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
* Naming change: Security*Access -> Dix*AccessMike Gabriel2017-02-201-5/+5
| | | | | | | | | | | | Backported from X.org: commit 6c46645cfc1afda8aeabfe0ed4d9342673b702f1 Author: Eamon Walsh <ewalsh@tycho.nsa.gov> Date: Thu Dec 14 14:45:42 2006 -0500 Naming change: Security*Access -> Dix*Access Backported-to-NX-by: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
* nxcompext: Move code into Xserver subtree, as nxcompext requires Xserver ↵Mike Gabriel2016-11-151-1/+1
| | | | | | includes at build time. Fixes ArcticaProject/nx-libs#276.
* Per-file copyright notices: Update copyright information in file headers ↵Mike Gabriel2016-07-061-7/+15
| | | | that NoMachine placed there own copyright statement in.
* nx-X11/programs/Xserver: Drop {X,x}alloc() macros, use malloc() instead.Mike Gabriel2016-07-021-1/+1
|
* nx-X11/programs/Xserver: Drop {X,x}free() macros, use free() instead.Mike Gabriel2016-07-021-4/+4
| | | | Fixes ArcticaProject/nx-libs#105
* Move each screen's root-window pointer into ScreenRec.Mike Gabriel2016-06-211-6/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | Backported from X.org: commit e7fae9ecc42ab5e73b89117722dbf4117d928f9a Author: Jamey Sharp <jamey@minilop.net> Date: Sat May 22 00:26:28 2010 -0700 Move each screen's root-window pointer into ScreenRec. Many references to the WindowTable array already had the corresponding screen pointer handy, which meant they usually looked like "WindowTable[pScreen->myNum]". Adding a field to ScreenRec instead of keeping this information in a parallel array simplifies those expressions, and eliminates a MAXSCREENS-sized array. Since dix uses this data, a screen private entry isn't appropriate. xf86-video-dummy currently uses WindowTable, so it needs to be updated to reflect this change. Signed-off-by: Jamey Sharp <jamey@minilop.net> Reviewed-by: Tiago Vignatti <tiago.vignatti@nokia.com> Tested-by: Tiago Vignatti <tiago.vignatti@nokia.com> (i686 GNU/Linux) Backport to nx-libs: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
* nx-X11/programs/Xserver: Include nxcomp{,ext,shad} headers like one would do ↵Mike Gabriel2016-06-131-1/+1
| | | | with system-wide shared libraries.
* hw/nxagent/Rootless.c: Type cast type Atom to type long unsigned int before ↵Mike Gabriel2016-05-021-1/+1
| | | | | | | | | | | | | handing it over to %lu in fprintf(). Amends the following compiler warnings: ``` Rootless.c: In function ‘nxagentExportProperty’: Rootless.c:775:21: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘Atom’ [-Wformat=] "Property [%lu] too long.\n", propertyX); ^ ```
* Replace 'pointer' type with 'void *'Keith Packard2015-12-281-1/+1
| | | | | | | | | | This lets us stop using the 'pointer' typedef in Xdefs.h as 'pointer' is used throughout the X server for other things, and having duplicate names generates compiler warnings. Signed-off-by: Keith Packard <keithp@keithp.com> Reviewed-by: Eric Anholt <eric@anholt.net> Rebased against NX: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
* Imported nxagent-3.5.0-2.tar.gznxagent/3.5.0-2Reinhard Tartler2011-10-101-4/+58
| | | | | | | | Summary: Imported nxagent-3.5.0-2.tar.gz Keywords: Imported nxagent-3.5.0-2.tar.gz into Git repository
* Imported nxagent-3.4.0-8.tar.gznxagent/3.4.0-8Reinhard Tartler2011-10-101-9/+34
| | | | | | | | Summary: Imported nxagent-3.4.0-8.tar.gz Keywords: Imported nxagent-3.4.0-8.tar.gz into Git repository
* Imported nxagent-3.4.0-5.tar.gznxagent/3.4.0-5Reinhard Tartler2011-10-101-1/+1
| | | | | | | | Summary: Imported nxagent-3.4.0-5.tar.gz Keywords: Imported nxagent-3.4.0-5.tar.gz into Git repository
* Imported nxagent-3.4.0-3.tar.gznxagent/3.4.0-3Reinhard Tartler2011-10-101-88/+11
| | | | | | | | Summary: Imported nxagent-3.4.0-3.tar.gz Keywords: Imported nxagent-3.4.0-3.tar.gz into Git repository