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/printing | |
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/printing')
-rw-r--r-- | source3/printing/print_generic.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/source3/printing/print_generic.c b/source3/printing/print_generic.c index cc4b744a11..2a324fdd5c 100644 --- a/source3/printing/print_generic.c +++ b/source3/printing/print_generic.c @@ -41,15 +41,18 @@ static int print_run_command(int snum, const char* printername, bool do_sub, /* check for a valid system printername and valid command to run */ if ( !printername || !*printername ) { + va_end(ap); return -1; } if (!command || !*command) { + va_end(ap); return -1; } syscmd = talloc_strdup(ctx, command); if (!syscmd) { + va_end(ap); return -1; } @@ -57,6 +60,7 @@ static int print_run_command(int snum, const char* printername, bool do_sub, char *value = va_arg(ap,char *); syscmd = talloc_string_sub(ctx, syscmd, arg, value); if (!syscmd) { + va_end(ap); return -1; } } |