diff options
author | Tim Potter <tpot@samba.org> | 2008-01-27 17:31:56 +1100 |
---|---|---|
committer | Volker Lendecke <vl@samba.org> | 2008-01-27 09:33:42 +0100 |
commit | 54db1839878641be1a9987ad3e0ddedbd6123b7c (patch) | |
tree | 392506abfd7346b5d0960c9836382c5e24c4c7a2 /source3/lib/dprintf.c | |
parent | 47e1251f721cfbc45ab26b15d5bdbc523c55d6e9 (diff) | |
download | samba-54db1839878641be1a9987ad3e0ddedbd6123b7c.tar.gz samba-54db1839878641be1a9987ad3e0ddedbd6123b7c.tar.bz2 samba-54db1839878641be1a9987ad3e0ddedbd6123b7c.zip |
Adding missing calls to va_end().
Just a small commit to get a handle on this git thingy. This patch
fixes some missing calls to va_end() to match various calls to va_start()
and VA_COPY().
Tim.
(This used to be commit ec367f307dff7948722b9ac97beb960efd91991f)
Diffstat (limited to 'source3/lib/dprintf.c')
-rw-r--r-- | source3/lib/dprintf.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/source3/lib/dprintf.c b/source3/lib/dprintf.c index 18b261623e..a3bb5be43a 100644 --- a/source3/lib/dprintf.c +++ b/source3/lib/dprintf.c @@ -47,7 +47,10 @@ lang_msg_free(msgstr); - if (ret <= 0) return ret; + if (ret <= 0) { + va_end(ap2); + return ret; + } /* now we have the string in unix format, convert it to the display charset, but beware of it growing */ @@ -56,6 +59,7 @@ again: p2 = (char *)SMB_MALLOC(maxlen); if (!p2) { SAFE_FREE(p); + va_end(ap2); return -1; } clen = convert_string(CH_UNIX, CH_DISPLAY, p, ret, p2, maxlen, True); @@ -72,6 +76,8 @@ again: ret = fwrite(p2, 1, clen, f); SAFE_FREE(p2); + va_end(ap2); + return ret; } |