diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2010-11-23 15:00:52 +0000 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2010-11-23 15:00:52 +0000 |
commit | b2594b047bf7c7c0ea71364ae1665228290e873b (patch) | |
tree | a96aaf3e59a75d9b558c1de0f6c2cc917c306307 /libdm/libdm-string.c | |
parent | 728074ac8356eee9436ea1c36b89173863f17560 (diff) | |
download | lvm2-b2594b047bf7c7c0ea71364ae1665228290e873b.tar.gz |
Move va_end(ap) so we do not leave with return -1 without calling it.
Remove unneeded ';'
Diffstat (limited to 'libdm/libdm-string.c')
-rw-r--r-- | libdm/libdm-string.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/libdm/libdm-string.c b/libdm/libdm-string.c index d8fca1a90..365e7ec0d 100644 --- a/libdm/libdm-string.c +++ b/libdm/libdm-string.c @@ -144,6 +144,8 @@ int dm_asprintf(char **result, const char *format, ...) while (!ok) { va_start(ap, format); n = vsnprintf(buf, size, format, ap); + va_end(ap); + if (0 <= n && n < size) ok = 1; else { @@ -152,8 +154,7 @@ int dm_asprintf(char **result, const char *format, ...) buf = dm_malloc(size); if (!buf) return -1; - }; - va_end(ap); + } } *result = dm_strdup(buf); |