From d97748e9b6bfebc5eca58a0247cd941ae5529081 Mon Sep 17 00:00:00 2001
From: atomizer <danila.gerasimov@gmail.com>
Date: Sat, 23 Apr 2011 23:57:00 +0400
Subject: [PATCH] clamp r,g,b to 0..255, alpha to 0..1 (assuming everything is
 >= 0)

Signed-off-by: Tj Holowaychuk <tj@vision-media.ca>
---
 src/color.cc | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/color.cc b/src/color.cc
index e03a827..bb768d5 100644
--- a/src/color.cc
+++ b/src/color.cc
@@ -20,14 +20,17 @@
  */
 
 #define CHANNEL(NAME) \
+   c = 0; \
    if (*str >= '0' && *str <= '9') { \
      do { \
-       NAME *= 10; \
-       NAME += *str++ - '0'; \
+       c *= 10; \
+       c += *str++ - '0'; \
      } while (*str >= '0' && *str <= '9'); \
    } else { \
      return 0; \
    } \
+   if (c > 255) c = 255; \
+   NAME = c; \
    while (' ' == *str || ',' == *str) str++;
 
 /*
@@ -313,6 +316,7 @@ rgba_from_rgb_string(const char *str, short *ok) {
     str += 4;
     WHITESPACE;
     uint8_t r = 0, g = 0, b = 0;
+    int c;
     CHANNEL(r);
     CHANNEL(g);
     CHANNEL(b);
@@ -331,15 +335,15 @@ rgba_from_rgba_string(const char *str, short *ok) {
     str += 5;
     WHITESPACE;
     uint8_t r = 0, g = 0, b = 0;
+    int c;
     float a = 0;
     CHANNEL(r);
     CHANNEL(g);
     CHANNEL(b);
-    // TODO: less strict
-    if ('1' == *str) {
+    if (*str >= '1' && *str <= '9') {
       a = 1;
     } else {
-      if ('0' == *str) a = 0, ++str;
+      if ('0' == *str) ++str;
       if ('.' == *str) {
         ++str;
         float n = .1;