samba-tool ldapcmp: use CommandError, not assertion

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>

diff --git a/python/samba/netcmd/ldapcmp.py b/python/samba/netcmd/ldapcmp.py
index dd34e0cf015ac9b6d14c88b0f1c823da2e12b073..5118e65ca98778c4b6b5b00ec18de0992de44d91 100644 (file)
--- a/python/samba/netcmd/ldapcmp.py
+++ b/python/samba/netcmd/ldapcmp.py
@@ -110,7 +110,10 @@ class LDAPBase(object):
     def find_netbios(self):
         res = self.ldb.search(base="CN=Partitions,%s" % self.config_dn,
                               scope=SCOPE_SUBTREE, attrs=["nETBIOSName"])
-        assert len(res) > 0
+
+        if len(res) == 0:
+            raise CommandError("Could not find netbios name")
+
         for x in res:
             if "nETBIOSName" in x:
                 return x["nETBIOSName"][0]