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

RE: commit: ldap/servers/slapd sasl.c



> -----Original Message-----
> From: owner-openldap-devel@OpenLDAP.org
> [mailto:owner-openldap-devel@OpenLDAP.org]On Behalf Of Pierangelo Masarati

> Hallvard B Furuseth wrote:
> >ando@OpenLDAP.org writes:
> >>clarify auxprop_store member type
> >>
> >>    http://www.openldap.org/devel/cvsweb.cgi/servers/slapd/sasl.c
> >>
> >>-       slap_auxprop_store
> >>+       slap_auxprop_store      /* the declaration of this member changed
> >>+                                * in cyrus SASL from 2.1.15 to 2.1.16 */

> >Looks like the code needs an #ifdef for SASL < 2.1.16.

> Exactly, but I didn't follow cyrus-sasl's evolution
> with auxprops enough to know how to deal with
> earlier versions.  I noticed the problem with 2.1.15,
> where the field is declared as void (*spare_fptr1)(),
> which results in a warning; in 2.1.16 beta it's
> declared as
>
> int (*auxprop_store)(void *glob_context,
>                          sasl_server_params_t *sparams,
>                          struct propctx *ctx,
>                          const char *user, unsigned ulen);
>
> note that the structure members have different
> names in the two versions!
>
> Personally, I can live with the warning until
> both OL 2.2 and cyrus 2.1 stabilize :)

I didn't think it was necessary to #ifdef it since the pointer is
unused/ignored in 2.1.15 and older. I.e., the auxprop_ function itself is
useless there, but I figured its presence is harmless. But if you think it
should be #if'd, that's fine.

  -- Howard Chu
  Chief Architect, Symas Corp.       Director, Highland Sun
  http://www.symas.com               http://highlandsun.com/hyc
  Symas: Premier OpenSource Development and Support