nx-libs/nx-X11/programs/Xserver/randr/rrcrtc.c, branch update-workflows

Update Copyright years

2023-04-28T12:39:16+00:00

Addresses ArcticaProject/nx-libs#985

randr: free crtc->outputs on destroy

2017-12-07T06:20:55+00:00

Backported from X.org: commit fc7fb5bbe1c8f787e53500b9a2ca4af815f310d1 Author: Giuseppe Bilotta giuseppe.bilotta@gmail.com Date: Thu Nov 9 10:21:21 2017 +0100 randr: free crtc->outputs on destroy Reviewed-by: Adam Jackson Signed-off-by: Giuseppe Bilotta Backported-to-NX-by: Ulrich Sibiller Fixes ArcticaProject/nx-libs#558

randr: always realloc crtcs and outputs

2017-12-07T06:20:55+00:00

Backported from X.org: commit 16381d186e7c791031392ed8afcfd33009854e9e Author: Giuseppe Bilotta giuseppe.bilotta@gmail.com Date: Thu Nov 9 10:21:20 2017 +0100 randr: always realloc crtcs and outputs When the last crtc (resp. output) is destroyed, the rrScrPriv crtcs (resp. outputs) fields do not get cleared, which can lead to a situation where the private's numCrtcs (resp. numOutputs) field is zero, but the associated memory is still allocated. Just checking if numCrtcs (resp. numOutputs) is zero is thus not a good criteria to determine whetehr to use a realloc or a malloc. Since crtcs (resp. outputs) are NULL-initialized anyway, relying on numCrtcs (resp. numOutputs) is actually unnecessary, because reallocation of a NULL ptr is equivalent to a malloc anyway. Therefore, just use realloc() unconditionally, and ensure that the fields are properly initialized. Reviewed-by: Adam Jackson Signed-off-by: Giuseppe Bilotta Backported-to-NX-by: Ulrich Sibiller Fixes ArcticaProject/nx-libs#558

misc nx-X11/programs/Xserver/randr/: switch to using xreallocarray, since it has been ported now.

2017-03-03T11:20:20+00:00

misc nx-X11/programs/Xserver/{composite,dix,randr}/: switch to using reallocarray, since it has been ported now.

2017-03-03T11:20:20+00:00

Fix "uninitialised byte(s)" complaints from valgrind

2017-03-02T11:22:40+00:00

Xserver/randr: Regression fix introduced by c2b1f48. Re-introduce RegisterResourceName() calls after each CreateNewResourceType() call.

2017-02-04T14:30:00+00:00

Fixes ArcticaProject/nx-libs#309.

Per-file copyright notices: Update copyright information in file headers that NoMachine placed there own copyright statement in.

2016-07-05T23:07:24+00:00

Remove unneccesary casts from WriteToClient calls

2016-07-05T11:32:05+00:00

Casting return to (void) was used to tell lint that you intended to ignore the return value, so it didn't warn you about it. Casting the third argument to (char *) was used as the most generic pointer type in the days before compilers supported C89 (void *) (except for a couple places it's used for byte-sized pointer math). Signed-off-by: Alan Coopersmith Reviewed-by: Keith Packard Tested-by: Daniel Stone Backport to nx-libs: Mike Gabriel

Set event sequence number in WriteEventsToClient instead of at callers.

2016-07-05T06:38:36+00:00

TryClientEvents already did this; this commit just moves the assignment one level down so that no event source has to worry about sequence numbers. ...No event source, that is, except XKB, which inexplicably calls WriteToClient directly for several events. Signed-off-by: Jamey Sharp Reviewed-by: Julien Cristau Reviewed-by: Adam Jackson Backport to nx-libs: Mike Gabriel