env: Add regex support to env_attrs

Allow the features that use env_attrs to specify regexs for the name

Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Joe Hershberger 2015-05-20 14:27:20 -05:00 committed by Tom Rini
parent cca98fd6aa
commit bdf1fe4e68
3 changed files with 101 additions and 2 deletions

8
README
View File

@ -4142,6 +4142,10 @@ Configuration Settings:
list, simply add an entry for the same variable name to the list, simply add an entry for the same variable name to the
".flags" variable. ".flags" variable.
If CONFIG_REGEX is defined, the variable_name above is evaluated as a
regular expression. This allows multiple variables to define the same
flags without explicitly listing them for each variable.
- CONFIG_ENV_ACCESS_IGNORE_FORCE - CONFIG_ENV_ACCESS_IGNORE_FORCE
If defined, don't allow the -f switch to env set override variable If defined, don't allow the -f switch to env set override variable
access flags. access flags.
@ -5540,6 +5544,10 @@ override any association in the static list. You can define
CONFIG_ENV_CALLBACK_LIST_DEFAULT to a list (string) to define the CONFIG_ENV_CALLBACK_LIST_DEFAULT to a list (string) to define the
".callbacks" environment variable in the default or embedded environment. ".callbacks" environment variable in the default or embedded environment.
If CONFIG_REGEX is defined, the variable_name above is evaluated as a
regular expression. This allows multiple variables to be connected to
the same callback without explicitly listing them all out.
Command Line Parsing: Command Line Parsing:
===================== =====================

View File

@ -11,6 +11,7 @@
#include <linux/linux_string.h> #include <linux/linux_string.h>
#else #else
#include <common.h> #include <common.h>
#include <slre.h>
#endif #endif
#include <env_attr.h> #include <env_attr.h>
@ -109,6 +110,89 @@ int env_attr_walk(const char *attr_list,
return 0; return 0;
} }
#if defined(CONFIG_REGEX)
struct regex_callback_priv {
const char *searched_for;
char *regex;
char *attributes;
};
static int regex_callback(const char *name, const char *attributes, void *priv)
{
int retval = 0;
struct regex_callback_priv *cbp = (struct regex_callback_priv *)priv;
struct slre slre;
char regex[strlen(name) + 3];
/* Require the whole string to be described by the regex */
sprintf(regex, "^%s$", name);
if (slre_compile(&slre, regex)) {
struct cap caps[slre.num_caps + 2];
if (slre_match(&slre, cbp->searched_for,
strlen(cbp->searched_for), caps)) {
free(cbp->regex);
cbp->regex = malloc(strlen(regex) + 1);
if (cbp->regex) {
strcpy(cbp->regex, regex);
} else {
retval = -ENOMEM;
goto done;
}
free(cbp->attributes);
cbp->attributes = malloc(strlen(attributes) + 1);
if (cbp->attributes) {
strcpy(cbp->attributes, attributes);
} else {
retval = -ENOMEM;
free(cbp->regex);
cbp->regex = NULL;
goto done;
}
}
} else {
printf("Error compiling regex: %s\n", slre.err_str);
retval = EINVAL;
}
done:
return retval;
}
/*
* Retrieve the attributes string associated with a single name in the list
* There is no protection on attributes being too small for the value
*/
int env_attr_lookup(const char *attr_list, const char *name, char *attributes)
{
if (!attributes)
/* bad parameter */
return -EINVAL;
if (!attr_list)
/* list not found */
return -EINVAL;
struct regex_callback_priv priv;
int retval;
priv.searched_for = name;
priv.regex = NULL;
priv.attributes = NULL;
retval = env_attr_walk(attr_list, regex_callback, &priv);
if (retval)
return retval; /* error */
if (priv.regex) {
strcpy(attributes, priv.attributes);
free(priv.attributes);
free(priv.regex);
/* success */
return 0;
}
return -ENOENT; /* not found in list */
}
#else
/* /*
* Search for the last exactly matching name in an attribute list * Search for the last exactly matching name in an attribute list
*/ */
@ -219,3 +303,4 @@ int env_attr_lookup(const char *attr_list, const char *name, char *attributes)
/* not found in list */ /* not found in list */
return -ENOENT; return -ENOENT;
} }
#endif

View File

@ -31,12 +31,18 @@
#define SPLASHIMAGE_CALLBACK #define SPLASHIMAGE_CALLBACK
#endif #endif
#ifdef CONFIG_REGEX
#define ENV_DOT_ESCAPE "\\"
#else
#define ENV_DOT_ESCAPE
#endif
/* /*
* This list of callback bindings is static, but may be overridden by defining * This list of callback bindings is static, but may be overridden by defining
* a new association in the ".callbacks" environment variable. * a new association in the ".callbacks" environment variable.
*/ */
#define ENV_CALLBACK_LIST_STATIC ENV_CALLBACK_VAR ":callbacks," \ #define ENV_CALLBACK_LIST_STATIC ENV_DOT_ESCAPE ENV_CALLBACK_VAR ":callbacks," \
ENV_FLAGS_VAR ":flags," \ ENV_DOT_ESCAPE ENV_FLAGS_VAR ":flags," \
"baudrate:baudrate," \ "baudrate:baudrate," \
"bootfile:bootfile," \ "bootfile:bootfile," \
"loadaddr:loadaddr," \ "loadaddr:loadaddr," \