commit 0ad71a97c6061e3b12d2308bd43e02dfeeb63db4
author Mathias Agopian <mathias@google.com> Wed May 11 20:37:47 2011 -0700
committer Mathias Agopian <mathias@google.com> Wed May 11 20:37:47 2011 -0700
tree 7bf57adeda6dc0496cc01599d824b79abafe52a9
parent 8747ce61e9012b18de46b9be0fc3ff5338dec4f9

fix (Again) adding OES postfix when looking for gl functions

Change-Id: Ib14723ed5355fdc423226ec20a32e26fe7dd68fe

opengl/libs/EGL/Loader.cpp

diff --git a/opengl/libs/EGL/Loader.cpp b/opengl/libs/EGL/Loader.cpp
index 75dec4a..da26229 100644
--- a/opengl/libs/EGL/Loader.cpp
+++ b/opengl/libs/EGL/Loader.cpp
@@ -179,7 +179,8 @@
         __eglMustCastToProperFunctionPointerType* curr, 
         getProcAddressType getProcAddress) 
 {
-    char scrap[256];
+    const size_t SIZE = 256;
+    char scrap[SIZE];
     while (*api) {
         char const * name = *api;
         __eglMustCastToProperFunctionPointerType f = 
@@ -191,7 +192,7 @@
         if (f == NULL) {
             // Try without the OES postfix
             ssize_t index = ssize_t(strlen(name)) - 3;
-            if ((index>0 && (index<255)) && (!strcmp(name+index, "OES"))) {
+            if ((index>0 && (index<SIZE-1)) && (!strcmp(name+index, "OES"))) {
                 strncpy(scrap, name, index);
                 scrap[index] = 0;
                 f = (__eglMustCastToProperFunctionPointerType)dlsym(dso, scrap);
@@ -200,13 +201,9 @@
         }
         if (f == NULL) {
             // Try with the OES postfix
-            ssize_t size  = ssize_t(strlen(name));
-            ssize_t index = size - 3;
-            if ((index>0 && (index<252)) && (strcmp(name+index, "OES"))) {
-                strncpy(scrap, name, sizeof(scrap) - 1);
-                scrap[size] = 0;
-                strncat(scrap, "OES", sizeof(scrap) - 1);
-                scrap[size + 3] = 0;
+            ssize_t index = ssize_t(strlen(name)) - 3;
+            if (index>0 && strcmp(name+index, "OES")) {
+                snprintf(scrap, SIZE, "%sOES", name);
                 f = (__eglMustCastToProperFunctionPointerType)dlsym(dso, scrap);
                 //LOGD_IF(f, "found <%s> instead", scrap);
             }