summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Disseldorp <ddiss@samba.org>2012-02-15 16:30:27 +0100
committerDavid Disseldorp <ddiss@samba.org>2012-02-15 19:34:38 +0100
commita6bd7f3d574998419cac01b1f7b4fa2e1067bb7c (patch)
tree1a76003c28a4ffdae3725d5756b0970fc39e4efc
parent9e25361203155c25d5deee4acd1fce638808cdbf (diff)
downloadsamba-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.c2
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);