GitHub-Mirror/u-boot
2
0
Fork 0
mirror of https://github.com/Fishwaldo/u-boot.git synced 2025-03-21 14:41:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

env: Simplify the reverse_strstr() interface

Browse source 
The logic to find the whole matching name was split needlessly between
the reverse_strstr function and its caller. Fully contain it to make the
interface for calling it more consistent.

Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
This commit is contained in:
Joe Hershberger 2015-05-20 14:27:18 -05:00 • committed by Tom Rini
parent 7a0ad2cce3
commit 032ea185d6
1 changed files with 50 additions and 43 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

93
common/env_attr.c
View file

@ -109,33 +109,59 @@ int env_attr_walk(const char *attr_list,
} }
/* /*
* Search for the last matching string in another string with the option to * Search for the last exactly matching name in an attribute list
* start looking at a certain point (i.e. ignore anything beyond that point).
*/ */
static char *reverse_strstr(const char *searched, const char *search_for, static int reverse_name_search(const char *searched, const char *search_for,
const char *searched_start) const char **result)
{ {
char *result = NULL; int result_size = 0;
const char *cur_searched = searched;
if (*search_for == '\0') if (result)
return (char *)searched; *result = NULL;
for (;;) { if (*search_for == '\0') {
char *match = strstr(searched, search_for); if (result)
*result = searched;
/* return strlen(searched);
* Stop looking if no new match is found or looking past the
* searched_start pointer
*/
if (match == NULL || (searched_start != NULL &&
match + strlen(search_for) > searched_start))
break;
result = match;
searched = match + 1;
} }
return result; for (;;) {
const char *match = strstr(cur_searched, search_for);
const char *prevch;
const char *nextch;
/* Stop looking if no new match is found */
if (match == NULL)
break;
prevch = match - 1;
nextch = match + strlen(search_for);
/* Skip spaces */
while (*prevch == ' ' && prevch >= searched)
prevch--;
while (*nextch == ' ')
nextch++;
/* Start looking past the current match so last is found */
cur_searched = match + 1;
/* Check for an exact match */
if (match != searched &&
*prevch != ENV_ATTR_LIST_DELIM &&
prevch != searched - 1)
continue;
if (*nextch != ENV_ATTR_SEP &&
*nextch != ENV_ATTR_LIST_DELIM &&
*nextch != '\0')
continue;
if (result)
*result = match;
result_size = strlen(search_for);
}
return result_size;
} }
/* /*
@ -145,6 +171,7 @@ static char *reverse_strstr(const char *searched, const char *search_for,
int env_attr_lookup(const char *attr_list, const char *name, char *attributes) int env_attr_lookup(const char *attr_list, const char *name, char *attributes)
{ {
const char *entry = NULL; const char *entry = NULL;
int entry_len;
if (!attributes) if (!attributes)
/* bad parameter */ /* bad parameter */
@ -153,32 +180,12 @@ int env_attr_lookup(const char *attr_list, const char *name, char *attributes)
/* list not found */ /* list not found */
return -EINVAL; return -EINVAL;
entry = reverse_strstr(attr_list, name, NULL); entry_len = reverse_name_search(attr_list, name, &entry);
while (entry != NULL) {
const char *prevch = entry - 1;
const char *nextch = entry + strlen(name);
/* Skip spaces */
while (*prevch == ' ')
prevch--;
while (*nextch == ' ')
nextch++;
/* check for an exact match */
if ((entry == attr_list ||
*prevch == ENV_ATTR_LIST_DELIM) &&
(*nextch == ENV_ATTR_SEP ||
*nextch == ENV_ATTR_LIST_DELIM ||
*nextch == '\0'))
break;
entry = reverse_strstr(attr_list, name, entry);
}
if (entry != NULL) { if (entry != NULL) {
int len; int len;
/* skip the name */ /* skip the name */
entry += strlen(name); entry += entry_len;
/* skip spaces */ /* skip spaces */
while (*entry == ' ') while (*entry == ' ')
entry++; entry++;