diff --git a/directory/xpcom/base/src/nsLDAPMessage.cpp b/directory/xpcom/base/src/nsLDAPMessage.cpp
index 7f2ecff8b3c..a5e57815d32 100644
--- a/directory/xpcom/base/src/nsLDAPMessage.cpp
+++ b/directory/xpcom/base/src/nsLDAPMessage.cpp
@@ -220,7 +220,8 @@ nsLDAPMessage::NextAttribute(char* *aAttribute)
 	if (lderr == LDAP_SUCCESS) {
 	    return NS_OK;
 	} else {
-	    // XXX should really propagate lderr upwards
+	    // XXX should really propagate lderr & associated strings upwards..
+	    // maybe need to turn LdErrno into an interface
 	    //
 	    return NS_ERROR_FAILURE;
 	}