diff options
author | David Disseldorp <ddiss@samba.org> | 2012-02-15 16:30:27 +0100 |
---|---|---|
committer | David Disseldorp <ddiss@samba.org> | 2012-02-15 19:34:38 +0100 |
commit | a6bd7f3d574998419cac01b1f7b4fa2e1067bb7c (patch) | |
tree | 1a76003c28a4ffdae3725d5756b0970fc39e4efc | |
parent | 9e25361203155c25d5deee4acd1fce638808cdbf (diff) | |
download | samba-a6bd7f3d574998419cac01b1f7b4fa2e1067bb7c.tar.gz samba-a6bd7f3d574998419cac01b1f7b4fa2e1067bb7c.tar.xz samba-a6bd7f3d574998419cac01b1f7b4fa2e1067bb7c.zip |
s3-printing: fix crash in printer_list_set_printer()
The printer list database format was recently changed to accommodate for
the printcap location field.
One of the tdb_pack calls is not provided with a location string
argument, this causes a crash on some platforms.
https://bugzilla.samba.org/show_bug.cgi?id=8762
Signed-off-by: Günther Deschner <gd@samba.org>
Signed-off-by: Jim McDonough <jmcd@samba.org>
Signed-off-by: Lars Müller <lars@samba.org>
Autobuild-User: David Disseldorp <ddiss@samba.org>
Autobuild-Date: Wed Feb 15 19:34:38 CET 2012 on sn-devel-104
-rw-r--r-- | source3/printing/printer_list.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/printing/printer_list.c b/source3/printing/printer_list.c index d9e87376606..7079ddc037a 100644 --- a/source3/printing/printer_list.c +++ b/source3/printing/printer_list.c @@ -191,7 +191,7 @@ NTSTATUS printer_list_set_printer(TALLOC_CTX *mem_ctx, data.dsize = len; len = tdb_pack(data.dptr, data.dsize, - PL_DATA_FORMAT, time_h, time_l, name, str); + PL_DATA_FORMAT, time_h, time_l, name, str, str2); status = dbwrap_store_bystring_upper(db, key, data, TDB_REPLACE); |