Re: parsing output from ldap_search_ext_s C API

On Thu, Apr 21, 2011 at 10:36 AM, sim123 Sim3159@gmail.com wrote:

Hi All,

I am using ldap_search_ext_s C API and having strange results, not sure if I am missing something or its an API bug:

My Directory Tree looks like this:

| -- dc=example,dc=com | ------ ou=users,dc=example,dc=com | ---------- uid=1,ou=users,dc=example,dc=com | --------------- cn=john | ------ ou=departments,dc=example,dc=com | ---------- uid=11,ou=departments,dc=example,dc=com | --------------- cn=hr | --------------- member=uid=1 | ---------- uid=12,ou=departments,dc=example,dc=com | --------------- cn=sales | --------------- member=uid=1 | ---------- uid=13,ou=departments,dc=example,dc=com | --------------- cn=marketing | --------------- member=uid=1

First I get all the departments for "john" by using *memberof* in sarch attribute. for this query my search filter contains only one criteria ("uid=1") Then I construct another search filter for getting names of all the departments john belongs to : (|(uid=11)(uid=12)(uid=13)) I pass this filter to ldap_search_ext_s, where base is "ou=departments,dc=example,dc=com", scope is one level down, and I want "cn" in the attribute

LDAPMessage* output; int retCode = ldap_search_ext_s(ld,base.c_str(), scope,filter.c_str(),attrs,false,NULL,NULL,NULL,0,&output); if(retCode == LDAP_SUCCESS){ //log success //send result to a static method for parsing LDAPUtil::parseResult(ld,output,result); ldap_msgfree(output); }else{ //log error & throw exception }

LDAPUtil parse result implementation

void LDAPUtil::parseResult(LDAP* ld, LDAPMessage* ldapResponse, LDAPSearchResult* parsedResult){

int numEntries = ldap_count_entries(ld,ldapResponse);
cout << "number of entries" << numEntries << endl;* // I get 3 here*
if(numEntries > 0){
    //parse result
    LDAPMessage * entry;
    BerElement * ber;
    char * attr;
    BerVarray* vals ;
    map<string,LDAPAttribute*> attributeValueMap ;
    int count = 0;
  *  //If I don't use count < numEntries this loop becomes an infinite

loop, this loop runs 3 times, however the dn value output is:* *// run 1 :: dn: uid=11,ou=departments,dc=examples,dc=com //run 2 :: dn: uid=12,ou=departments,dc=examples,dc=com // run 3 :: dn: uid=12,ou=departments,dc=examples,dc=com*

    for ( entry = ldap_first_entry(ld,ldapResponse);entry != NULL &&

count < numEntries; entry = ldap_next_entry(ld,ldapResponse)){ count++;

            //create LDAP Attributes
            LDAPAttribute* attribute = new LDAPAttribute();
            //set DN
            attribute->setDn(ldap_get_dn(ld,entry));
            cout << "dn is " << ldap_get_dn(ld,entry) <<endl;

            for(attr = ldap_first_attribute(ld,entry,&ber);
                    attr != NULL;

attr=ldap_next_attribute(ld,entry,ber)) { string temp = attr; cout << "attribute :: " << attr << endl; vals = ldap_get_values_len(ld,entry,attr); if((ldap_count_values_len(vals))> 0 ){ LDAPUtil::processAttribute(attribute,temp,vals); } ldap_value_free_len(vals); }

attributeValueMap.insert(pair<string,LDAPAttribute*>(attribute->getDn(),attribute)); } parsedResult->setAttributeValueMap(attributeValueMap); } }

Basically above code is working only if I have one entry returned in the output. I would really appreciate if someone can help me with this. As I have hard time beliving its an API bug since I am just doing basic operation.

Thanks,

• Simon

I guess I was right, it my code who had bug not the API :) I need to do this :

for ( entry = ldap_first_entry(ld,ldapResponse);entry != NULL && count < numEntries; * entry = ldap_next_entry(ld,entry)*)

instead of

for ( entry = ldap_first_entry(ld,ldapResponse);entry != NULL && count < numEntries; * entry = ldap_next_entry(ld,ldapResponse))

*thanks for the help. * *