[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#4038) crash in slapi_entry_attr_set_charptr

On Thu, 2005-09-22 at 05:01 +0000, andrewb@cse.unsw.edu.au wrote:

>         /* check if the entry contains the "gecos" attribute */
>         if (entry == NULL || slapi_entry_attr_find(entry, "gecos", &attr) != 0)
>                 return 0;

If the intention of this bit of code is to skip entries that already
contain the "gecos" attribute then it's incorrect, since reading the
code it appears that slapi_entry_attr_find() returns 0 if the requested
attribute is __present__.  The subsequent SIGSEGV may be related to
further problems in SLAPI code or be a result of this initial error, I
don't have enough knowledge of SLAPI to tell.


    SysNet - via Dossi,8 27100 Pavia Tel: +390382573859 Fax: +390382476497