]> git.ipfire.org Git - people/ms/u-boot.git/commitdiff
api: Use hashtable function for API_env_enum
authorEmmanuel Vadot <manu@bidouilliste.com>
Mon, 26 Dec 2016 17:57:56 +0000 (18:57 +0100)
committerTom Rini <trini@konsulko.com>
Fri, 20 Jan 2017 14:15:24 +0000 (09:15 -0500)
The current code can loop undefinitly as it doesn't parse
correctly the env data.
Since the env is an hashtable, use the hashtable function for
the API_ENV_ENUM api call.

Signed-off-by: Emmanuel Vadot <manu@bidouilliste.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
api/api.c

index 8a1433af78a98357f8e7ac58773c2b32a49aebad..c368511704e5e9877757a49b2e8e9cf2256c2b95 100644 (file)
--- a/api/api.c
+++ b/api/api.c
@@ -495,45 +495,47 @@ static int API_env_set(va_list ap)
  */
 static int API_env_enum(va_list ap)
 {
-       int i, n;
-       char *last, **next;
+       int i, buflen;
+       char *last, **next, *s;
+       ENTRY *match, search;
+       static char *var;
 
        last = (char *)va_arg(ap, unsigned long);
 
        if ((next = (char **)va_arg(ap, uintptr_t)) == NULL)
                return API_EINVAL;
 
-       if (last == NULL)
-               /* start over */
-               *next = ((char *)env_get_addr(0));
-       else {
-               *next = last;
-
-               for (i = 0; env_get_char(i) != '\0'; i = n + 1) {
-                       for (n = i; env_get_char(n) != '\0'; ++n) {
-                               if (n >= CONFIG_ENV_SIZE) {
-                                       /* XXX shouldn't we set *next = NULL?? */
-                                       return 0;
-                               }
-                       }
-
-                       if (envmatch((uchar *)last, i) < 0)
-                               continue;
-
-                       /* try to get next name */
-                       i = n + 1;
-                       if (env_get_char(i) == '\0') {
-                               /* no more left */
-                               *next = NULL;
-                               return 0;
-                       }
-
-                       *next = ((char *)env_get_addr(i));
-                       return 0;
+       if (last == NULL) {
+               var = NULL;
+               i = 0;
+       } else {
+               var = strdup(last);
+               s = strchr(var, '=');
+               if (s != NULL)
+                       *s = 0;
+               search.key = var;
+               i = hsearch_r(search, FIND, &match, &env_htab, 0);
+               if (i == 0) {
+                       i = API_EINVAL;
+                       goto done;
                }
        }
 
+       /* match the next entry after i */
+       i = hmatch_r("", i, &match, &env_htab);
+       if (i == 0)
+               goto done;
+       buflen = strlen(match->key) + strlen(match->data) + 2;
+       var = realloc(var, buflen);
+       snprintf(var, buflen, "%s=%s", match->key, match->data);
+       *next = var;
        return 0;
+
+done:
+       free(var);
+       var = NULL;
+       *next = NULL;
+       return i;
 }
 
 /*