Issue 8885 - hdb_generate_key_set_password prototype check for smbk5pwd
Summary: hdb_generate_key_set_password prototype check for smbk5pwd
Status: VERIFIED FIXED
Alias: None
Product: OpenLDAP
Classification: Unclassified
Component: slapd (show other issues)
Version: unspecified
Hardware: All All
: --- normal
Target Milestone: ---
Assignee: OpenLDAP project
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-30 01:44 UTC by manu@openldap.org
Modified: 2020-02-06 21:18 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 manu@openldap.org 2018-07-30 01:44:31 UTC
Full_Name: Emmanuel Dreyfus
Version: HEAD
OS: NetBSD
URL: ftp://ftp.openldap.org/incoming/manu-180729.patch
Submission from: (NULL) (213.41.141.172)


contrib/slapd-modules/smbk5pwd uses hdb_generate_key_set_password() from
Heimdal, which was shortly turned from a 5 arguments function to a 7 arguments
function before the prototype change was rolled back to address API
incompatibility.

Unfortunately, the 7 arguments hdb_generate_key_set_password() made it into
released NetBSD 8.0, causing a build break in contrib/slapd-modules/smbk5pwd.

The attached patch adds a configure test for 7 arguments prototype so that
contrib/slapd-modules/smbk5pwd build again on NetBSD 8.0, and other OS that
would include the 7 arguments hdb_generate_key_set_password(). 
Comment 1 Ryan Tandy 2018-07-30 01:53:03 UTC
On Mon, Jul 30, 2018 at 01:44:31AM +0000, manu@netbsd.org wrote:
>--- a/configure.in
>+++ b/configure.in
>@@ -1312,6 +1312,14 @@ if test $ol_enable_lmpasswd != no; then
> 	AC_DEFINE(SLAPD_LMHASH, 1, [define to support LAN Manager passwords])
> fi
> 
>+dnl ----------------------------------------------------------------
>+dnl Threads?

Copypasta from the paragraph below? Looks off topic for this hunk.

Comment 2 manu@openldap.org 2018-07-30 02:13:01 UTC
Ryan Tandy <ryan@nardis.ca> wrote:

> Copypasta from the paragraph below? Looks off topic for this hunk.

I just updated the patch.

-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
manu@netbsd.org

Comment 3 OpenLDAP project 2020-02-06 21:18:22 UTC
Heimdal fixed on their end
Comment 4 Quanah Gibson-Mount 2020-02-06 21:18:22 UTC
changed notes
changed state Open to Closed