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

RE: Untested patch: back-tcl used wrong types (ITS#1719)



The nnewRdn and nnewSuperior arguments are the normalized version of the
newRdn and newSuperior. back-perl apparently doesn't care about
normalization. I've never set up back-tcl and I have no idea what it needs
to do. It's been unsupported since about when I joined the team, I think...

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

> -----Original Message-----
> From: owner-openldap-bugs@OpenLDAP.org
> [mailto:owner-openldap-bugs@OpenLDAP.org]On Behalf Of
> h.b.furuseth@usit.uio.no
> Sent: Wednesday, April 03, 2002 12:37 AM
> To: openldap-its@OpenLDAP.org
> Subject: Untested patch: back-tcl used wrong types (ITS#1719)
>
>
> Full_Name: Hallvard B. Furuseth
> Version: HEAD (2002-04-02)
> OS:
> URL: http://folk.uio.no/hbf/OpenLDAP/tcl.txt
> Submission from: (NULL) (158.36.148.34)
>
>
> This patch is untested (except for compiling it); a compiler bug
> prevented me from compiling it completely.
>
> back-tcl is outdated:
> - It expected be->be_suffix to be a char**, not a struct berval**.
> - It expected the string arguments to the tcl_back_*() routines
>   to be const char*, not struct berval.
> - tcl_back_compare() expected an Ava* instead of AttributeAssertion*.
> - tcl_back_modify() expected an LDAPModList* instead of Modifications*.
> - tcl_back_modrdn() expected too few arguments.
> - And finally, back-tcl did printf(%ld, ber_int_t) instead of
>   printf(%ld, (long) ber_int_t).
>
> Note: This patch merely ignores the two new arguments nnewrdn and
> nnewSuperior to tcl_back_modrdn(), because that's what back-perl does.
> I have no idea what these arguments do.  Whoever invented them should
> have a look at tcl_modrdn.c.  (And maybe back-perl/modrdn.c too.)
>