summaryrefslogtreecommitdiff
path: root/source3/printing
diff options
context:
space:
mode:
authorTim Potter <tpot@samba.org>2008-01-27 17:31:56 +1100
committerVolker Lendecke <vl@samba.org>2008-01-27 09:33:42 +0100
commit54db1839878641be1a9987ad3e0ddedbd6123b7c (patch)
tree392506abfd7346b5d0960c9836382c5e24c4c7a2 /source3/printing
parent47e1251f721cfbc45ab26b15d5bdbc523c55d6e9 (diff)
downloadsamba-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.c4
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;
}
}