[evolution-patches] Re: pls review patch for bug 50583

["leon.zhang" <leon zhang sun com>]

patch for bug 50583 has been checked into branch


leon


Chris Toshok wrote:
> Looks good to me.
>
> Please commit to the branch - HEAD no longer has the cursor stuff, and
> so doesn't have this bug.
>
> actually, it looks like HEAD ldap is missing the get_contact_list stuff
> altogether.  whee.
>
> Chris
>
> On Tue, 2003-11-04 at 23:24, leon.zhang wrote:
>
> > hi, toshok
> >
> > Attachment is a patch for bug 50583, which will lead to hang of evolution.
> >
> > Steps to reproduce:
> > 1.Setup a LDAP contacts(Sun-ds.ebay.sun.com)
> > 2.Set this LDAP caontacts to default contacts
> > folder(Top Menu->Tools->Settings->Folder Settings->Default Folders->Contacts)
> > 3.Create a new meeting in local calendar
> > 4.Input any user name and press Enter in "Meeting" TAB page
> > 5.Click "Save" button
> >
> > Actual results:
> > after #4,evolution-wombat process eat over 97% CPU
> >
> >
> > Cause of this bug:
> > There is a infinite cycles in function:
> > get_cursor_handler() in pas-backend-ldap.c.
> > we should call ldap_next_entry() in each cycle.