[fedfs-utils] [PATCH 13/13] libjunction: Always list the NSDB port attribute in junction XML

[Chuck Lever]

Don't depend on the current default behavior of mapping a zero or
missing nsdbport value to 389.

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

 src/libjunction/fedfs.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/libjunction/fedfs.c b/src/libjunction/fedfs.c
index a06298a..f86ce5c 100644
--- a/src/libjunction/fedfs.c
+++ b/src/libjunction/fedfs.c
@@ -140,10 +140,10 @@ fedfs_name_xml(const char *pathname, xmlNodePtr fileset,
 
 	xmlSetProp(new, FEDFS_XML_FSN_UUID_ATTR, (const xmlChar *)fsn_uuid);
 	xmlSetProp(new, FEDFS_XML_FSN_NSDBNAME_ATTR, (const xmlChar *)nsdb_name);
-	if (nsdb_port != LDAP_PORT)
-		junction_xml_set_int_attribute(new,
-						FEDFS_XML_FSN_NSDBPORT_ATTR,
-						nsdb_port);
+	if (nsdb_port == 0)
+		nsdb_port = LDAP_PORT;
+	junction_xml_set_int_attribute(new, FEDFS_XML_FSN_NSDBPORT_ATTR,
+								nsdb_port);
 
 	return FEDFS_OK;
 }