diff options
author | June McEnroe <june@causal.agency> | 2021-06-09 14:38:28 -0400 |
---|---|---|
committer | June McEnroe <june@causal.agency> | 2021-06-09 17:56:43 -0400 |
commit | 16ae4d6f42a8f3502e5c596568ec7a9bd69ce909 (patch) | |
tree | b81a82f3b98b632f8e81d4e0da78e42c81ee7266 | |
parent | Use seprintf for reserialize (diff) | |
download | pounce-16ae4d6f42a8f3502e5c596568ec7a9bd69ce909.tar.gz pounce-16ae4d6f42a8f3502e5c596568ec7a9bd69ce909.zip |
Use seprintf for snip, removing strlcpyn
-rw-r--r-- | client.c | 30 |
1 files changed, 9 insertions, 21 deletions
diff --git a/client.c b/client.c index a1750bc..ebc40f4 100644 --- a/client.c +++ b/client.c @@ -449,34 +449,22 @@ static int wordcmp(const char *line, size_t i, const char *word) { : (int)len - (int)strlen(word); } -static size_t strlcpyn(char *dst, const char *src, size_t cap, size_t len) { - if (len < cap) { - memcpy(dst, src, len); - dst[len] = '\0'; - } else { - memcpy(dst, src, cap - 1); - dst[cap - 1] = '\0'; - } - return len; -} - // s/..(..)../\1/g -static char *snip(char *dst, size_t cap, const char *src, const regex_t *regex) { - size_t len = 0; +static char * +snip(char *dst, size_t cap, const char *src, const regex_t *regex) { + char *ptr = dst, *end = &dst[cap]; regmatch_t match[2]; assert(regex->re_nsub); for (; *src; src += match[0].rm_eo) { if (regexec(regex, src, 2, match, 0)) break; - len += strlcpyn(&dst[len], src, cap - len, match[0].rm_so); - if (len >= cap) return NULL; - len += strlcpyn( - &dst[len], &src[match[1].rm_so], - cap - len, match[1].rm_eo - match[1].rm_so + ptr = seprintf( + ptr, end, "%.*s%.*s", + (int)match[0].rm_so, src, + (int)(match[1].rm_eo - match[1].rm_so), &src[match[1].rm_so] ); - if (len >= cap) return NULL; } - len += snprintf(&dst[len], cap - len, "%s", src); - return (len < cap ? dst : NULL); + ptr = seprintf(ptr, end, "%s", src); + return (ptr == end ? NULL : dst); } static regex_t *compile(regex_t *regex, const char *pattern) { |