[Nagiosplug-devel] [ nagiosplug-Bugs-1985489 ] check_ldap doesn't allow empty bases
SourceForge.net
noreply at sourceforge.net
Mon Apr 12 17:18:35 CEST 2010
Bugs item #1985489, was opened at 2008-06-05 16:12
Message generated for change (Comment added) made by hweiss
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1985489&group_id=29880
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Argument proccessing
Group: None
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Jan Wagner (cyco_dd)
Assigned to: Nobody/Anonymous (nobody)
Summary: check_ldap doesn't allow empty bases
Initial Comment:
The following Bugreport we got against our debian package:
Hiya,
$ /usr/lib/nagios/plugins/check_ldap -H localhost -b ''
check_ldap: Please specify the LDAP base
doesn't work.
For me, a check with an empty base seems like the most obvious
thing to do.
So I don't understand why it is explicitely disallowed in the
code (note that the error message is also confusing)
A simple fix:
--- plugins/check_ldap.c-before-base-fix 2008-05-07 11:32:13.000000000 +0100
+++ plugins/check_ldap.c 2008-05-07 11:34:50.000000000 +0100
@@ -374,7 +374,7 @@ validate_arguments ()
if (ld_host==NULL || strlen(ld_host)==0)
usage4 (_("Please specify the host name\n"));
- if (ld_base==NULL || strlen(ld_base)==0)
+ if (ld_base == NULL)
usage4 (_("Please specify the LDAP base\n"));
return OK;
The similar check for strlen(ld_host) could be removed as well,
because if ld_host is "", it's because the user specified -H '',
so did specify the hostname, so that the error message is
misleading.
You can track the bugreport via http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=479984
Thanks and kind regards, Jan.
----------------------------------------------------------------------
>Comment By: Holger Weiss (hweiss)
Date: 2010-04-12 17:18
Message:
I committed the patch.
Thank you!
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1985489&group_id=29880
More information about the Devel
mailing list