https://bugs.openldap.org/show_bug.cgi?id=10460
Issue ID: 10460 Summary: Stray uses of fprintf in libldap Product: OpenLDAP Version: 2.6.12 Hardware: All OS: All Status: UNCONFIRMED Keywords: needs_review Severity: minor Priority: --- Component: libraries Assignee: bugs@openldap.org Reporter: hyc@openldap.org Target Milestone: ---
avl.c, cyrus.c, error.c, ldap_sync.c, ldifutil.c contain uses of `fprintf(stderr,...)` which should probably use Debug instead.
https://bugs.openldap.org/show_bug.cgi?id=10460
--- Comment #1 from Howard Chu hyc@openldap.org --- In error.c it's just in the deprecated ldap_perror(), we can probably leave that as-is.
https://bugs.openldap.org/show_bug.cgi?id=10460
--- Comment #2 from Howard Chu hyc@openldap.org --- In cyrus.c it's in ldap_int_sasl_bind which is only called from ldap_sasl_interactive_bind. These would presumably only be used in clients, but it's called multiple places in slapd, including back-asyncmeta, back-ldap, and back-meta. Each of those callers should be providing all the required inputs such that no prompts are ever displayed, so the fprintf's should never actually get invoked. So does this need to be changed?
https://bugs.openldap.org/show_bug.cgi?id=10460
--- Comment #3 from Howard Chu hyc@openldap.org --- In ldap_sync.c, all uses are #ifdef LDAP_SYNC_TRACE, which is not defined anywhere in the source tree. I guess we can ignore this.
https://bugs.openldap.org/show_bug.cgi?id=10460
--- Comment #4 from Howard Chu hyc@openldap.org --- thr_debug.c also uses fprintf(stderr) but I guess it would only be used in an explicit debugging session, so can probably leave that too.
https://bugs.openldap.org/show_bug.cgi?id=10460
Quanah Gibson-Mount quanah@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords|needs_review | Target Milestone|--- |2.7.0 Assignee|bugs@openldap.org |hyc@openldap.org
https://bugs.openldap.org/show_bug.cgi?id=10460
Howard Chu hyc@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |IN_PROGRESS Ever confirmed|0 |1
--- Comment #5 from Howard Chu hyc@openldap.org --- https://git.openldap.org/openldap/openldap/-/merge_requests/852
Just changed ldifutil, ignoring the rest.
https://bugs.openldap.org/show_bug.cgi?id=10460
Quanah Gibson-Mount quanah@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |TEST Status|IN_PROGRESS |RESOLVED
--- Comment #6 from Quanah Gibson-Mount quanah@openldap.org --- • cb38aa58 by Howard Chu at 2026-04-07T19:53:54+00:00 ITS#10460 libldap: don't use fprintf(stderr) in ldifutil
https://bugs.openldap.org/show_bug.cgi?id=10460
Quanah Gibson-Mount quanah@openldap.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |VERIFIED Resolution|TEST |FIXED
-
openldap-its＠openldap.org