From 351670ca0e9f135b473dce52ff9e1ae76d55f787 Mon Sep 17 00:00:00 2001
From: "jan.nijtmans" <nijtmans@users.sourceforge.net>
Date: Tue, 13 Mar 2012 15:05:36 +0000
Subject: make sure that "burly wood" is not a valid color

---
 xlib/xcolors.c | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/xlib/xcolors.c b/xlib/xcolors.c
index e0e7f6e..cc5ad69 100644
--- a/xlib/xcolors.c
+++ b/xlib/xcolors.c
@@ -732,13 +732,20 @@ _strtoi64(const char *spec, char **p, int base)
 static int colorcmp(const char *spec, const char *pname) {
     int r;
     int c, d;
+    int notequal = 0;
     do {
 	d = *pname++;
+	c = (*spec == ' ');
+	if (c) {
+	    spec++;
+	}
 	if ((d >= 'A') && (d <= 'Z')) {
 	    d += 'a' - 'A';
-	    if (*spec == ' ') {
-		spec++;
-	    }
+	} else if (c) {
+	    /* A space doesn't match a lowercase, but we don't know
+	     * yet whether we should return a negative or positive
+	     * number. That depends on what follows. */
+	    notequal = 1;
 	}
 	c = *spec++;
 	if ((c >= 'A') && (c <= 'Z')) {
@@ -746,6 +753,11 @@ static int colorcmp(const char *spec, const char *pname) {
 	}
 	r = c - d;
     } while(!r && c);
+    if (!r && notequal) {
+	/* Strings are equal, but difference in spacings only. We should still
+	 * report not-equal, so "burly wood" is not a valid color */
+	r = 1;
+    }
     return r;
 }
 
-- 
cgit v0.12