summaryrefslogtreecommitdiff
path: root/composite/compalloc.c
diff options
context:
space:
mode:
authorVille Syrjälä <ville.syrjala@nokia.com>2010-12-20 16:33:36 +0200
committerVille Syrjälä <ville.syrjala@nokia.com>2011-05-04 19:01:01 +0300
commitce9eff9e4c30deb16f059ed84b436d37da45d5d5 (patch)
tree1fa3f63310a1b21ef03ba717052241a09e44dae6 /composite/compalloc.c
parent5cb31cd0cbf83fff5f17a475e7b0e45246b19bf3 (diff)
composite: Call ValidateGC after ChangeGC
ChangeGC changes the GC, so ValidateGC should be called after it, not before. Also pass NullClient instead of serverClient to ChangeGC() since we know the changed values to be valid, and setting serverClient->errorValue seems pointless anyway. Signed-off-by: Ville Syrjälä <ville.syrjala@nokia.com> Reviewed-by: Keith Packard <keithp@keithp.com> Reviewed-by: Adam Jackson <ajax@redhat.com> Reviewed-by: Daniel Stone <daniel@fooishbar.org>
Diffstat (limited to 'composite/compalloc.c')
-rw-r--r--composite/compalloc.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/composite/compalloc.c b/composite/compalloc.c
index 7164c0d3c..2b6864519 100644
--- a/composite/compalloc.c
+++ b/composite/compalloc.c
@@ -536,9 +536,8 @@ compNewPixmap (WindowPtr pWin, int x, int y, int w, int h, Bool map)
{
ChangeGCVal val;
val.val = IncludeInferiors;
-
+ ChangeGC (NullClient, pGC, GCSubwindowMode, &val);
ValidateGC(&pPixmap->drawable, pGC);
- ChangeGC (serverClient, pGC, GCSubwindowMode, &val);
(*pGC->ops->CopyArea) (&pParent->drawable,
&pPixmap->drawable,
pGC,