summaryrefslogtreecommitdiff
path: root/source3/printing
diff options
context:
space:
mode:
authorDavid Disseldorp <ddiss@suse.de>2010-12-20 16:08:02 +0100
committerAndreas Schneider <asn@samba.org>2010-12-20 18:44:41 +0100
commit139e2cbb13ca230b65ad4b1f5ff91a795d8cca02 (patch)
treeeb18ed0458f8b314396374c55e2ea9d2af13dba4 /source3/printing
parentbfc4fe401259085f9e17c75a7ec171be0bc5f35f (diff)
downloadsamba-139e2cbb13ca230b65ad4b1f5ff91a795d8cca02.tar.gz
samba-139e2cbb13ca230b65ad4b1f5ff91a795d8cca02.tar.bz2
samba-139e2cbb13ca230b65ad4b1f5ff91a795d8cca02.zip
s3-printing: fix printer_list_traverse()
The tdb traverse function returns the number of elements traversed, or less than zero on error, printer_list_traverse() is incorrectly checking for non-zero return. Autobuild-User: Andreas Schneider <asn@samba.org> Autobuild-Date: Mon Dec 20 18:44:41 CET 2010 on sn-devel-104
Diffstat (limited to 'source3/printing')
-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 edc398ba77..f3f00f0c0f 100644
--- a/source3/printing/printer_list.c
+++ b/source3/printing/printer_list.c
@@ -293,7 +293,7 @@ static NTSTATUS printer_list_traverse(printer_list_trv_fn_t *fn,
}
ret = db->traverse(db, fn, private_data);
- if (ret != 0) {
+ if (ret < 0) {
return NT_STATUS_UNSUCCESSFUL;
}