diff options
author | Pieter Noordhuis <pcnoordhuis@gmail.com> | 2013-07-10 21:12:46 -0700 |
---|---|---|
committer | antirez <antirez@gmail.com> | 2014-08-12 11:50:02 +0200 |
commit | cf85b5ba812070442c450b5dbc6c5e12a6c0819b (patch) | |
tree | 058e0639fa59adf9b457e0ad4e04046470bec0c1 | |
parent | 3426866e0a62926e62143b9ede8880e48e51a038 (diff) | |
download | redis-cf85b5ba812070442c450b5dbc6c5e12a6c0819b.tar.gz |
va_copy must be matched by va_end
Hat tip to @rfuchs. See: https://github.com/redis/hiredis/pull/178.
Fixes #1187
-rw-r--r-- | src/sds.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -388,6 +388,7 @@ sds sdscatvprintf(sds s, const char *fmt, va_list ap) { buf[buflen-2] = '\0'; va_copy(cpy,ap); vsnprintf(buf, buflen, fmt, cpy); + va_end(ap); if (buf[buflen-2] != '\0') { if (buf != staticbuf) zfree(buf); buflen *= 2; |