Full_Name: Ryan Tandy Version: master, 2.4 OS: Debian URL: Submission from: (NULL) (24.68.37.4) Submitted by: ryan This is sort of a follow-up to ITS#7871. ldif_sput gained an LDIF_KLUDGE, but ldif_put did not, so the latter now wraps at 76 columns instead of 78. (IMO ldif_sput and ldif_put could both just pass 0 for wrap instead of a specific number, and let ldif_sput_wrap set the default.) LDIF_SIZE_NEEDED still assumes wrapping to LDIF_LINE_WIDTH columns, so code that does LDIF_SIZE_NEEDED and then ldif_sput is now allocating (slightly) more memory than it actually needs. 389-ds does this, for example. (And actually, couldn't LDIF_SIZE_NEEDED just be a thinner wrapper around LDIF_SIZE_NEEDED_WRAP?) Not sure exactly what the policies are around API changes in exposed headers, but for 2.5, maybe LDIF_LINE_WIDTH could change to 78 and LDIF_KLUDGE could go away entirely? Should I take this part to -devel? BTW, is it still necessary for ldif.c to re-implement base64 instead of using the lutil functions?
changed notes
See also ITS#7859 base64 functions question needs resolution
changed notes moved from Incoming to Software Bugs
That sounds fine by me. BTW liblutil is currently a static-only library, which is why none of the libldap code uses it. Same issue I'm hitting with using AVL code in ITS#9001
Ondrej to do suggested patch. Larger issues around issue#9001 still need resolving separately.
Commits: • 56823973 by Ondřej Kuzník at 2021-03-03T10:37:19+00:00 ITS#8345 Remove LDIF_KLUDGE and defer defaults to _wrap()