diff options
author | David Herrmann <dh.herrmann@gmail.com> | 2015-04-10 17:43:04 +0200 |
---|---|---|
committer | David Herrmann <dh.herrmann@gmail.com> | 2015-04-11 13:14:57 +0200 |
commit | ad922737f743a82ce95c5554c36e38016678e68e (patch) | |
tree | a775e432d54f0d35df9a88d3c811857373bd151f /src/shared/bus-label.c | |
parent | cfe561a456c9ce61579c8e1207f9a13faf050e8a (diff) |
bus: implement bus_label_unescape_n()
This is like bus_label_unescape() but takes a maximum length instead of
relying on NULL-terminated strings. This is highly useful to unescape
labels that are not at the end of a path.
Diffstat (limited to 'src/shared/bus-label.c')
-rw-r--r-- | src/shared/bus-label.c | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/src/shared/bus-label.c b/src/shared/bus-label.c index 9e9eaf4003..ccc9f2bf8e 100644 --- a/src/shared/bus-label.c +++ b/src/shared/bus-label.c @@ -63,34 +63,35 @@ char *bus_label_escape(const char *s) { return r; } -char *bus_label_unescape(const char *f) { +char *bus_label_unescape_n(const char *f, size_t l) { char *r, *t; + size_t i; assert_return(f, NULL); /* Special case for the empty string */ - if (streq(f, "_")) + if (l == 1 && *f == '_') return strdup(""); - r = new(char, strlen(f) + 1); + r = new(char, l + 1); if (!r) return NULL; - for (t = r; *f; f++) { - - if (*f == '_') { + for (i = 0, t = r; i < l; ++i) { + if (f[i] == '_') { int a, b; - if ((a = unhexchar(f[1])) < 0 || - (b = unhexchar(f[2])) < 0) { + if (l - i < 3 || + (a = unhexchar(f[i + 1])) < 0 || + (b = unhexchar(f[i + 2])) < 0) { /* Invalid escape code, let's take it literal then */ *(t++) = '_'; } else { *(t++) = (char) ((a << 4) | b); - f += 2; + i += 2; } } else - *(t++) = *f; + *(t++) = f[i]; } *t = 0; |