summaryrefslogtreecommitdiffstats
path: root/source/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
commitec367f307dff7948722b9ac97beb960efd91991f (patch)
treeba2900c2d7e543c68ac9a7420b4ea89a1b4652b0 /source/printing
parent147033b9dff7160ff324fa52aba1f054f962961b (diff)
downloadsamba-ec367f307dff7948722b9ac97beb960efd91991f.tar.gz
samba-ec367f307dff7948722b9ac97beb960efd91991f.tar.xz
samba-ec367f307dff7948722b9ac97beb960efd91991f.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.
Diffstat (limited to 'source/printing')
-rw-r--r--source/printing/print_generic.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/source/printing/print_generic.c b/source/printing/print_generic.c
index cc4b744a11c..2a324fdd5c3 100644
--- a/source/printing/print_generic.c
+++ b/source/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;
}
}