diff options
| author | David Disseldorp <ddiss@suse.de> | 2010-12-20 16:08:02 +0100 |
|---|---|---|
| committer | Andreas Schneider <asn@samba.org> | 2010-12-20 18:44:41 +0100 |
| commit | 139e2cbb13ca230b65ad4b1f5ff91a795d8cca02 (patch) | |
| tree | eb18ed0458f8b314396374c55e2ea9d2af13dba4 /source3/printing | |
| parent | bfc4fe401259085f9e17c75a7ec171be0bc5f35f (diff) | |
| download | samba-139e2cbb13ca230b65ad4b1f5ff91a795d8cca02.tar.gz samba-139e2cbb13ca230b65ad4b1f5ff91a795d8cca02.tar.xz 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.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/printing/printer_list.c b/source3/printing/printer_list.c index edc398ba771..f3f00f0c0f5 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; } |