diff options
| author | Andrew Kroeger <andrew@id10ts.net> | 2009-06-11 02:15:28 -0500 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2009-06-12 10:29:37 +0200 |
| commit | 6c5c99130462ecddb449e99138c88930003325a3 (patch) | |
| tree | 5349bba1ad99cc609da76296d568da7cada27264 /source3/lib/ldb/ldb_sqlite3/ldb_sqlite3.c | |
| parent | ff4acca13aa53865182bd26715c92e45f6e8f861 (diff) | |
| download | samba-6c5c99130462ecddb449e99138c88930003325a3.tar.gz samba-6c5c99130462ecddb449e99138c88930003325a3.tar.bz2 samba-6c5c99130462ecddb449e99138c88930003325a3.zip | |
s3: Call va_end() after all va_start()/va_copy() calls.
There are error paths in S3 where va_end() is not properly called after
va_start() or va_copy() have been called.
These issues were noted while performing an inspection for S4 bug #6129. Thanks
to Erik Hovland <erik@hovland.org> for the original bug report.
Diffstat (limited to 'source3/lib/ldb/ldb_sqlite3/ldb_sqlite3.c')
| -rw-r--r-- | source3/lib/ldb/ldb_sqlite3/ldb_sqlite3.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/source3/lib/ldb/ldb_sqlite3/ldb_sqlite3.c b/source3/lib/ldb/ldb_sqlite3/ldb_sqlite3.c index cb516b6e75..d8fc162741 100644 --- a/source3/lib/ldb/ldb_sqlite3/ldb_sqlite3.c +++ b/source3/lib/ldb/ldb_sqlite3/ldb_sqlite3.c @@ -534,6 +534,7 @@ query_int(const struct lsqlite3_private * lsqlite3, /* Format the query */ if ((p = sqlite3_vmprintf(pSql, args)) == NULL) { + va_end(args); return SQLITE_NOMEM; } |