[fedfs-utils] [PATCH 06/13] libnsdb: Fix nsdb_search_nsdb_attr_s() auditing message

Chuck Lever chuck.lever at oracle.com
Fri Feb 1 09:20:36 PST 2013


The xlog() call sites in __nsdb_search_nsdb_attr_s() are missing
an argument.  Neither gcc nor sparse caught this; I discovered it
during operation.

Introduced by commit 5e94ae03 "libnsdb: Add debugging message near
ldap_search_ext_s(3) call sites", Wed Nov 28 20:35:24 2012.

Signed-off-by: Chuck Lever <chuck.lever at oracle.com>
---

 src/libnsdb/fileserver.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/libnsdb/fileserver.c b/src/libnsdb/fileserver.c
index b08ce5c..ee339b0 100644
--- a/src/libnsdb/fileserver.c
+++ b/src/libnsdb/fileserver.c
@@ -111,11 +111,11 @@ __nsdb_search_nsdb_attr_s(const char *func, LDAP *ld, const char *base,
 
 	if (ldap_get_option(ld, LDAP_OPT_URI, &uri) == LDAP_OPT_SUCCESS) {
 		xlog(D_CALL, "%s:\n  ldapsearch -H %s -b \"%s\" -s base '%s' %s",
-			func, uri, filter, attr);
+			func, uri, base, filter, attr);
 		ldap_memfree(uri);
 	} else {
 		xlog(D_CALL, "%s:\n  ldapsearch -b \"%s\" -s base '%s' %s",
-			func, filter, attr);
+			func, base, filter, attr);
 	}
 
 	return ldap_search_ext_s(ld, (char *)base, LDAP_SCOPE_BASE, filter,




More information about the fedfs-utils-devel mailing list