diff options
author | Maxime Ripard | 2016-07-05 10:26:41 +0200 |
---|---|---|
committer | Tom Rini | 2016-08-20 11:35:02 -0400 |
commit | 8e9685715bf78421da48a84040aba3801f66bf47 (patch) | |
tree | 0f5d37f8512180cc93367dbb9a90e2d206ffcb59 /lib/libfdt | |
parent | 6f5f92c60b7cc86bcfbd4acd310c5efbb6b68a8f (diff) |
libfdt: Add fdt_path_offset_namelen
Add a namelen variant of fdt_path_offset to retrieve the node offset using
only a fixed number of characters.
Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Diffstat (limited to 'lib/libfdt')
-rw-r--r-- | lib/libfdt/fdt_ro.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/lib/libfdt/fdt_ro.c b/lib/libfdt/fdt_ro.c index 6b737b211d2..9cc98db6e2b 100644 --- a/lib/libfdt/fdt_ro.c +++ b/lib/libfdt/fdt_ro.c @@ -145,10 +145,10 @@ int fdt_subnode_offset(const void *fdt, int parentoffset, * "foo/bar:option" and "bar:option/otheroption", both of which happen, so * first searching for either ':' or '/' does not work. */ -static const char *fdt_path_next_separator(const char *path) +static const char *fdt_path_next_separator(const char *path, int len) { - const char *sep1 = strchr(path, '/'); - const char *sep2 = strchr(path, ':'); + const void *sep1 = memchr(path, '/', len); + const void *sep2 = memchr(path, ':', len); if (sep1 && sep2) return (sep1 < sep2) ? sep1 : sep2; @@ -158,9 +158,9 @@ static const char *fdt_path_next_separator(const char *path) return sep2; } -int fdt_path_offset(const void *fdt, const char *path) +int fdt_path_offset_namelen(const void *fdt, const char *path, int namelen) { - const char *end = path + strlen(path); + const char *end = path + namelen; const char *p = path; int offset = 0; @@ -168,7 +168,7 @@ int fdt_path_offset(const void *fdt, const char *path) /* see if we have an alias */ if (*path != '/') { - const char *q = fdt_path_next_separator(path); + const char *q = fdt_path_next_separator(path, namelen); if (!q) q = end; @@ -181,14 +181,16 @@ int fdt_path_offset(const void *fdt, const char *path) p = q; } - while (*p) { + while (*p && (p < end)) { const char *q; while (*p == '/') p++; + if (*p == '\0' || *p == ':') return offset; - q = fdt_path_next_separator(p); + + q = fdt_path_next_separator(p, end - p); if (!q) q = end; |