s3:libads: Don’t do first loop iteration if ‘attr’ is NULL

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>

diff --git a/source3/libads/ldap.c b/source3/libads/ldap.c
index 3f93b06f984af2d94e263d6bac8b0b47e8c9c5e4..a639cd23dc429537e5c0067c420da3b79eda2a0b 100644 (file)
--- a/source3/libads/ldap.c
+++ b/source3/libads/ldap.c
@@ -3176,6 +3176,10 @@ int ads_count_replies(ADS_STRUCT *ads, void *res)
        for (attr = ldap_first_attribute(ads->ldap.ld, (LDAPMessage *)msg, &ptr);
             attr;
             attr = ldap_next_attribute(ads->ldap.ld, (LDAPMessage *)msg, ptr)) {
+               if (attr == NULL) {
+                       break;
+               }
+
                /* we ignore the fact that this is utf8, as all attributes are ascii... */
                if (strnequal(attr, expected_range_attrib, strlen(expected_range_attrib))) {
                        range_attr = attr;