Issue 5366 - ldap_get_dn man page errors
Summary: ldap_get_dn man page errors
Status: VERIFIED FIXED
Alias: None
Product: OpenLDAP
Classification: Unclassified
Component: documentation (show other issues)
Version: 2.4.7
Hardware: All All
: --- normal
Target Milestone: ---
Assignee: OpenLDAP project
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-10 19:58 UTC by rra@debian.org
Modified: 2014-08-01 21:04 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description rra@debian.org 2008-02-10 19:58:22 UTC
Full_Name: Russ Allbery
Version: 2.4.7
OS: Debian GNU/Linux
URL: 
Submission from: (NULL) (171.66.157.14)


The ldap_get_dn man page has the following prototype for ldap_str2dn:

int ldap_str2dn( const char *str, LDAPDN **dn, unsigned flags )

However, ldap.h has:

LDAP_F( int )
ldap_str2dn LDAP_P((
        LDAP_CONST char *str,
        LDAPDN *dn,
        unsigned flags ));

Note the difference for the second argument.   This is related to, later in the
man page:

       typedef struct ldap_ava {
           char *la_attr;
           struct berval *la_value;
           unsigned la_flags;
       } LDAPAVA;

       typedef LDAPAVA** LDAPRDN;
       typedef LDAPRDN** LDAPDN;

but ldap.h says:

typedef struct ldap_ava {
        struct berval la_attr;
        struct berval la_value;
        unsigned la_flags;
        void *la_private;
} LDAPAVA;

typedef LDAPAVA** LDAPRDN;
typedef LDAPRDN* LDAPDN;

Note the type difference for LDAPDN (which explains the earlier varience) and
for la_attr.

Original Debian bug report: http://bugs.debian.org/465024

Comment 1 Hallvard Furuseth 2008-02-10 21:24:37 UTC
moved from Incoming to Documentation
Comment 2 Gavin Henry 2008-02-19 21:32:59 UTC
Should I change code man pages?
Comment 3 Gavin Henry 2008-04-16 08:15:30 UTC
changed state Open to Test
Comment 4 peter@adpm.de 2009-03-22 17:56:57 UTC
Hi,

I filed a patch that should fix this ITS.
It can be found at 
  ftp://ftp.openldap.org/incoming/PeterMarschall-090322.patch
and brings the man page in line with the code.

Regards
Peter
-- 
Peter Marschall
peter@adpm.de

Comment 5 Gavin Henry 2009-10-07 11:15:02 UTC
changed notes
Comment 6 Quanah Gibson-Mount 2009-10-30 17:57:44 UTC
changed notes
changed state Test to Release
Comment 7 Howard Chu 2009-11-30 05:44:49 UTC
changed notes
changed state Release to Closed
Comment 8 OpenLDAP project 2014-08-01 21:04:10 UTC
FIXED in HEAD
Fixed in RE24