On Thu, Apr 21, 2011 at 10:36 AM, sim123 <[email protected]> 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.
*
*