tasn pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=48579d8e609d65a0e7e8ffde4b6e08bb6bced9ff

commit 48579d8e609d65a0e7e8ffde4b6e08bb6bced9ff
Author: Tom Hacohen <t...@stosb.com>
Date:   Tue Dec 8 12:55:06 2015 +0000

    Colorselector: Fix out of bound read.
    
    high should be the last item of the array, not the length of the array.
    As part of this change, I also changed the code to use the better way of
    getting the array size.
    
    CID1339827
---
 src/lib/elm_colorselector.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/elm_colorselector.c b/src/lib/elm_colorselector.c
index d919e7b..8d69c63 100644
--- a/src/lib/elm_colorselector.c
+++ b/src/lib/elm_colorselector.c
@@ -203,7 +203,7 @@ _get_color_name (unsigned int R, unsigned int G, unsigned 
int B, unsigned int A)
 
    if (A != 255) return NULL;
    low = 0;
-   high = sizeof(_color_name)/ sizeof(_color_name[0]);
+   high = EINA_C_ARRAY_LENGTH(_color_name) - 1;
    while (low <= high)
      {
         mid = (low + high) / 2;

-- 


Reply via email to